cgroup: fix cftype->file_offset handling
authorTejun Heo <tj@kernel.org>
Thu, 5 Nov 2015 05:12:24 +0000 (00:12 -0500)
committerTejun Heo <tj@kernel.org>
Mon, 16 Nov 2015 15:58:26 +0000 (10:58 -0500)
commit34c06254ff82a815fdccdfae7517a06c9b768cee
tree95b146ae8a28f04a91a84983a407b463460662cb
parent8005c49d9aea74d382f474ce11afbbc7d7130bec
cgroup: fix cftype->file_offset handling

6f60eade2433 ("cgroup: generalize obtaining the handles of and
notifying cgroup files") introduced cftype->file_offset so that the
handles for per-css file instances can be recorded.  These handles
then can be used, for example, to generate file modified
notifications.

Unfortunately, it made the wrong assumption that files are created
once for a given css and removed on its destruction.  Due to the
dependencies among subsystems, a css may be hidden from userland and
then later shown again.  This is implemented by removing and
re-creating the affected files, so the associated kernfs_node for a
given cgroup file may change over time.  This incorrect assumption led
to the corruption of css->files lists.

Reimplement cftype->file_offset handling so that cgroup_file->kn is
protected by a lock and updated as files are created and destroyed.
This also makes keeping them on per-cgroup list unnecessary.

Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: James Sedgwick <jsedgwick@fb.com>
Fixes: 6f60eade2433 ("cgroup: generalize obtaining the handles of and notifying cgroup files")
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Acked-by: Zefan Li <lizefan@huawei.com>
include/linux/cgroup-defs.h
include/linux/cgroup.h
kernel/cgroup.c