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 | 1d7e4ebf291d3103466006926329e39aa355944f (patch) | |
| tree | a4b2f6323bdcb246dd921905df7a54ffb818ac14 /fs/afs | |
| parent | 944c74f472f926785b1948efa0e73e2f1b3b539b (diff) | |
| download | linux-1d7e4ebf291d3103466006926329e39aa355944f.tar.gz linux-1d7e4ebf291d3103466006926329e39aa355944f.tar.bz2 linux-1d7e4ebf291d3103466006926329e39aa355944f.zip | |
afs: inode: 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/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/afs/inode.c b/fs/afs/inode.c index ade6ec3873cf..e083e086b7ca 100644 --- a/fs/afs/inode.c +++ b/fs/afs/inode.c @@ -445,7 +445,7 @@ void afs_evict_inode(struct inode *inode) 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) call_rcu(&permits->rcu, afs_zap_permits); |