From: Sha Zhengju <handai.szj@gmail.com> To: Andrew Morton <akpm@linux-foundation.org> Cc: Michal Hocko <mhocko@suse.cz>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Greg Thelen <gthelen@google.com>, "linux-mm@kvack.org" <linux-mm@kvack.org>, Cgroups <cgroups@vger.kernel.org>, "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>, Sha Zhengju <handai.szj@taobao.com> Subject: Fwd: [PATCH V5 5/8] memcg: add per cgroup writeback pages accounting Date: Thu, 22 Aug 2013 17:46:31 +0800 [thread overview] Message-ID: <CAFj3OHXy5XkwhxKk=WNywp2pq__FD7BrSQwFkp+NZj15_k6BEQ@mail.gmail.com> (raw) In-Reply-To: <20130804100855.GD24005@dhcp22.suse.cz> Hi Andrew, After several rounds of review, parts of the memcg page accounting patchset is ready (leaving memcg dirty page accounting under more review): 1/8 memcg: remove MEMCG_NR_FILE_MAPPED 2/8 ceph: vfs __set_page_dirty_nobuffers interface instead of doing it inside filesystem 3/8 memcg: check for proper lock held in mem_cgroup_update_page_stat 5/8 memcg: add per cgroup writeback pages accounting 8/8 memcg: Document cgroup dirty/writeback memory statistics But the 2/8 ceph one has been improved again and will be merged in ceph tree, so only the other 4 patches need to be added to -mm tree. I've moved the 5/8 writeback one up the stack and updated the 8/8 to only document writeback changes. Could you please merge them earlier? I'll post these updated patches soon. :) Thank you! ---------- Forwarded message ---------- From: Michal Hocko <mhocko@suse.cz> Date: Sun, Aug 4, 2013 at 6:08 PM Subject: Re: [PATCH V5 5/8] memcg: add per cgroup writeback pages accounting To: Sha Zhengju <handai.szj@gmail.com> Cc: "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>, "linux-mm@kvack.org" <linux-mm@kvack.org>, Cgroups <cgroups@vger.kernel.org>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Glauber Costa <glommer@gmail.com>, Greg Thelen <gthelen@google.com>, Wu Fengguang <fengguang.wu@intel.com>, Andrew Morton <akpm@linux-foundation.org>, Sha Zhengju <handai.szj@taobao.com> On Sat 03-08-13 17:25:01, Sha Zhengju wrote: > On Thu, Aug 1, 2013 at 10:53 PM, Michal Hocko <mhocko@suse.cz> wrote: > > On Thu 01-08-13 19:54:11, Sha Zhengju wrote: > >> From: Sha Zhengju <handai.szj@taobao.com> > >> > >> Similar to dirty page, we add per cgroup writeback pages accounting. The lock > >> rule still is: > >> mem_cgroup_begin_update_page_stat() > >> modify page WRITEBACK stat > >> mem_cgroup_update_page_stat() > >> mem_cgroup_end_update_page_stat() > >> > >> There're two writeback interfaces to modify: test_{clear/set}_page_writeback(). > >> Lock order: > >> --> memcg->move_lock > >> --> mapping->tree_lock > >> > >> Signed-off-by: Sha Zhengju <handai.szj@taobao.com> > > > > Looks good to me. Maybe I would suggest moving this patch up the stack > > so that it might get merged earlier as it is simpler than dirty pages > > accounting. Unless you insist on having the full series merged at once. > > I think the following three patches can be merged earlier: > 1/8 memcg: remove MEMCG_NR_FILE_MAPPED > 3/8 memcg: check for proper lock held in mem_cgroup_update_page_stat > 5/8 memcg: add per cgroup writeback pages accounting > > Do I need to resent them again for you or they're enough? This is a question for Andrew. I would go with them as they are. > One more word, since dirty accounting is essential to future memcg > dirty page throttling and it is not an optional feature now, I suspect > whether we can merge the following two as well and leave the overhead > optimization a separate series. :p I wouldn't hurry it. We need numbers for serious testing to see the overhead. It is still just a small step towards dirty throttling. > 4/5 memcg: add per cgroup dirty pages accounting > 8/8 memcg: Document cgroup dirty/writeback memory statistics > > The 2/8 ceph one still need more improvement, I'll separate it next version. > > > > > Acked-by: Michal Hocko <mhocko@suse.cz> > > Thank you. [...] -- Michal Hocko SUSE Labs -- Thanks, Sha
WARNING: multiple messages have this Message-ID (diff)
From: Sha Zhengju <handai.szj@gmail.com> To: Andrew Morton <akpm@linux-foundation.org> Cc: Michal Hocko <mhocko@suse.cz>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Greg Thelen <gthelen@google.com>, "linux-mm@kvack.org" <linux-mm@kvack.org>, Cgroups <cgroups@vger.kernel.org>, "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>, Sha Zhengju <handai.szj@taobao.com> Subject: Fwd: [PATCH V5 5/8] memcg: add per cgroup writeback pages accounting Date: Thu, 22 Aug 2013 17:46:31 +0800 [thread overview] Message-ID: <CAFj3OHXy5XkwhxKk=WNywp2pq__FD7BrSQwFkp+NZj15_k6BEQ@mail.gmail.com> (raw) In-Reply-To: <20130804100855.GD24005@dhcp22.suse.cz> Hi Andrew, After several rounds of review, parts of the memcg page accounting patchset is ready (leaving memcg dirty page accounting under more review): 1/8 memcg: remove MEMCG_NR_FILE_MAPPED 2/8 ceph: vfs __set_page_dirty_nobuffers interface instead of doing it inside filesystem 3/8 memcg: check for proper lock held in mem_cgroup_update_page_stat 5/8 memcg: add per cgroup writeback pages accounting 8/8 memcg: Document cgroup dirty/writeback memory statistics But the 2/8 ceph one has been improved again and will be merged in ceph tree, so only the other 4 patches need to be added to -mm tree. I've moved the 5/8 writeback one up the stack and updated the 8/8 to only document writeback changes. Could you please merge them earlier? I'll post these updated patches soon. :) Thank you! ---------- Forwarded message ---------- From: Michal Hocko <mhocko@suse.cz> Date: Sun, Aug 4, 2013 at 6:08 PM Subject: Re: [PATCH V5 5/8] memcg: add per cgroup writeback pages accounting To: Sha Zhengju <handai.szj@gmail.com> Cc: "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>, "linux-mm@kvack.org" <linux-mm@kvack.org>, Cgroups <cgroups@vger.kernel.org>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Glauber Costa <glommer@gmail.com>, Greg Thelen <gthelen@google.com>, Wu Fengguang <fengguang.wu@intel.com>, Andrew Morton <akpm@linux-foundation.org>, Sha Zhengju <handai.szj@taobao.com> On Sat 03-08-13 17:25:01, Sha Zhengju wrote: > On Thu, Aug 1, 2013 at 10:53 PM, Michal Hocko <mhocko@suse.cz> wrote: > > On Thu 01-08-13 19:54:11, Sha Zhengju wrote: > >> From: Sha Zhengju <handai.szj@taobao.com> > >> > >> Similar to dirty page, we add per cgroup writeback pages accounting. The lock > >> rule still is: > >> mem_cgroup_begin_update_page_stat() > >> modify page WRITEBACK stat > >> mem_cgroup_update_page_stat() > >> mem_cgroup_end_update_page_stat() > >> > >> There're two writeback interfaces to modify: test_{clear/set}_page_writeback(). > >> Lock order: > >> --> memcg->move_lock > >> --> mapping->tree_lock > >> > >> Signed-off-by: Sha Zhengju <handai.szj@taobao.com> > > > > Looks good to me. Maybe I would suggest moving this patch up the stack > > so that it might get merged earlier as it is simpler than dirty pages > > accounting. Unless you insist on having the full series merged at once. > > I think the following three patches can be merged earlier: > 1/8 memcg: remove MEMCG_NR_FILE_MAPPED > 3/8 memcg: check for proper lock held in mem_cgroup_update_page_stat > 5/8 memcg: add per cgroup writeback pages accounting > > Do I need to resent them again for you or they're enough? This is a question for Andrew. I would go with them as they are. > One more word, since dirty accounting is essential to future memcg > dirty page throttling and it is not an optional feature now, I suspect > whether we can merge the following two as well and leave the overhead > optimization a separate series. :p I wouldn't hurry it. We need numbers for serious testing to see the overhead. It is still just a small step towards dirty throttling. > 4/5 memcg: add per cgroup dirty pages accounting > 8/8 memcg: Document cgroup dirty/writeback memory statistics > > The 2/8 ceph one still need more improvement, I'll separate it next version. > > > > > Acked-by: Michal Hocko <mhocko@suse.cz> > > Thank you. [...] -- Michal Hocko SUSE Labs -- Thanks, Sha -- 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:[~2013-08-22 9:46 UTC|newest] Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-08-01 11:43 [PATCH V5 0/8] Add memcg dirty/writeback page accounting Sha Zhengju 2013-08-01 11:43 ` Sha Zhengju 2013-08-01 11:44 ` [PATCH 1/8] memcg: remove MEMCG_NR_FILE_MAPPED Sha Zhengju 2013-08-01 11:51 ` [PATCH V5 2/8] fs/ceph: vfs __set_page_dirty_nobuffers interface instead of doing it inside filesystem Sha Zhengju 2013-08-01 15:19 ` Yan, Zheng 2013-08-01 18:27 ` Sage Weil 2013-08-02 10:04 ` Sha Zhengju [not found] ` <CAFj3OHVXvtr5BDMrGatHZi7M9y+dh1ZKRMQZGjZmNBcg3pNQtw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2013-08-02 20:30 ` Sage Weil 2013-08-02 20:30 ` Sage Weil 2013-08-03 8:58 ` Sha Zhengju 2013-08-02 9:04 ` Sha Zhengju 2013-08-02 9:04 ` Sha Zhengju 2013-08-02 13:11 ` Yan, Zheng 2013-08-01 11:52 ` [PATCH V5 3/8] memcg: check for proper lock held in mem_cgroup_update_page_stat Sha Zhengju 2013-08-01 14:34 ` Michal Hocko 2013-08-01 14:34 ` Michal Hocko 2013-08-04 18:48 ` Greg Thelen 2013-08-04 18:48 ` Greg Thelen 2013-08-01 11:53 ` [PATCH V5 4/8] memcg: add per cgroup dirty pages accounting Sha Zhengju 2013-08-01 11:54 ` [PATCH V5 5/8] memcg: add per cgroup writeback " Sha Zhengju 2013-08-01 14:53 ` Michal Hocko [not found] ` <20130801145302.GJ5198-2MMpYkNvuYDjFM9bn6wA6Q@public.gmane.org> 2013-08-03 9:25 ` Sha Zhengju 2013-08-03 9:25 ` Sha Zhengju [not found] ` <CAFj3OHV-VCKJfe6bv4UMvv+uj4LELDXsieRZFJD06Yrdyy=XxA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2013-08-04 10:08 ` Michal Hocko 2013-08-04 10:08 ` Michal Hocko 2013-08-22 9:46 ` Sha Zhengju [this message] 2013-08-22 9:46 ` Fwd: " Sha Zhengju 2013-08-22 9:50 ` [PATCH 1/4] memcg: remove MEMCG_NR_FILE_MAPPED Sha Zhengju 2013-08-22 9:52 ` [PATCH 2/4] memcg: check for proper lock held in mem_cgroup_update_page_stat Sha Zhengju 2013-08-22 9:53 ` [PATCH 3/4] memcg: add per cgroup writeback pages accounting Sha Zhengju 2013-08-22 22:40 ` Andrew Morton 2013-08-23 16:11 ` Sha Zhengju 2013-08-23 16:11 ` Sha Zhengju 2013-08-22 9:53 ` [PATCH 4/4] memcg: Document cgroup dirty/writeback memory statistics Sha Zhengju 2013-08-04 18:51 ` [PATCH V5 5/8] memcg: add per cgroup writeback pages accounting Greg Thelen 2013-08-04 18:51 ` Greg Thelen 2013-08-01 11:55 ` [PATCH V5 6/8] memcg: make nocpu_base available for non-hotplug Sha Zhengju 2013-08-01 12:00 ` [PATCH V5 7/8] memcg: don't account root memcg page stats if only root exists Sha Zhengju 2013-08-01 16:20 ` Johannes Weiner 2013-08-02 4:32 ` Sha Zhengju 2013-08-02 4:32 ` Sha Zhengju 2013-08-05 21:58 ` Johannes Weiner 2013-08-05 21:58 ` Johannes Weiner 2013-08-01 12:00 ` [PATCH V5 8/8] memcg: Document cgroup dirty/writeback memory statistics Sha Zhengju 2013-08-01 12:00 ` Sha Zhengju 2013-08-01 14:43 ` [PATCH V5 0/8] Add memcg dirty/writeback page accounting Michal Hocko 2013-08-03 9:30 ` 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='CAFj3OHXy5XkwhxKk=WNywp2pq__FD7BrSQwFkp+NZj15_k6BEQ@mail.gmail.com' \ --to=handai.szj@gmail.com \ --cc=akpm@linux-foundation.org \ --cc=cgroups@vger.kernel.org \ --cc=gthelen@google.com \ --cc=handai.szj@taobao.com \ --cc=kamezawa.hiroyu@jp.fujitsu.com \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --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.