[Crash-utility] [PATCH v4] files: support dump file memory mapping
Dave Anderson
anderson at redhat.com
Thu Jul 2 19:22:44 UTC 2015
Oliver,
The patch is queued for crash-7.1.2:
https://github.com/crash-utility/crash/commit/3106fee2bebfdb0f1c850911f452824237598d92
Thanks,
Dave
----- Original Message -----
> Dave,
>
> I reviewed all you changes. It looked good to me.
>
> It seemed your patch had some trailing whitespace issues.
>
> Just FYI.
>
> $ git apply ~/files_v5.patch
> /home/oliver/files_v5.patch:35: trailing whitespace.
> " %s> files -c 1954",
> /home/oliver/files_v5.patch:259: trailing whitespace.
> count, nrpages);
> /home/oliver/files_v5.patch:362: trailing whitespace.
> mkstring(buf2, MAX(VADDR_PRLEN, strlen("I_MAPPING")),
> warning: 3 lines add whitespace errors.
>
>
>
>
>
> From: yang_oliver at hotmail.com
> To: crash-utility at redhat.com
> Date: Wed, 1 Jul 2015 00:00:12 +0000
> Subject: Re: [Crash-utility] [PATCH v4] files: support dump file memory
> mapping
>
>
>
> Date: Tue, 30 Jun 2015 17:02:53 -0400
> From: anderson at redhat.com
> To: crash-utility at redhat.com
> Subject: Re: [Crash-utility] [PATCH v4] files: support dump file memory
> mapping
>
>
>
> ----- Original Message -----
> > >
> > > So in the interest of expediency, what I will do is this:
> > >
> > > (1) change "files -m" to "files -c"
> > > (2) drop the MAPPING column from "files -p"
> > > (3) reword the description of the two options in the help page to
> > > emphasize
> > > that the NRPAGES count and page dumps are page cache counts/page-dumps
> > > (4) either figure out a way to compress the help page example outputs
> > > into
> > > 80 columns, or drop the files -p example completely
> > >
> > > I'll post the patch this afternoon and you can verify it tonight.
> >
> > Sure, I will. Thanks for your helps and comments.
> > I hope I can do better for my next patch. :-)
> >
> >
>
> Hi Oliver,
>
> I've attached the patch for your review. In addition to the items listed
> above, I also made a few other minor changes to better handle some error
> conditions I ran into while testing the patch, I renamed a few functions and
> macros to be more related to their actual purpose, globally exposed just the
> single-page dump callback function, and made it return a legitimate return
> value.
> (And there may be other minor changes that I can't remember.)
>
> Let me know what you think about the changes, and if we're in agreement,
> I'll check it in tomorrow.
>
>
>
>
> -- Crash-utility mailing list Crash-utility at redhat.com
> https://www.redhat.com/mailman/listinfo/crash-utility
>
> --
> Crash-utility mailing list
> Crash-utility at redhat.com
> https://www.redhat.com/mailman/listinfo/crash-utility
More information about the Crash-utility
mailing list