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, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]
  Powered by Linux