All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vineet Gupta <Vineet.Gupta1@synopsys.com>
To: Jiang Liu <liuj97@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Jiang Liu <jiang.liu@huawei.com>,
	David Rientjes <rientjes@google.com>,
	Wen Congyang <wency@cn.fujitsu.com>, Mel Gorman <mgorman@suse.de>,
	Minchan Kim <minchan@kernel.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Michal Hocko <mhocko@suse.cz>,
	James Bottomley <James.Bottomley@HansenPartnership.com>,
	Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	David Howells <dhowells@redhat.com>,
	Mark Salter <msalter@redhat.com>,
	Jianguo Wu <wujianguo@huawei.com>,
	linux-mm@kvack.org, linux-arch@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7, part3 14/16] mm: concentrate modification of totalram_pages into the mm core
Date: Wed, 29 May 2013 14:19:42 +0530	[thread overview]
Message-ID: <51A5C126.9030105@synopsys.com> (raw)
In-Reply-To: <1368805518-2634-15-git-send-email-jiang.liu@huawei.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

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

WARNING: multiple messages have this Message-ID (diff)
From: Vineet Gupta <Vineet.Gupta1@synopsys.com>
To: Jiang Liu <liuj97@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Jiang Liu <jiang.liu@huawei.com>,
	David Rientjes <rientjes@google.com>,
	Wen Congyang <wency@cn.fujitsu.com>, Mel Gorman <mgorman@suse.de>,
	Minchan Kim <minchan@kernel.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Michal Hocko <mhocko@suse.cz>,
	James Bottomley <James.Bottomley@HansenPartnership.com>,
	Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	David Howells <dhowells@redhat.com>,
	Mark Salter <msalter@redhat.com>,
	Jianguo Wu <wujianguo@huawei.com>,
	linux-mm@kvack.org, linux-arch@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7, part3 14/16] mm: concentrate modification of totalram_pages into the mm core
Date: Wed, 29 May 2013 14:19:42 +0530	[thread overview]
Message-ID: <51A5C126.9030105@synopsys.com> (raw)
Message-ID: <20130529084942.GherJZlQCgGCRwAczrQg8LuqazCRTi2Y7Iltc3f3EdY@z> (raw)
In-Reply-To: <1368805518-2634-15-git-send-email-jiang.liu@huawei.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


WARNING: multiple messages have this Message-ID (diff)
From: Vineet Gupta <Vineet.Gupta1@synopsys.com>
To: Jiang Liu <liuj97@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Jiang Liu <jiang.liu@huawei.com>,
	David Rientjes <rientjes@google.com>,
	Wen Congyang <wency@cn.fujitsu.com>, Mel Gorman <mgorman@suse.de>,
	Minchan Kim <minchan@kernel.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Michal Hocko <mhocko@suse.cz>,
	James Bottomley <James.Bottomley@HansenPartnership.com>,
	Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	David Howells <dhowells@redhat.com>,
	Mark Salter <msalter@redhat.com>,
	Jianguo Wu <wujianguo@huawei.com>, <linux-mm@kvack.org>,
	<linux-arch@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v7, part3 14/16] mm: concentrate modification of totalram_pages into the mm core
Date: Wed, 29 May 2013 14:19:42 +0530	[thread overview]
Message-ID: <51A5C126.9030105@synopsys.com> (raw)
In-Reply-To: <1368805518-2634-15-git-send-email-jiang.liu@huawei.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


  reply	other threads:[~2013-05-29  8:49 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-17 15:45 [PATCH v7, part3 00/16] accurately calculate memory statisitic information Jiang Liu
2013-05-17 15:45 ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 01/16] mm: change signature of free_reserved_area() to fix building warnings Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 02/16] mm: enhance free_reserved_area() to support poisoning memory with zero Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 03/16] mm/ARM64: kill poison_init_mem() Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 04/16] mm/x86: use free_reserved_area() to simplify code Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 05/16] mm/tile: use common help functions to free reserved pages Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 06/16] mm, powertv: use free_reserved_area() to simplify code Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 07/16] mm, acornfb: " Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 08/16] mm: fix some trivial typos in comments Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 09/16] mm: use managed_pages to calculate default zonelist order Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 10/16] mm: accurately calculate zone->managed_pages for highmem zones Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 11/16] mm: use a dedicated lock to protect totalram_pages and zone->managed_pages Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 12/16] mm: make __free_pages_bootmem() only available at boot time Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 13/16] mm: correctly update zone->mamaged_pages Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 14/16] mm: concentrate modification of totalram_pages into the mm core Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-29  8:49   ` Vineet Gupta [this message]
2013-05-29  8:49     ` Vineet Gupta
2013-05-29  8:49     ` Vineet Gupta
2013-05-17 15:45 ` [PATCH v7, part3 15/16] mm: report available pages as "MemTotal" for each NUMA node Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-17 15:45 ` [PATCH v7, part3 16/16] AVR32: fix building warnings caused by redifinitions of HZ Jiang Liu
2013-05-17 15:45   ` Jiang Liu
2013-05-18 19:33   ` Hans-Christian Egtvedt
2013-05-18 19:33     ` Hans-Christian Egtvedt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51A5C126.9030105@synopsys.com \
    --to=vineet.gupta1@synopsys.com \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=akpm@linux-foundation.org \
    --cc=dhowells@redhat.com \
    --cc=jiang.liu@huawei.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=liuj97@gmail.com \
    --cc=mgorman@suse.de \
    --cc=mhocko@suse.cz \
    --cc=minchan@kernel.org \
    --cc=msalter@redhat.com \
    --cc=rientjes@google.com \
    --cc=sergei.shtylyov@cogentembedded.com \
    --cc=wency@cn.fujitsu.com \
    --cc=wujianguo@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.