[PATCH v9 01/12] x86/mm: setting fields in deferred pages
Pasha Tatashin
pasha.tatashin at oracle.com
Tue Oct 3 08:07:54 PDT 2017
Hi Michal,
>
> I hope I haven't missed anything but it looks good to me.
>
> Acked-by: Michal Hocko <mhocko at suse.com>
Thank you for your review.
>
> one nit below
>> ---
>> arch/x86/mm/init_64.c | 9 +++++++--
>> 1 file changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c
>> index 5ea1c3c2636e..30fe22558720 100644
>> --- a/arch/x86/mm/init_64.c
>> +++ b/arch/x86/mm/init_64.c
>> @@ -1182,12 +1182,17 @@ void __init mem_init(void)
>>
>> /* clear_bss() already clear the empty_zero_page */
>>
>> - register_page_bootmem_info();
>> -
>> /* this will put all memory onto the freelists */
>> free_all_bootmem();
>> after_bootmem = 1;
>>
>> + /* Must be done after boot memory is put on freelist, because here we
>
> standard code style is to do
> /*
> * text starts here
OK, will change for both patch 1 and 2.
Pasha
More information about the linux-arm-kernel
mailing list