Re: [RFC][PATCH -mm 0/7] memcg: lockless page_cgroup v1

From: Hirokazu Takahashi
Date: Wed Aug 20 2008 - 07:33:41 EST


Hi,

> Hi, this is a patch set for lockless page_cgroup.
>
> dropped patches related to mem+swap controller for easy review.
> (I'm rewriting it, too.)
>
> Changes from current -mm is.
> - page_cgroup->flags operations is set to be atomic.
> - lock_page_cgroup() is removed.
> - page->page_cgroup is changed from unsigned long to struct page_cgroup*
> - page_cgroup is freed by RCU.
> - For avoiding race, charge/uncharge against mm/memory.c::insert_page() is
> omitted. This is ususally used for mapping device's page. (I think...)
>
> In my quick test, perfomance is improved a little. But the benefit of this
> patch is to allow access page_cgroup without lock. I think this is good
> for Yamamoto's Dirty page tracking for memcg.
> For I/O tracking people, I added a header file for allowing access to
> page_cgroup from out of memcontrol.c

Thanks, Kame.
It is a good news that the page tracking framework is open.
I think I can send some feedback to you to make it more generic.

> The base kernel is recent mmtom. Any comments are welcome.
> This is still under test. I have to do long-run test before removing "RFC".
>

Thanks,
Hirokazu Takahashi.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/