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@xxxxxxxxxx> > > Thanks > Baoquan _______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec
