diff options
author | Monam Agarwal <monamagarwal123@gmail.com> | 2014-03-23 18:47:18 (GMT) |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-03-24 12:48:02 (GMT) |
commit | 01a971406177c2ca9834be6914a67e20f463a3e6 (patch) | |
tree | 0e1808318398da6ae3c13d6cb279487c318e9352 /kernel | |
parent | e1b2dc176f2d5be7952c47a4e4e8f3b06a90db1c (diff) | |
download | linux-01a971406177c2ca9834be6914a67e20f463a3e6.tar.xz |
cgroup: Use RCU_INIT_POINTER(x, NULL) in cgroup.c
This patch replaces rcu_assign_pointer(x, NULL) with
RCU_INIT_POINTER(x, NULL)
The rcu_assign_pointer() ensures that the initialization of a
structure is carried out before storing a pointer to that structure.
And in the case of the NULL pointer, there is no structure to
initialize. So, rcu_assign_pointer(p, NULL) can be safely converted
to RCU_INIT_POINTER(p, NULL)
Signed-off-by: Monam Agarwal <monamagarwal123@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/cgroup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 58c67b3..e378cb2 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c @@ -3545,7 +3545,7 @@ static void css_release(struct percpu_ref *ref) struct cgroup_subsys_state *css = container_of(ref, struct cgroup_subsys_state, refcnt); - rcu_assign_pointer(css->cgroup->subsys[css->ss->id], NULL); + RCU_INIT_POINTER(css->cgroup->subsys[css->ss->id], NULL); call_rcu(&css->rcu_head, css_free_rcu_fn); } |