lkml.org 
[lkml]   [2019]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH V2] x86/kexec: fix a kexec_file_load failure
On 01/08/19 at 04:51pm, Baoquan He wrote:
> On 01/08/19 at 04:46pm, Dave Young wrote:
> > > Wondering why this place doesn't need the initialization assignment.
> > > Isn't it to assign in all places before kexec_add_buffer() calling?
> >
> > C designated initializers will make sure to initialize it as zero.
> > We set KEXEC_BUF_MEM_UNKNOWN as 0 so it just works.
>
> Got it, it works, thanks. People may need check code to find out
> KEXEC_BUF_MEM_UNKNOWN is 0, then realize this fact.

Agreed, it is not very clear now. It's better to improve it with some explict
initial value since we have the macro. But since this is a regression
I suggest to fix the bug first, I can send a patch later for the
improvement.

Thanks!
>
> Other than this, it looks good to me, ack it.
>
> Acked-by: Baoquan He <bhe@redhat.com>
>
> Thanks
> Baoquan

\
 
 \ /
  Last update: 2019-01-08 10:12    [W:0.060 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site