diff options
author | Tejun Heo <tj@kernel.org> | 2013-11-29 10:42:58 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2013-11-29 10:42:58 -0500 |
commit | ac1e69aa78e2e799a8d5475595ba744bdf29b597 (patch) | |
tree | 5a33e778c533270e44b6243337d8b49f83b37555 /include/linux/cgroup.h | |
parent | c729b11edf7458a939556e1889446184f42f37a7 (diff) | |
download | blackbird-op-linux-ac1e69aa78e2e799a8d5475595ba744bdf29b597.tar.gz blackbird-op-linux-ac1e69aa78e2e799a8d5475595ba744bdf29b597.zip |
cgroup: don't skip seq_open on write only opens on pidlist files
Currently, cgroup_pidlist_open() skips seq_open() and pidlist loading
if the file is opened write-only, which is a sensible optimization as
pidlist loading can be costly and there often are occasions where
tasks or cgroup.procs is opened write-only. However, pidlist init and
release are planned to be moved to cgroup_pidlist_start/stop()
respectively which would make this optimization unnecessary.
This patch removes the optimization and always fully initializes
pidlist files regardless of open mode. This will help moving pidlist
handling to start/stop by unifying rw paths and removes the need for
specifying cftype->release() in addition to .release in
cgroup_pidlist_operations as file->f_op is now always overridden. As
pidlist files were the only user of cftype->release(), the next patch
will remove the method.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
Diffstat (limited to 'include/linux/cgroup.h')
0 files changed, 0 insertions, 0 deletions