From: Glauber Costa <glommer@gmail.com> To: Sha Zhengju <handai.szj@gmail.com> Cc: Michal Hocko <mhocko@suse.cz>, "linux-mm@kvack.org" <linux-mm@kvack.org>, Cgroups <cgroups@vger.kernel.org>, Greg Thelen <gthelen@google.com>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Andrew Morton <akpm@linux-foundation.org>, Wu Fengguang <fengguang.wu@intel.com>, Mel Gorman <mgorman@suse.de>, Sha Zhengju <handai.szj@taobao.com> Subject: Re: [PATCH V4 5/6] memcg: patch mem_cgroup_{begin,end}_update_page_stat() out if only root memcg exists Date: Mon, 15 Jul 2013 10:32:26 +0400 [thread overview] Message-ID: <20130715063224.GA3745@localhost.localdomain> (raw) In-Reply-To: <CAFj3OHXF+ZjnaDS2L6ZmuHPx20+7XC9r-s7Gh=_TYOr4Opr4Bw@mail.gmail.com> On Fri, Jul 12, 2013 at 09:13:56PM +0800, Sha Zhengju wrote: > Ooops.... it seems unreachable, change Glauber's email... > > > On Fri, Jul 12, 2013 at 8:59 PM, Sha Zhengju <handai.szj@gmail.com> wrote: > > > Add cc to Glauber > > Thanks > > > > On Thu, Jul 11, 2013 at 10:56 PM, Michal Hocko <mhocko@suse.cz> wrote: > > > On Sat 06-07-13 01:33:43, Sha Zhengju wrote: > > >> From: Sha Zhengju <handai.szj@taobao.com> > > >> > > >> If memcg is enabled and no non-root memcg exists, all allocated > > >> pages belongs to root_mem_cgroup and wil go through root memcg > > >> statistics routines. So in order to reduce overheads after adding > > >> memcg dirty/writeback accounting in hot paths, we use jump label to > > >> patch mem_cgroup_{begin,end}_update_page_stat() in or out when not > > >> used. > > > > > > I do not think this is enough. How much do you save? One atomic read. > > > This doesn't seem like a killer. > > > > > > I hoped we could simply not account at all and move counters to the root > > > cgroup once the label gets enabled. > > > > I have thought of this approach before, but it would probably run into > > another issue, e.g, each zone has a percpu stock named ->pageset to > > optimize the increment and decrement operations, and I haven't figure out a > > simpler and cheaper approach to handle that stock numbers if moving global > > counters to root cgroup, maybe we can just leave them and can afford the > > approximation? > > > > Glauber have already done lots of works here, in his previous patchset he > > also tried to move some global stats to root ( > > http://comments.gmane.org/gmane.linux.kernel.cgroups/6291). May I steal > > some of your ideas here, Glauber? :P > > Of course. Please go ahead and keep me posted in my new address. -- 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>
WARNING: multiple messages have this Message-ID (diff)
From: Glauber Costa <glommer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> To: Sha Zhengju <handai.szj-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Michal Hocko <mhocko-AlSwsSmVLrQ@public.gmane.org>, "linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org" <linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org>, Cgroups <cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Greg Thelen <gthelen-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>, KAMEZAWA Hiroyuki <kamezawa.hiroyu-+CUm20s59erQFUHtdCDX3A@public.gmane.org>, Andrew Morton <akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>, Wu Fengguang <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Mel Gorman <mgorman-l3A5Bk7waGM@public.gmane.org>, Sha Zhengju <handai.szj-3b8fjiQLQpfQT0dZR+AlfA@public.gmane.org> Subject: Re: [PATCH V4 5/6] memcg: patch mem_cgroup_{begin,end}_update_page_stat() out if only root memcg exists Date: Mon, 15 Jul 2013 10:32:26 +0400 [thread overview] Message-ID: <20130715063224.GA3745@localhost.localdomain> (raw) In-Reply-To: <CAFj3OHXF+ZjnaDS2L6ZmuHPx20+7XC9r-s7Gh=_TYOr4Opr4Bw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> On Fri, Jul 12, 2013 at 09:13:56PM +0800, Sha Zhengju wrote: > Ooops.... it seems unreachable, change Glauber's email... > > > On Fri, Jul 12, 2013 at 8:59 PM, Sha Zhengju <handai.szj-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > > > Add cc to Glauber > > Thanks > > > > On Thu, Jul 11, 2013 at 10:56 PM, Michal Hocko <mhocko-AlSwsSmVLrQ@public.gmane.org> wrote: > > > On Sat 06-07-13 01:33:43, Sha Zhengju wrote: > > >> From: Sha Zhengju <handai.szj-3b8fjiQLQpfQT0dZR+AlfA@public.gmane.org> > > >> > > >> If memcg is enabled and no non-root memcg exists, all allocated > > >> pages belongs to root_mem_cgroup and wil go through root memcg > > >> statistics routines. So in order to reduce overheads after adding > > >> memcg dirty/writeback accounting in hot paths, we use jump label to > > >> patch mem_cgroup_{begin,end}_update_page_stat() in or out when not > > >> used. > > > > > > I do not think this is enough. How much do you save? One atomic read. > > > This doesn't seem like a killer. > > > > > > I hoped we could simply not account at all and move counters to the root > > > cgroup once the label gets enabled. > > > > I have thought of this approach before, but it would probably run into > > another issue, e.g, each zone has a percpu stock named ->pageset to > > optimize the increment and decrement operations, and I haven't figure out a > > simpler and cheaper approach to handle that stock numbers if moving global > > counters to root cgroup, maybe we can just leave them and can afford the > > approximation? > > > > Glauber have already done lots of works here, in his previous patchset he > > also tried to move some global stats to root ( > > http://comments.gmane.org/gmane.linux.kernel.cgroups/6291). May I steal > > some of your ideas here, Glauber? :P > > Of course. Please go ahead and keep me posted in my new address.
next prev parent reply other threads:[~2013-07-15 6:32 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-07-05 17:18 [PATCH V4 0/6] Memcg dirty/writeback page accounting Sha Zhengju 2013-07-05 17:21 ` [PATCH V4 1/6] memcg: remove MEMCG_NR_FILE_MAPPED Sha Zhengju 2013-07-11 13:39 ` Michal Hocko 2013-07-11 15:53 ` Sha Zhengju 2013-07-05 17:26 ` [PATCH V4 2/6] fs/ceph: vfs __set_page_dirty_nobuffers interface instead of doing it inside filesystem Sha Zhengju [not found] ` <1373044710-27371-1-git-send-email-handai.szj-3b8fjiQLQpfQT0dZR+AlfA@public.gmane.org> 2013-07-05 17:30 ` [PATCH V4 3/6] memcg: add per cgroup dirty pages accounting Sha Zhengju 2013-07-05 17:30 ` Sha Zhengju 2013-07-11 14:31 ` Michal Hocko 2013-07-11 16:49 ` Sha Zhengju 2013-07-05 17:32 ` [PATCH V4 4/6] memcg: add per cgroup writeback " Sha Zhengju 2013-07-11 14:40 ` Michal Hocko 2013-07-11 14:40 ` Michal Hocko [not found] ` <20130711144003.GJ21667-2MMpYkNvuYDjFM9bn6wA6Q@public.gmane.org> 2013-07-11 16:56 ` Sha Zhengju 2013-07-11 16:56 ` Sha Zhengju 2013-07-05 17:33 ` [PATCH V4 5/6] memcg: patch mem_cgroup_{begin,end}_update_page_stat() out if only root memcg exists Sha Zhengju 2013-07-11 14:56 ` Michal Hocko 2013-07-12 12:59 ` Sha Zhengju 2013-07-12 13:13 ` Sha Zhengju 2013-07-15 6:32 ` Glauber Costa [this message] 2013-07-15 6:32 ` Glauber Costa 2013-07-12 13:25 ` Michal Hocko 2013-07-12 13:25 ` Michal Hocko 2013-07-13 4:15 ` Sha Zhengju 2013-07-15 17:58 ` Greg Thelen 2013-07-16 4:26 ` Sha Zhengju 2013-07-05 17:34 ` [PATCH V4 6/6] memcg: Document cgroup dirty/writeback memory statistics 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=20130715063224.GA3745@localhost.localdomain \ --to=glommer@gmail.com \ --cc=akpm@linux-foundation.org \ --cc=cgroups@vger.kernel.org \ --cc=fengguang.wu@intel.com \ --cc=gthelen@google.com \ --cc=handai.szj@gmail.com \ --cc=handai.szj@taobao.com \ --cc=kamezawa.hiroyu@jp.fujitsu.com \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ --cc=mhocko@suse.cz \ /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.