Re: [PATCH v2] memory hotplug: update the variables after memory removed

2014-07-29 Thread Zhang Zhen
On 2014/7/29 17:18, David Rientjes wrote: > On Tue, 29 Jul 2014, Zhang Zhen wrote: > diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c index df1a992..fd7bd6b 100644 --- a/arch/x86/mm/init_64.c +++ b/arch/x86/mm/init_64.c @@ -673,15 +673,11 @@ void __init paging_in

Re: [PATCH v2] memory hotplug: update the variables after memory removed

2014-07-29 Thread David Rientjes
On Tue, 29 Jul 2014, Zhang Zhen wrote: > >> diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c > >> index df1a992..fd7bd6b 100644 > >> --- a/arch/x86/mm/init_64.c > >> +++ b/arch/x86/mm/init_64.c > >> @@ -673,15 +673,11 @@ void __init paging_init(void) > >> * After memory hotplug the var

Re: [PATCH v2] memory hotplug: update the variables after memory removed

2014-07-29 Thread Zhang Zhen
On 2014/7/29 15:53, David Rientjes wrote: > On Tue, 29 Jul 2014, Zhang Zhen wrote: > >> Commit ea0854170c95245a258b386c7a9314399c949fe0 added a fuction > > This would normally be written as > > Commit ea0854170c95 ("memory hotplug: fix a bug on /dev/mem for 64-bit > kernels") ... > ok. >> upda

Re: [PATCH v2] memory hotplug: update the variables after memory removed

2014-07-29 Thread David Rientjes
On Tue, 29 Jul 2014, Zhang Zhen wrote: > Commit ea0854170c95245a258b386c7a9314399c949fe0 added a fuction This would normally be written as Commit ea0854170c95 ("memory hotplug: fix a bug on /dev/mem for 64-bit kernels") ... > update_end_of_memory_vars() to update high_memory, max_pfn and > max

[PATCH v2] memory hotplug: update the variables after memory removed

2014-07-29 Thread Zhang Zhen
Commit ea0854170c95245a258b386c7a9314399c949fe0 added a fuction update_end_of_memory_vars() to update high_memory, max_pfn and max_low_pfn. I modified the function according to Dave Hansen and David Rientjes's suggestions. And call it in arch_remove_memory() to update these variables too. Change