Re: [PATCH v7, part3 14/16] mm: concentrate modification of totalram_pages into the mm core

[Vineet Gupta <Vineet.Gupta1@synopsys.com>]

On 05/17/2013 09:15 PM, Jiang Liu wrote:
> Concentrate code to modify totalram_pages into the mm core, so the arch
> memory initialized code doesn't need to take care of it. With these
> changes applied, only following functions from mm core modify global
> variable totalram_pages:
> free_bootmem_late(), free_all_bootmem(), free_all_bootmem_node(),
> adjust_managed_page_count().
> 
> With this patch applied, it will be much more easier for us to keep
> totalram_pages and zone->managed_pages in consistence.
> 
> Signed-off-by: Jiang Liu <jiang.liu@huawei.com>
> Acked-by: David Howells <dhowells@redhat.com>
> ---
>  arch/alpha/mm/init.c             | 2 +-
>  arch/alpha/mm/numa.c             | 2 +-
>  arch/arc/mm/init.c               | 2 +-

...

> diff --git a/arch/arc/mm/init.c b/arch/arc/mm/init.c
> index f9c7077..c668a60 100644
> --- a/arch/arc/mm/init.c
> +++ b/arch/arc/mm/init.c
> @@ -111,7 +111,7 @@ void __init mem_init(void)
>  
>  	high_memory = (void *)(CONFIG_LINUX_LINK_BASE + arc_mem_sz);
>  
> -	totalram_pages = free_all_bootmem();
> +	free_all_bootmem();
>  
>  	/* count all reserved pages [kernel code/data/mem_map..] */
>  	reserved_pages = 0;

Acked-by: Vineet Gupta <vgupta@synopsys.com>	for arch/arc bits

Thx,
-Vineet