diff options
| author | Andreea-Cristina Bernat <bernat.ada@gmail.com> | 2017-03-16 16:27:45 +0000 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2017-03-16 16:27:45 +0000 |
| commit | df8a09d1b8f9e693ec3f6b7e0162fc817f2cf0db (patch) | |
| tree | e712d74d4cdd8f449f0b367108508d686242cc33 /fs/afs | |
| parent | 1d7e4ebf291d3103466006926329e39aa355944f (diff) | |
| download | op-kernel-dev-df8a09d1b8f9e693ec3f6b7e0162fc817f2cf0db.zip op-kernel-dev-df8a09d1b8f9e693ec3f6b7e0162fc817f2cf0db.tar.gz | |
afs: security: Replace rcu_assign_pointer() with RCU_INIT_POINTER()
The use of "rcu_assign_pointer()" is NULLing out the pointer.
According to RCU_INIT_POINTER()'s block comment:
"1. This use of RCU_INIT_POINTER() is NULLing out the pointer"
it is better to use it instead of rcu_assign_pointer() because it has a
smaller overhead.
The following Coccinelle semantic patch was used:
@@
@@
- rcu_assign_pointer
+ RCU_INIT_POINTER
(..., NULL)
Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs')
| -rw-r--r-- | fs/afs/security.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/afs/security.c b/fs/afs/security.c index bfa9d34..ecb86a6 100644 --- a/fs/afs/security.c +++ b/fs/afs/security.c @@ -114,7 +114,7 @@ void afs_clear_permits(struct afs_vnode *vnode) mutex_lock(&vnode->permits_lock); permits = vnode->permits; - rcu_assign_pointer(vnode->permits, NULL); + RCU_INIT_POINTER(vnode->permits, NULL); mutex_unlock(&vnode->permits_lock); if (permits) |
