[Patch] kexec: fix several issues in get_crash_notes()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 21, 2011 at 03:43:34PM +0800, Cong Wang wrote:
> On Fri, Nov 18, 2011 at 10:11 AM, Simon Horman <horms at verge.net.au> wrote:
> > On Wed, Nov 16, 2011 at 09:09:26PM +0800, WANG Cong wrote:
> >> a) We don't need 'crash_notes' array at all, save some bytes on stack.
> >> b) We forgot to fclose 'fp' before return.
> >>
> >> Signed-off-by: WANG Cong <xiyou.wangcong at gmail.com>
> >>
> >> ---
> >> diff --git a/kexec/arch/i386/crashdump-x86.c b/kexec/arch/i386/crashdump-x86.c
> >> index 98cda72..333f6ba 100644
> >> --- a/kexec/arch/i386/crashdump-x86.c
> >> +++ b/kexec/arch/i386/crashdump-x86.c
> >> @@ -589,20 +589,18 @@ static int cmdline_add_elfcorehdr(char *cmdline, unsigned long addr)
> >> ? */
> >> ?static int get_crash_notes(int cpu, uint64_t *addr, uint64_t *len)
> >> ?{
> >> - ? ? char crash_notes[PATH_MAX];
> >
> > I wonder if the following might reduce memory usage:
> >
> > ? ? ? ?const char *crash_notes = "/sys/kernel/crash_notes";
> 
> Yeah, sure. Below is the new patch.
> -------------------->
> 
> a) We don't need 'crash_notes' array at all, save some bytes on stack.
> b) We forgot to fclose 'fp' before return.

Thanks, applied.




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux