[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20081107115030.7ccf3f07.kamezawa.hiroyu@jp.fujitsu.com>
Date: Fri, 7 Nov 2008 11:50:30 +0900
From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
To: vgoyal@...hat.com
Cc: linux-kernel@...r.kernel.org,
containers@...ts.linux-foundation.org,
virtualization@...ts.linux-foundation.org, jens.axboe@...cle.com,
Hirokazu Takahashi <taka@...inux.co.jp>,
Ryo Tsuruta <ryov@...inux.co.jp>,
Andrea Righi <righi.andrea@...il.com>,
Satoshi UCHIDA <s-uchida@...jp.nec.com>,
Rik van Riel <riel@...hat.com>, fernando@....ntt.co.jp,
Jeff Moyer <jmoyer@...hat.com>, menage@...gle.com,
ngupta@...gle.com, Andrew Morton <akpm@...ux-foundation.org>,
balbir@...ux.vnet.ibm.com
Subject: Re: [patch 2/4] io controller: biocgroup implementation
On Thu, 06 Nov 2008 10:30:24 -0500
vgoyal@...hat.com wrote:
>
> o biocgroup functionality.
> o Implemented new controller "bio"
> o Most of it picked from dm-ioband biocgroup implementation patches.
>
page_cgroup implementation is changed and most of this patch needs rework.
please see the latest one. (I think most of new characteristics are useful
for you.)
One comment from me is
==
> +struct page_cgroup {
> + struct list_head lru; /* per cgroup LRU list */
> + struct page *page;
> + struct mem_cgroup *mem_cgroup;
> + int flags;
> +#ifdef CONFIG_CGROUP_BIO
> + struct list_head blist; /* for bio_cgroup page list */
> + struct bio_cgroup *bio_cgroup;
> +#endif
> +};
==
this blist is too bad. please keep this object small...
Maybe dm-ioband people will post his own new one. just making use of it is an idea.
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