From: Michal Hocko <mhocko@suse.cz> To: Sha Zhengju <handai.szj@gmail.com> Cc: linux-mm@kvack.org, cgroups@vger.kernel.org, kamezawa.hiroyu@jp.fujitsu.com, gthelen@google.com, yinghan@google.com, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, Sha Zhengju <handai.szj@taobao.com> Subject: Re: [PATCH 2/7] memcg: remove MEMCG_NR_FILE_MAPPED Date: Wed, 4 Jul 2012 14:56:09 +0200 [thread overview] Message-ID: <20120704125608.GF29842@tiehlicka.suse.cz> (raw) In-Reply-To: <1340881111-5576-1-git-send-email-handai.szj@taobao.com> On Thu 28-06-12 18:58:31, Sha Zhengju wrote: > From: Sha Zhengju <handai.szj@taobao.com> > > While accounting memcg page stat, it's not worth to use MEMCG_NR_FILE_MAPPED > as an extra layer of indirection because of the complexity and presumed > performance overhead. We can use MEM_CGROUP_STAT_FILE_MAPPED directly. > > Signed-off-by: Sha Zhengju <handai.szj@taobao.com> Acked-by: Michal Hocko <mhocko@suse.cz> > --- > include/linux/memcontrol.h | 25 +++++++++++++++++-------- > mm/memcontrol.c | 24 +----------------------- > mm/rmap.c | 4 ++-- > 3 files changed, 20 insertions(+), 33 deletions(-) > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > index 83e7ba9..20b0f2d 100644 > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -27,9 +27,18 @@ struct page_cgroup; > struct page; > struct mm_struct; > > -/* Stats that can be updated by kernel. */ > -enum mem_cgroup_page_stat_item { > - MEMCG_NR_FILE_MAPPED, /* # of pages charged as file rss */ > +/* > + * Statistics for memory cgroup. > + */ > +enum mem_cgroup_stat_index { > + /* > + * For MEM_CONTAINER_TYPE_ALL, usage = pagecache + rss. > + */ > + MEM_CGROUP_STAT_CACHE, /* # of pages charged as cache */ > + MEM_CGROUP_STAT_RSS, /* # of pages charged as anon rss */ > + MEM_CGROUP_STAT_FILE_MAPPED, /* # of pages charged as file rss */ > + MEM_CGROUP_STAT_SWAP, /* # of pages, swapped out */ > + MEM_CGROUP_STAT_NSTATS, > }; > > struct mem_cgroup_reclaim_cookie { > @@ -164,17 +173,17 @@ static inline void mem_cgroup_end_update_page_stat(struct page *page, > } > > void mem_cgroup_update_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx, > + enum mem_cgroup_stat_index idx, > int val); > > static inline void mem_cgroup_inc_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > mem_cgroup_update_page_stat(page, idx, 1); > } > > static inline void mem_cgroup_dec_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > mem_cgroup_update_page_stat(page, idx, -1); > } > @@ -349,12 +358,12 @@ static inline void mem_cgroup_end_update_page_stat(struct page *page, > } > > static inline void mem_cgroup_inc_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > } > > static inline void mem_cgroup_dec_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > } > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index a2677e0..ebed1ca 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -77,20 +77,6 @@ static int really_do_swap_account __initdata = 0; > #endif > > > -/* > - * Statistics for memory cgroup. > - */ > -enum mem_cgroup_stat_index { > - /* > - * For MEM_CONTAINER_TYPE_ALL, usage = pagecache + rss. > - */ > - MEM_CGROUP_STAT_CACHE, /* # of pages charged as cache */ > - MEM_CGROUP_STAT_RSS, /* # of pages charged as anon rss */ > - MEM_CGROUP_STAT_FILE_MAPPED, /* # of pages charged as file rss */ > - MEM_CGROUP_STAT_SWAP, /* # of pages, swapped out */ > - MEM_CGROUP_STAT_NSTATS, > -}; > - > static const char * const mem_cgroup_stat_names[] = { > "cache", > "rss", > @@ -1926,7 +1912,7 @@ void __mem_cgroup_end_update_page_stat(struct page *page, unsigned long *flags) > } > > void mem_cgroup_update_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx, int val) > + enum mem_cgroup_stat_index idx, int val) > { > struct mem_cgroup *memcg; > struct page_cgroup *pc = lookup_page_cgroup(page); > @@ -1939,14 +1925,6 @@ void mem_cgroup_update_page_stat(struct page *page, > if (unlikely(!memcg || !PageCgroupUsed(pc))) > return; > > - switch (idx) { > - case MEMCG_NR_FILE_MAPPED: > - idx = MEM_CGROUP_STAT_FILE_MAPPED; > - break; > - default: > - BUG(); > - } > - > this_cpu_add(memcg->stat->count[idx], val); > } > > diff --git a/mm/rmap.c b/mm/rmap.c > index 2144160..d6b93df 100644 > --- a/mm/rmap.c > +++ b/mm/rmap.c > @@ -1148,7 +1148,7 @@ void page_add_file_rmap(struct page *page) > mem_cgroup_begin_update_page_stat(page, &locked, &flags); > if (atomic_inc_and_test(&page->_mapcount)) { > __inc_zone_page_state(page, NR_FILE_MAPPED); > - mem_cgroup_inc_page_stat(page, MEMCG_NR_FILE_MAPPED); > + mem_cgroup_inc_page_stat(page, MEM_CGROUP_STAT_FILE_MAPPED); > } > mem_cgroup_end_update_page_stat(page, &locked, &flags); > } > @@ -1202,7 +1202,7 @@ void page_remove_rmap(struct page *page) > NR_ANON_TRANSPARENT_HUGEPAGES); > } else { > __dec_zone_page_state(page, NR_FILE_MAPPED); > - mem_cgroup_dec_page_stat(page, MEMCG_NR_FILE_MAPPED); > + mem_cgroup_dec_page_stat(page, MEM_CGROUP_STAT_FILE_MAPPED); > } > /* > * It would be tidy to reset the PageAnon mapping here, > -- > 1.7.1 > -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic
WARNING: multiple messages have this Message-ID (diff)
From: Michal Hocko <mhocko@suse.cz> To: Sha Zhengju <handai.szj@gmail.com> Cc: linux-mm@kvack.org, cgroups@vger.kernel.org, kamezawa.hiroyu@jp.fujitsu.com, gthelen@google.com, yinghan@google.com, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, Sha Zhengju <handai.szj@taobao.com> Subject: Re: [PATCH 2/7] memcg: remove MEMCG_NR_FILE_MAPPED Date: Wed, 4 Jul 2012 14:56:09 +0200 [thread overview] Message-ID: <20120704125608.GF29842@tiehlicka.suse.cz> (raw) In-Reply-To: <1340881111-5576-1-git-send-email-handai.szj@taobao.com> On Thu 28-06-12 18:58:31, Sha Zhengju wrote: > From: Sha Zhengju <handai.szj@taobao.com> > > While accounting memcg page stat, it's not worth to use MEMCG_NR_FILE_MAPPED > as an extra layer of indirection because of the complexity and presumed > performance overhead. We can use MEM_CGROUP_STAT_FILE_MAPPED directly. > > Signed-off-by: Sha Zhengju <handai.szj@taobao.com> Acked-by: Michal Hocko <mhocko@suse.cz> > --- > include/linux/memcontrol.h | 25 +++++++++++++++++-------- > mm/memcontrol.c | 24 +----------------------- > mm/rmap.c | 4 ++-- > 3 files changed, 20 insertions(+), 33 deletions(-) > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > index 83e7ba9..20b0f2d 100644 > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -27,9 +27,18 @@ struct page_cgroup; > struct page; > struct mm_struct; > > -/* Stats that can be updated by kernel. */ > -enum mem_cgroup_page_stat_item { > - MEMCG_NR_FILE_MAPPED, /* # of pages charged as file rss */ > +/* > + * Statistics for memory cgroup. > + */ > +enum mem_cgroup_stat_index { > + /* > + * For MEM_CONTAINER_TYPE_ALL, usage = pagecache + rss. > + */ > + MEM_CGROUP_STAT_CACHE, /* # of pages charged as cache */ > + MEM_CGROUP_STAT_RSS, /* # of pages charged as anon rss */ > + MEM_CGROUP_STAT_FILE_MAPPED, /* # of pages charged as file rss */ > + MEM_CGROUP_STAT_SWAP, /* # of pages, swapped out */ > + MEM_CGROUP_STAT_NSTATS, > }; > > struct mem_cgroup_reclaim_cookie { > @@ -164,17 +173,17 @@ static inline void mem_cgroup_end_update_page_stat(struct page *page, > } > > void mem_cgroup_update_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx, > + enum mem_cgroup_stat_index idx, > int val); > > static inline void mem_cgroup_inc_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > mem_cgroup_update_page_stat(page, idx, 1); > } > > static inline void mem_cgroup_dec_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > mem_cgroup_update_page_stat(page, idx, -1); > } > @@ -349,12 +358,12 @@ static inline void mem_cgroup_end_update_page_stat(struct page *page, > } > > static inline void mem_cgroup_inc_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > } > > static inline void mem_cgroup_dec_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx) > + enum mem_cgroup_stat_index idx) > { > } > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index a2677e0..ebed1ca 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -77,20 +77,6 @@ static int really_do_swap_account __initdata = 0; > #endif > > > -/* > - * Statistics for memory cgroup. > - */ > -enum mem_cgroup_stat_index { > - /* > - * For MEM_CONTAINER_TYPE_ALL, usage = pagecache + rss. > - */ > - MEM_CGROUP_STAT_CACHE, /* # of pages charged as cache */ > - MEM_CGROUP_STAT_RSS, /* # of pages charged as anon rss */ > - MEM_CGROUP_STAT_FILE_MAPPED, /* # of pages charged as file rss */ > - MEM_CGROUP_STAT_SWAP, /* # of pages, swapped out */ > - MEM_CGROUP_STAT_NSTATS, > -}; > - > static const char * const mem_cgroup_stat_names[] = { > "cache", > "rss", > @@ -1926,7 +1912,7 @@ void __mem_cgroup_end_update_page_stat(struct page *page, unsigned long *flags) > } > > void mem_cgroup_update_page_stat(struct page *page, > - enum mem_cgroup_page_stat_item idx, int val) > + enum mem_cgroup_stat_index idx, int val) > { > struct mem_cgroup *memcg; > struct page_cgroup *pc = lookup_page_cgroup(page); > @@ -1939,14 +1925,6 @@ void mem_cgroup_update_page_stat(struct page *page, > if (unlikely(!memcg || !PageCgroupUsed(pc))) > return; > > - switch (idx) { > - case MEMCG_NR_FILE_MAPPED: > - idx = MEM_CGROUP_STAT_FILE_MAPPED; > - break; > - default: > - BUG(); > - } > - > this_cpu_add(memcg->stat->count[idx], val); > } > > diff --git a/mm/rmap.c b/mm/rmap.c > index 2144160..d6b93df 100644 > --- a/mm/rmap.c > +++ b/mm/rmap.c > @@ -1148,7 +1148,7 @@ void page_add_file_rmap(struct page *page) > mem_cgroup_begin_update_page_stat(page, &locked, &flags); > if (atomic_inc_and_test(&page->_mapcount)) { > __inc_zone_page_state(page, NR_FILE_MAPPED); > - mem_cgroup_inc_page_stat(page, MEMCG_NR_FILE_MAPPED); > + mem_cgroup_inc_page_stat(page, MEM_CGROUP_STAT_FILE_MAPPED); > } > mem_cgroup_end_update_page_stat(page, &locked, &flags); > } > @@ -1202,7 +1202,7 @@ void page_remove_rmap(struct page *page) > NR_ANON_TRANSPARENT_HUGEPAGES); > } else { > __dec_zone_page_state(page, NR_FILE_MAPPED); > - mem_cgroup_dec_page_stat(page, MEMCG_NR_FILE_MAPPED); > + mem_cgroup_dec_page_stat(page, MEM_CGROUP_STAT_FILE_MAPPED); > } > /* > * It would be tidy to reset the PageAnon mapping here, > -- > 1.7.1 > -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2012-07-04 12:56 UTC|newest] Thread overview: 132+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-06-28 10:54 [PATCH 0/7] Per-cgroup page stat accounting Sha Zhengju 2012-06-28 10:54 ` Sha Zhengju 2012-06-28 10:54 ` Sha Zhengju 2012-06-28 10:57 ` [PATCH 1/7] memcg: update cgroup memory document Sha Zhengju 2012-06-28 10:57 ` Sha Zhengju 2012-06-28 10:57 ` Sha Zhengju 2012-07-02 7:00 ` Kamezawa Hiroyuki 2012-07-02 7:00 ` Kamezawa Hiroyuki 2012-07-04 12:47 ` Michal Hocko 2012-07-04 12:47 ` Michal Hocko 2012-07-04 12:47 ` Michal Hocko 2012-07-07 13:45 ` Fengguang Wu 2012-07-07 13:45 ` Fengguang Wu 2012-07-07 13:45 ` Fengguang Wu 2012-06-28 10:58 ` [PATCH 2/7] memcg: remove MEMCG_NR_FILE_MAPPED Sha Zhengju 2012-06-28 10:58 ` Sha Zhengju 2012-07-02 10:44 ` Kamezawa Hiroyuki 2012-07-02 10:44 ` Kamezawa Hiroyuki 2012-07-02 10:44 ` Kamezawa Hiroyuki 2012-07-04 12:56 ` Michal Hocko [this message] 2012-07-04 12:56 ` Michal Hocko 2012-07-04 12:58 ` Michal Hocko 2012-07-04 12:58 ` Michal Hocko 2012-07-04 12:58 ` Michal Hocko 2012-07-07 13:48 ` Fengguang Wu 2012-07-07 13:48 ` Fengguang Wu 2012-07-09 21:01 ` Greg Thelen 2012-07-09 21:01 ` Greg Thelen 2012-07-09 21:01 ` Greg Thelen 2012-07-11 8:00 ` Sha Zhengju 2012-07-11 8:00 ` Sha Zhengju 2012-06-28 11:01 ` [PATCH 3/7] Make TestSetPageDirty and dirty page accounting in one func Sha Zhengju 2012-06-28 11:01 ` Sha Zhengju 2012-07-02 11:14 ` Kamezawa Hiroyuki 2012-07-02 11:14 ` Kamezawa Hiroyuki 2012-07-02 11:14 ` Kamezawa Hiroyuki 2012-07-07 14:42 ` Fengguang Wu 2012-07-07 14:42 ` Fengguang Wu 2012-07-04 14:23 ` Michal Hocko 2012-07-04 14:23 ` Michal Hocko 2012-06-28 11:03 ` [PATCH 4/7] Use vfs __set_page_dirty interface instead of doing it inside filesystem Sha Zhengju 2012-06-28 11:03 ` Sha Zhengju 2012-06-28 11:03 ` Sha Zhengju 2012-06-29 5:21 ` Sage Weil 2012-06-29 5:21 ` Sage Weil 2012-06-29 5:21 ` Sage Weil 2012-07-02 8:10 ` Sha Zhengju 2012-07-02 8:10 ` Sha Zhengju 2012-07-02 14:49 ` Sage Weil 2012-07-02 14:49 ` Sage Weil 2012-07-04 8:11 ` Sha Zhengju 2012-07-04 8:11 ` Sha Zhengju 2012-07-05 15:20 ` Sage Weil 2012-07-05 15:20 ` Sage Weil 2012-07-05 15:40 ` Sha Zhengju 2012-07-05 15:40 ` Sha Zhengju 2012-07-04 14:27 ` Michal Hocko 2012-07-04 14:27 ` Michal Hocko 2012-06-28 11:04 ` [PATCH 5/7] memcg: add per cgroup dirty pages accounting Sha Zhengju 2012-06-28 11:04 ` Sha Zhengju 2012-06-28 11:04 ` Sha Zhengju 2012-07-03 5:57 ` Kamezawa Hiroyuki 2012-07-03 5:57 ` Kamezawa Hiroyuki 2012-07-08 14:45 ` Fengguang Wu 2012-07-08 14:45 ` Fengguang Wu 2012-07-04 16:11 ` Michal Hocko 2012-07-04 16:11 ` Michal Hocko 2012-07-04 16:11 ` Michal Hocko 2012-07-09 21:02 ` Greg Thelen 2012-07-09 21:02 ` Greg Thelen 2012-07-11 9:32 ` Sha Zhengju 2012-07-11 9:32 ` Sha Zhengju 2012-07-19 6:33 ` Kamezawa Hiroyuki 2012-07-19 6:33 ` Kamezawa Hiroyuki 2012-07-19 6:33 ` Kamezawa Hiroyuki 2012-06-28 11:05 ` [PATCH 6/7] memcg: add per cgroup writeback " Sha Zhengju 2012-06-28 11:05 ` Sha Zhengju 2012-07-03 6:31 ` Kamezawa Hiroyuki 2012-07-03 6:31 ` Kamezawa Hiroyuki 2012-07-04 8:24 ` Sha Zhengju 2012-07-04 8:24 ` Sha Zhengju 2012-07-08 14:44 ` Fengguang Wu 2012-07-08 14:44 ` Fengguang Wu 2012-07-08 23:01 ` Johannes Weiner 2012-07-08 23:01 ` Johannes Weiner 2012-07-09 1:37 ` Fengguang Wu 2012-07-09 1:37 ` Fengguang Wu 2012-07-09 1:37 ` Fengguang Wu 2012-07-04 16:15 ` Michal Hocko 2012-07-04 16:15 ` Michal Hocko 2012-06-28 11:06 ` Sha Zhengju 2012-06-28 11:06 ` Sha Zhengju 2012-06-28 11:06 ` Sha Zhengju 2012-07-08 14:53 ` Fengguang Wu 2012-07-08 14:53 ` Fengguang Wu 2012-07-08 14:53 ` Fengguang Wu 2012-07-09 3:36 ` Sha Zhengju 2012-07-09 3:36 ` Sha Zhengju 2012-07-09 3:36 ` Sha Zhengju 2012-07-09 4:14 ` Fengguang Wu 2012-07-09 4:14 ` Fengguang Wu 2012-07-09 4:14 ` Fengguang Wu 2012-07-09 4:18 ` Kamezawa Hiroyuki 2012-07-09 4:18 ` Kamezawa Hiroyuki 2012-07-09 5:22 ` Sha Zhengju 2012-07-09 5:22 ` Sha Zhengju 2012-07-09 5:22 ` Sha Zhengju 2012-07-09 5:28 ` Fengguang Wu 2012-07-09 5:28 ` Fengguang Wu 2012-07-09 5:28 ` Fengguang Wu 2012-07-09 5:19 ` Sha Zhengju 2012-07-09 5:19 ` Sha Zhengju 2012-07-09 5:25 ` Fengguang Wu 2012-07-09 5:25 ` Fengguang Wu 2012-07-09 21:02 ` Greg Thelen 2012-07-09 21:02 ` Greg Thelen 2012-06-28 11:06 ` [PATCH 7/7] memcg: print more detailed info while memcg oom happening Sha Zhengju 2012-06-28 11:06 ` Sha Zhengju 2012-06-28 11:06 ` Sha Zhengju 2012-07-04 8:25 ` Sha Zhengju 2012-07-04 8:25 ` Sha Zhengju 2012-07-04 8:25 ` Sha Zhengju 2012-07-04 8:29 ` Kamezawa Hiroyuki 2012-07-04 8:29 ` Kamezawa Hiroyuki 2012-07-04 11:20 ` Sha Zhengju 2012-07-04 11:20 ` Sha Zhengju 2012-07-04 11:20 ` Sha Zhengju 2012-06-29 8:23 ` [PATCH 0/7] Per-cgroup page stat accounting Kamezawa Hiroyuki 2012-06-29 8:23 ` Kamezawa Hiroyuki 2012-07-02 7:51 ` Sha Zhengju 2012-07-02 7:51 ` Sha Zhengju 2012-07-02 7:51 ` Sha Zhengju
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20120704125608.GF29842@tiehlicka.suse.cz \ --to=mhocko@suse.cz \ --cc=akpm@linux-foundation.org \ --cc=cgroups@vger.kernel.org \ --cc=gthelen@google.com \ --cc=handai.szj@gmail.com \ --cc=handai.szj@taobao.com \ --cc=kamezawa.hiroyu@jp.fujitsu.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=yinghan@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.