[PATCH] makedumpfile: Enable compressed dump formats for Xen

[Atsushi Kumagai]

Hello Petr,

>Hello all,
>
>this patch enables compressed file formats for Xen. I was suprized
>that nobody noticed earlier, given how trivial the change is. ;-)

Good catch.
Since there is no Xen dump in the kdump-compressed format, is it also
necessary to fix the crash ?

BTW, I think we should fix the manuals as well. There are some wrong
statements "-E option must be specified with this option" and
some examples implicate that -E option is necessary to Xen dump.


Thanks
Atsushi Kumagai

>Originally, the compressed KDUMP format was not sufficient for Xen
>Dom0 dumps, because it did not contain required Xen-specific
>information. However, all this info is stored in ELF notes, which
>have been present in the compressed file since version 4, so there
>is no longer any reason to restrict Xen dumps to ELF only.
>
>One minor fix is necessary. In a Xen dump, PFN refers to physical
>pages _inside_ Dom0, but when compressing the whole file, the machine
>pages are used in fact (although the variable is still called pfn).
>Consequently, readmem() must interpret the address as a machine
>address rather than Dom0 physical address when saving Xen files.
>
>Signed-off-by: Petr Tesarik <ptesarik at suse.com>
>
>diff --git a/makedumpfile.c b/makedumpfile.c
>index de40932..2636036 100644
>--- a/makedumpfile.c
>+++ b/makedumpfile.c
>@@ -6015,9 +6015,11 @@ int
> read_pfn(mdf_pfn_t pfn, unsigned char *buf)
> {
> 	unsigned long long paddr;
>+	int type_addr;
>
> 	paddr = pfn_to_paddr(pfn);
>-	if (!readmem(PADDR, paddr, buf, info->page_size)) {
>+	type_addr = is_xen_memory() ? MADDR_XEN : PADDR;
>+	if (!readmem(type_addr, paddr, buf, info->page_size)) {
> 		ERRMSG("Can't get the page data.\n");
> 		return FALSE;
> 	}
>@@ -7654,12 +7656,6 @@ initial_xen(void)
> 	MSG("Xen is not supported on powerpc.\n");
> 	return FALSE;
> #else
>-	if(!info->flag_elf_dumpfile && !info->flag_dmesg) {
>-		MSG("Specify '-E' option for Xen.\n");
>-		MSG("Commandline parameter is invalid.\n");
>-		MSG("Try `makedumpfile --help' for more information.\n");
>-		return FALSE;
>-	}
> #ifndef __x86_64__
> 	if (DL_EXCLUDE_ZERO < info->max_dump_level) {
> 		MSG("Dump_level is invalid. It should be 0 or 1.\n");