Thread (52 messages) 52 messages, 4 authors, 2017-10-09

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

From: Pasha Tatashin <hidden>
Date: 2017-10-03 15:09:22
Also in: linux-arm-kernel, linux-mm, lkml, sparclinux

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

Acked-by: Michal Hocko <mhocko@suse.com>
Thank you for your review.
one nit below
quoted
---
  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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help