Re: [PATCH v9 01/12] x86/mm: setting fields in deferred pages

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

 



Hi Michal,


I hope I haven't missed anything but it looks good to me.

Acked-by: Michal Hocko <mhocko@xxxxxxxx>

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
--
To unsubscribe from this list: send the line "unsubscribe linux-s390" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux