From: Tejun Heo <tj@kernel.org>
To: lizefan@huawei.com
Cc: cgroups@vger.kernel.org, linux-kernel@vger.kernel.org,
hannes@cmpxchg.org, mhocko@suse.cz, vgoyal@redhat.com,
axboe@kernel.dk
Subject: [PATCHSET cgroup/for-3.17] cgroup, blkcg, memcg: make blkcg depend on memcg on unified hierarchy
Date: Fri, 27 Jun 2014 21:03:06 -0400 [thread overview]
Message-ID: <1403917392-32555-1-git-send-email-tj@kernel.org> (raw)
Hello, guys.
Currently, the blkio subsystem attributes all of writeback IOs to the
root. One of the issues is that there's no way to tell who originated
a writeback IO from block layer. Those IOs are usually issued
asynchronously from a task which didn't have anything to do with
actually generating the dirty pages. The memory subsystem, when
enabled, already keeps track of the ownership of each dirty page and
it's desirable for blkio to piggyback instead of adding its own
per-page tag.
This can be achieved on the unified hierarchy without too much
difficulty. This patchset implements a dependency mechanism in the
cgroup such that a subsystem can depends on other subsystems. If
available, the depended-upon subsystems are enabled together
implicitly when the subsystem is turned on. Implicitly enabled
subsystems are invisible and the dependencies are transparent to
userland.
This patchset implements the dependency mechanism in cgroup core and
make blkcg depend on memcg. This doesn't actually solve the writeback
problem yet but is an important step.
This patchset contains the following six patches.
0001-cgroup-reorganize-cgroup_subtree_control_write.patch
0002-cgroup-introduce-cgroup-subtree_control.patch
0003-cgroup-make-interface-files-visible-iff-enabled-on-c.patch
0004-cgroup-implement-cgroup_subsys-css_reset.patch
0005-cgroup-implement-cgroup_subsys-depends_on.patch
0006-blkcg-memcg-make-blkcg-depend-on-memcg-on-the-defaul.patch
0001-0005 gradually implement the dependency mechanism.
0006 makes blkcg depend on memcg.
This patchset is on top of a497c3ba1d97 ("Linux 3.16-rc2") and
available in the following git branch.
git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-cgroup-dependency
diffstat follows. Thanks.
Documentation/cgroups/cgroups.txt | 14 +
Documentation/cgroups/unified-hierarchy.txt | 23 ++-
block/blk-cgroup.c | 7
include/linux/cgroup.h | 20 ++
kernel/cgroup.c | 201 ++++++++++++++++++++++------
mm/memcontrol.c | 24 +++
6 files changed, 243 insertions(+), 46 deletions(-)
--
tejun
next reply other threads:[~2014-06-28 1:03 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-28 1:03 Tejun Heo [this message]
2014-06-28 1:03 ` [PATCH 1/6] cgroup: reorganize cgroup_subtree_control_write() Tejun Heo
2014-06-28 1:03 ` [PATCH 2/6] cgroup: introduce cgroup->subtree_control Tejun Heo
2014-06-28 1:03 ` [PATCH 3/6] cgroup: make interface files visible iff enabled on cgroup->subtree_control Tejun Heo
2014-06-28 1:03 ` [PATCH 4/6] cgroup: implement cgroup_subsys->css_reset() Tejun Heo
2014-06-28 1:03 ` [PATCH 5/6] cgroup: implement cgroup_subsys->depends_on Tejun Heo
2014-06-28 1:03 ` [PATCH 6/6] blkcg, memcg: make blkcg depend on memcg on the default hierarchy Tejun Heo
2014-06-28 11:49 ` [PATCH v2 " Tejun Heo
2014-07-08 19:42 ` Vivek Goyal
2014-07-08 21:53 ` Tejun Heo
2014-07-09 11:57 ` Vivek Goyal
2014-07-04 6:29 ` [PATCHSET cgroup/for-3.17] cgroup, blkcg, memcg: make blkcg depend on memcg on unified hierarchy Li Zefan
2014-07-07 18:33 ` Johannes Weiner
2014-07-08 22:03 ` Tejun Heo
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=1403917392-32555-1-git-send-email-tj@kernel.org \
--to=tj@kernel.org \
--cc=axboe@kernel.dk \
--cc=cgroups@vger.kernel.org \
--cc=hannes@cmpxchg.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lizefan@huawei.com \
--cc=mhocko@suse.cz \
--cc=vgoyal@redhat.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).