[Crash-utility] [PATCH] crash: Support a dumpfile in the flattened format.
Ken'ichi Ohmichi
oomichi at mxs.nes.nec.co.jp
Tue Feb 1 23:50:51 UTC 2011
Hi Dave,
On Tue, 1 Feb 2011 12:10:14 -0500 (EST)
Dave Anderson <anderson at redhat.com> wrote:
>
> I renamed the multi-purpose read_dump_file() function to be
> read_flattened_format(), and only call it when the dumpfile
> is a flat vmcore. For clarification/maintainability, I prefer
> to keep the original lseek/read calls in diskdump.c and netdump.c
> as they were, mainly because it's not intuitive to have the
> traditional netdump, kdump, and diskdump dumpfiles require an
> access function that is located in the new makedumpfile.c.
I also felt read_dump_file() was not a good name, but I did not
have a better one.
So all of your fixing is reasonable for maintainability, thanks.
Thanks
Ken'ichi Ohmichi
More information about the Crash-utility
mailing list