[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20071129122532.68ff4e75.kamezawa.hiroyu@jp.fujitsu.com>
Date: Thu, 29 Nov 2007 12:25:32 +0900
From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
To: yamamoto@...inux.co.jp (YAMAMOTO Takashi)
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org,
containers@...ts.osdl.org, akpm@...ux-foundation.org,
balbir@...ux.vnet.ibm.com
Subject: Re: [PATCH][for -mm] per-zone and reclaim enhancements for memory
controller take 3 [3/10] per-zone active inactive counter
On Thu, 29 Nov 2007 12:19:37 +0900 (JST)
yamamoto@...inux.co.jp (YAMAMOTO Takashi) wrote:
> > @@ -651,10 +758,11 @@
> > /* Avoid race with charge */
> > atomic_set(&pc->ref_cnt, 0);
> > if (clear_page_cgroup(page, pc) == pc) {
> > + int active;
> > css_put(&mem->css);
> > + active = pc->flags & PAGE_CGROUP_FLAG_ACTIVE;
> > res_counter_uncharge(&mem->res, PAGE_SIZE);
> > - list_del_init(&pc->lru);
> > - mem_cgroup_charge_statistics(mem, pc->flags, false);
> > + __mem_cgroup_remove_list(pc);
> > kfree(pc);
> > } else /* being uncharged ? ...do relax */
> > break;
>
> 'active' seems unused.
>
ok, I will post clean-up against -mm2.
Thanks,
-Kame
-
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