[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20110704132650.da7329c5.randy.dunlap@oracle.com>
Date: Mon, 4 Jul 2011 13:26:50 -0700
From: Randy Dunlap <randy.dunlap@...cle.com>
To: linux-kernel@...r.kernel.org, sfr@...b.auug.org.au
Cc: akpm@...ux-foundation.org, kamezawa.hiroyu@...fujitsu.com
Subject: Re: + memcg-fix-reclaimable-lru-check-in-memcg-fix-2.patch added to
-mm tree
On Fri, 01 Jul 2011 12:36:20 -0700 akpm@...ux-foundation.org wrote:
>
> The patch titled
> memcg-fix-reclaimable-lru-check-in-memcg-fix-2
> has been added to the -mm tree. Its filename is
> memcg-fix-reclaimable-lru-check-in-memcg-fix-2.patch
>
Is there some way to get this patch added to linux-next?
thanks.
> ------------------------------------------------------
> Subject: memcg-fix-reclaimable-lru-check-in-memcg-fix-2
> From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
>
> memcg-fix-reclaimable-lru-check-in-memcg.patch
> causes following error with !CONFIG_NUMA.
>
> > mmotm-2011-0630-1559/mm/memcontrol.c:1579: error: implicit declaration of function 'mem_cgroup_node_nr_file_lru_pages'
> > mmotm-2011-0630-1559/mm/memcontrol.c:1583: error: implicit declaration of function 'mem_cgroup_node_nr_anon_lru_pages'
> >
>
> This patch fixes it by moving functions out of #ifdef.
>
> Reported-by: Randy Dunlap <randy.dunlap@...cle.com>
> Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
> cked-by: Randy Dunlap <randy.dunlap@...cle.com>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
>
> mm/memcontrol.c | 23 +++++++++++------------
> 1 file changed, 11 insertions(+), 12 deletions(-)
>
> diff -puN mm/memcontrol.c~memcg-fix-reclaimable-lru-check-in-memcg-fix-2 mm/memcontrol.c
> --- a/mm/memcontrol.c~memcg-fix-reclaimable-lru-check-in-memcg-fix-2
> +++ a/mm/memcontrol.c
> @@ -1120,7 +1120,6 @@ unsigned long mem_cgroup_zone_nr_lru_pag
> return MEM_CGROUP_ZSTAT(mz, lru);
> }
>
> -#ifdef CONFIG_NUMA
> static unsigned long mem_cgroup_node_nr_file_lru_pages(struct mem_cgroup *memcg,
> int nid)
> {
> @@ -1132,6 +1131,17 @@ static unsigned long mem_cgroup_node_nr_
> return ret;
> }
>
> +static unsigned long mem_cgroup_node_nr_anon_lru_pages(struct mem_cgroup *memcg,
> + int nid)
> +{
> + unsigned long ret;
> +
> + ret = mem_cgroup_get_zonestat_node(memcg, nid, LRU_INACTIVE_ANON) +
> + mem_cgroup_get_zonestat_node(memcg, nid, LRU_ACTIVE_ANON);
> + return ret;
> +}
> +
> +#if MAX_NUMNODES > 1
> static unsigned long mem_cgroup_nr_file_lru_pages(struct mem_cgroup *memcg)
> {
> u64 total = 0;
> @@ -1143,17 +1153,6 @@ static unsigned long mem_cgroup_nr_file_
> return total;
> }
>
> -static unsigned long mem_cgroup_node_nr_anon_lru_pages(struct mem_cgroup *memcg,
> - int nid)
> -{
> - unsigned long ret;
> -
> - ret = mem_cgroup_get_zonestat_node(memcg, nid, LRU_INACTIVE_ANON) +
> - mem_cgroup_get_zonestat_node(memcg, nid, LRU_ACTIVE_ANON);
> -
> - return ret;
> -}
> -
> static unsigned long mem_cgroup_nr_anon_lru_pages(struct mem_cgroup *memcg)
> {
> u64 total = 0;
> _
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists