linux-kernel - Re: [PATCHSET cgroup/for-3.16] cgroup: implement cftype->write()

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5371BD0D.9080801@huawei.com>
Date:	Tue, 13 May 2014 14:34:53 +0800
From:	Li Zefan <lizefan@...wei.com>
To:	Tejun Heo <tj@...nel.org>
CC:	<cgroups@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<vgoyal@...hat.com>, <axboe@...nel.dk>, <hannes@...xchg.org>,
	<mhocko@...e.cz>, <arozansk@...hat.com>
Subject: Re: [PATCHSET cgroup/for-3.16] cgroup: implement cftype->write()

On 2014/5/6 20:44, Tejun Heo wrote:
> Hello,
> 
> This patchset implements a new cftype operation ->write() and replace
> ->write_string() and ->trigger() with it.  ->write() is similar to
> ->write_string() but maps directly to the kernfs write operation and
> has access to all available context information including the
> associated kernfs_open_file, which will make things like implementing
> kernfs active protection manipulation easier.
> 
> ->write_string() becomes redundant and ->trigger() has always been
> spurious.  This patch replaces all users of the two methods to
> ->write() and remove them.
> 
> This patchset contains the following five patches.
> 
>  0001-cgroup-implement-cftype-write.patch
>  0002-cgroup-replace-cftype-write_string-with-cftype-write.patch
>  0003-cgroup-replace-cftype-trigger-with-cftype-write.patch
>  0004-cgroup-convert-tasks-and-cgroup.procs-handle-to-use-.patch
>  0005-cgroup-remove-cgroup-control_kn.patch
> 
> 0001 implements cftype->write().
> 
> 0002-0003 replace cftype->write_string() and ->trigger() with it.
> 
> 0004 convert "tasks" and "cgroup.procs" handler too.
> 
> 0005 removes cgroup->control_kn as the kernfs_node is now directly
> accessible from kernfs_open_file.
> 
> This patchset is on top of
> 
>   cgroup/for-3.16 12d3089c192c ("kernel/cpuset.c: convert printk to pr_foo()")
> + [1] [PATCHSET cgroup/for-3.16] cgroup: post unified hierarchy fixes and updates
> 
> and available on the following git branch.
> 
>  git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-post-unified-updates
> 
> diffstat follows.
> 
>  block/blk-throttle.c      |   32 ++++++++---------
>  block/cfq-iosched.c       |   28 +++++++--------
>  include/linux/cgroup.h    |   40 +++++++++++----------
>  kernel/cgroup.c           |   85 +++++++++++++++++++++++-----------------------
>  kernel/cgroup_freezer.c   |   20 ++++------
>  kernel/cpuset.c           |   16 ++++----
>  mm/hugetlb_cgroup.c       |   33 +++++++++--------
>  mm/memcontrol.c           |   80 +++++++++++++++++++++++--------------------
>  net/ipv4/tcp_memcontrol.c |   31 +++++++++-------
>  security/device_cgroup.c  |   14 +++----
>  10 files changed, 197 insertions(+), 182 deletions(-)
> 

Acked-by: Li Zefan <lizefan@...wei.com>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/