Hello,

On the legacy hierarchies, different controllers implemented interface
without much guidance and as a result there isn't much consistency
across controllers even when they're doing similar things.

Controllers which already implemented interface for the unified
hierarchy are already pretty consistent and this patchset explicitly
sets up the conventions so that it's easier to make future conversions
conformant.

For blkcg, the only thing which needs to be changed is the range and
default value of io.weight which is currently [10, 1000] and 500.  For
the unified hierarchy, cgroup now defines the standard range and
default value to be [1, 10000] and 100 so that there's enough room for
100x bias in both directions.

This patchset contains the following four patches.

 0001-cgroup-export-cgrp_dfl_root.patch
 0002-cgroup-define-controller-file-conventions.patch
 0003-blkcg-s-CFQ_WEIGHT_-CFQ_WEIGHT_LEGACY_.patch
 0004-blkcg-use-CGROUP_WEIGHT_-scale-for-io.weight-on-the-.patch

0001-0002 explicitly defines the controller interface file conventions
on the unified hierarchy.  0003-0004 update blkcg so that it conforms
to the standard weight range.

This patchset is also available in the following git branch.

 git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git 
review-blkcg-normalize-weight

and is on top of

  block/for-linus f3f5da624e0a ("block: Do a full clone when splitting discard 
bios")
+ [1] [PATCHSET block/for-4.3] writeback: cgroup writeback updates
+ [2] [PATCHSET v2 block/for-4.3] block, cgroup: make cfq charge async IOs to 
the appropriate blkcgs
+ [3] [PATCHSET v3 block/for-4.3] blkcg: blkcg policy methods and data handling 
cleanup
+ [4] [PATCHSET v2 block/for-4.3] blkcg: blkcg stats cleanup
+ [5] [PATCHSET block/for-4.3] blkcg: implement interface for the unified 
hierarchy

diffstat follows.  Thanks.

 Documentation/cgroups/unified-hierarchy.txt |   77 ++++++++++++++++++++++++----
 block/blk-cgroup.c                          |   21 +++++++
 block/cfq-iosched.c                         |   63 ++++++++++++++++------
 include/linux/blk-cgroup.h                  |    2 
 include/linux/cgroup.h                      |    9 +++
 kernel/cgroup.c                             |    1 
 6 files changed, 145 insertions(+), 28 deletions(-)

--
tejun

[L] http://lkml.kernel.org/g/1435268337-1738-1-git-send-email...@kernel.org
[1] http://lkml.kernel.org/g/1436281823-1947-1-git-send-email...@kernel.org
[2] http://lkml.kernel.org/g/1436283361-3889-1-git-send-email...@kernel.org
[3] http://lkml.kernel.org/g/1436637654-28110-1-git-send-email...@kernel.org
[4] http://lkml.kernel.org/g/1436724043-12986-1-git-send-email...@kernel.org
[5] http://lkml.kernel.org/g/1437763434-28699-1-git-send-email...@kernel.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to