cgroups.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: lizefan@huawei.com
Cc: cgroups@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCHSET cgroup/for-3.17] cgroup: distinguish the default and legacy hierarchies when handling cftypes
Date: Wed,  2 Jul 2014 19:50:50 -0400	[thread overview]
Message-ID: <1404345055-8196-1-git-send-email-tj@kernel.org> (raw)

Hello,

Until now, cftype arrays carried files for both the default and legacy
hierarchies and the files which needed to be used on only one of them
were flagged with either CFTYPE_ONLY_ON_DFL or CFTYPE_INSANE.  This
gets confusing very quickly and we may end up exposing interface files
to the default hierarchy without thinking it through.

This patchset makes cgroup core provide separate sets of interfaces
for cftype handling so that the cftypes for the default and legacy
hierarchies are clearly distinguished.  This makes all the existing
subsystem interface files legacy-only by default and all subsystems
will have no interface file created when enabled on the default
hierarchy.  Each subsystem should explicitly review and compose the
interface for the default hierarchy.

This patchset contains the following five patches.

 0001-cgroup-split-cgroup_base_files-into-cgroup_-dfl-lega.patch
 0002-cgroup-rename-cgroup_subsys-base_cftypes-to-legacy_c.patch
 0003-cgroup-replace-cgroup_add_cftypes-with-cgroup_add_le.patch
 0004-cgroup-distinguish-the-default-and-legacy-hierarchie.patch
 0005-cgroup-make-CFTYPE_ONLY_ON_DFL-and-CFTYPE_NO_-intern.patch

This patchset is on top of

  a497c3ba1d97 ("Linux 3.16-rc2")
+ [1] [PATCHSET cgroup/for-3.17] cgroup: remove sane_behavior support on non-default hierarchies

and available in the following git branch.

 git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-dfl_files

diffstat follows.  Thanks.

 block/blk-cgroup.c           |    5 -
 include/linux/cgroup.h       |   17 +++--
 kernel/cgroup.c              |  146 ++++++++++++++++++++++++++++++++-----------
 kernel/cgroup_freezer.c      |    2
 kernel/cpuset.c              |    2
 kernel/sched/core.c          |    2
 kernel/sched/cpuacct.c       |    2
 mm/hugetlb_cgroup.c          |    5 -
 mm/memcontrol.c              |    6 -
 net/core/netclassid_cgroup.c |    2
 net/core/netprio_cgroup.c    |    2
 net/ipv4/tcp_memcontrol.c    |    2
 security/device_cgroup.c     |    2
 13 files changed, 139 insertions(+), 56 deletions(-)

--
tejun

[1] http://lkml.kernel.org/g/1404344747-7957-1-git-send-email-tj@kernel.org

             reply	other threads:[~2014-07-02 23:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-02 23:50 Tejun Heo [this message]
2014-07-02 23:50 ` [PATCH 2/5] cgroup: rename cgroup_subsys->base_cftypes to ->legacy_cftypes Tejun Heo
     [not found]   ` <1404345055-8196-3-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2014-07-03 11:02     ` Neil Horman
2014-07-02 23:50 ` [PATCH 3/5] cgroup: replace cgroup_add_cftypes() with cgroup_add_legacy_cftypes() Tejun Heo
     [not found]   ` <1404345055-8196-4-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2014-07-03 11:08     ` Neil Horman
2014-07-02 23:50 ` [PATCH 4/5] cgroup: distinguish the default and legacy hierarchies when handling cftypes Tejun Heo
     [not found]   ` <1404345055-8196-5-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2014-07-03 11:14     ` Neil Horman
2014-07-03 15:20     ` [PATCH v2 " Tejun Heo
2014-07-14  2:13     ` [PATCH " Li Zefan
     [not found]       ` <53C33CC7.3020707-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2014-07-14 14:42         ` Tejun Heo
2014-07-02 23:50 ` [PATCH 5/5] cgroup: make CFTYPE_ONLY_ON_DFL and CFTYPE_NO_ internal to cgroup core Tejun Heo
     [not found] ` <1404345055-8196-1-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2014-07-02 23:50   ` [PATCH 1/5] cgroup: split cgroup_base_files[] into cgroup_{dfl|legacy}_base_files[] Tejun Heo
     [not found]     ` <1404345055-8196-2-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2014-07-04 17:11       ` [PATCH v2 " Tejun Heo
2014-07-03 15:21   ` [PATCH 6/5] cgroup: initialize cgrp_dfl_root_inhibit_ss_mask from !->dfl_files test 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=1404345055-8196-1-git-send-email-tj@kernel.org \
    --to=tj@kernel.org \
    --cc=cgroups@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizefan@huawei.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).