On 09/22/14 at 05:02pm, Michael Holzheu wrote: > Hello Baoquan, > > I looked into your patches and tried to add s390x support. > > My naive approach was to just enable the is_vmalloc_addr() > for s390x: > > --- a/makedumpfile.h > +++ b/makedumpfile.h > @@ -814,13 +814,15 @@ unsigned long long vaddr_to_paddr_ppc(un > #endif /* powerpc32 */ > > #ifdef __s390x__ /* s390x */ > +int is_vmalloc_addr_s390x(ulong vaddr); > int get_machdep_info_s390x(void); > unsigned long long vaddr_to_paddr_s390x(unsigned long vaddr); > #define get_phys_base() TRUE > #define get_machdep_info() get_machdep_info_s390x() > #define get_versiondep_info() TRUE > #define vaddr_to_paddr(X) vaddr_to_paddr_s390x(X) > -#define is_vmalloc_addr(X) TRUE > +#define is_vmalloc_addr(X) is_vmalloc_addr_s390x(X) > #endif /* s390x */ > > #ifdef __ia64__ /* ia64 */ Hi Michael, Please alse provide a get_versiondep_info_s390x since page_offset is needed in set_kcore_vmcoreinfo() and other information need it too, such as VMALLOC_START/VMEMMAP_START/MODULES_VADDR, if you want to provide a is_vmalloc_addr_s390x before initial() is called. Thanks Baoquan