diff options
author | Tejun Heo <tj@kernel.org> | 2016-12-27 14:49:09 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2016-12-27 14:49:09 -0500 |
commit | e0aed7c74f0bf6c643103f3a10ff988b7ee6545a (patch) | |
tree | b35d04595b30594bf4d9fac4759d10a2570590e1 /kernel | |
parent | dcfe149b9f45aaf89bb95e8b314210da626417d9 (diff) |
cgroup: fix RCU related sparse warnings
kn->priv which is a void * is used as a RCU pointer by cgroup. When
dereferencing it, it was passing kn->priv to rcu_derefreence() without
casting it into a RCU pointer triggering address space mismatch
warning from sparse. Fix them.
Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Acked-by: Acked-by: Zefan Li <lizefan@huawei.com>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/cgroup/cgroup-v1.c | 2 | ||||
-rw-r--r-- | kernel/cgroup/cgroup.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/kernel/cgroup/cgroup-v1.c b/kernel/cgroup/cgroup-v1.c index 37be09e09740..465b10111eaf 100644 --- a/kernel/cgroup/cgroup-v1.c +++ b/kernel/cgroup/cgroup-v1.c | |||
@@ -694,7 +694,7 @@ int cgroupstats_build(struct cgroupstats *stats, struct dentry *dentry) | |||
694 | * @kn->priv is RCU safe. Let's do the RCU dancing. | 694 | * @kn->priv is RCU safe. Let's do the RCU dancing. |
695 | */ | 695 | */ |
696 | rcu_read_lock(); | 696 | rcu_read_lock(); |
697 | cgrp = rcu_dereference(kn->priv); | 697 | cgrp = rcu_dereference(*(void __rcu __force **)&kn->priv); |
698 | if (!cgrp || cgroup_is_dead(cgrp)) { | 698 | if (!cgrp || cgroup_is_dead(cgrp)) { |
699 | rcu_read_unlock(); | 699 | rcu_read_unlock(); |
700 | mutex_unlock(&cgroup_mutex); | 700 | mutex_unlock(&cgroup_mutex); |
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c index b6b9068ef468..d9d82e96d67f 100644 --- a/kernel/cgroup/cgroup.c +++ b/kernel/cgroup/cgroup.c | |||
@@ -4932,7 +4932,7 @@ struct cgroup_subsys_state *css_tryget_online_from_dir(struct dentry *dentry, | |||
4932 | * have been or be removed at any point. @kn->priv is RCU | 4932 | * have been or be removed at any point. @kn->priv is RCU |
4933 | * protected for this access. See css_release_work_fn() for details. | 4933 | * protected for this access. See css_release_work_fn() for details. |
4934 | */ | 4934 | */ |
4935 | cgrp = rcu_dereference(kn->priv); | 4935 | cgrp = rcu_dereference(*(void __rcu __force **)&kn->priv); |
4936 | if (cgrp) | 4936 | if (cgrp) |
4937 | css = cgroup_css(cgrp, ss); | 4937 | css = cgroup_css(cgrp, ss); |
4938 | 4938 | ||