Messages in this thread |  | | | Date | Tue, 13 Sep 2011 23:35:48 +0300 | | From | "Kirill A. Shutemov" <> | | Subject | Re: [patch 0/11] mm: memcg naturalization -rc3 |
| |
On Mon, Sep 12, 2011 at 12:57:17PM +0200, Johannes Weiner wrote: > Hi everyone, > > this is the third revision of the memcg naturalization patch set. Due > to controversy, I dropped the reclaim statistics and the soft limit > reclaim rewrite. What's left is mostly making the per-memcg LRU lists > exclusive. > > Christoph suggested making struct mem_cgroup part of the core and have > reclaim always operate on at least a skeleton root_mem_cgroup with > basic LRU info even on !CONFIG_MEMCG kernels. I agree that we should > go there, but in its current form this would drag a lot of ugly memcg > internals out into the public and I'd prefer another struct mem_cgroup > shakedown and the soft limit stuff to be done before this step. But > we are getting there. > > Changelog since -rc2 > - consolidated all memcg hierarchy iteration constructs > - pass struct mem_cgroup_zone down the reclaim stack > - fix concurrent full hierarchy round-trip detection > - split out moving memcg reclaim from hierarchical global reclaim > - drop reclaim statistics > - rename do_shrink_zone to shrink_mem_cgroup_zone > - fix anon pre-aging to operate on per-memcg lrus > - revert to traditional limit reclaim hierarchy iteration > - split out lruvec introduction > - kill __add_page_to_lru_list > - fix LRU-accounting during swapcache/pagecache charging > - fix LRU-accounting of uncharged swapcache > - split out removing array id from pc->flags > - drop soft limit rework > > More introduction and test results are included in the changelog of > the first patch. > > include/linux/memcontrol.h | 74 +++-- > include/linux/mm_inline.h | 21 +- > include/linux/mmzone.h | 10 +- > include/linux/page_cgroup.h | 34 --- > mm/memcontrol.c | 688 ++++++++++++++++++++----------------------- > mm/page_alloc.c | 2 +- > mm/page_cgroup.c | 59 +---- > mm/swap.c | 24 +- > mm/vmscan.c | 447 +++++++++++++++++----------- > 9 files changed, 674 insertions(+), 685 deletions(-)
Nice patchset. Thank you.
Reviewed-by: Kirill A. Shutemov <kirill@shutemov.name>
-- Kirill A. Shutemov
|  |