Return statements in functions returning bool should use true or false
instead of an integer value.
This issue was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
if (dentry->d_lockref.count > 1) {
dentry->d_lockref.count--;
spin_unlock(&dentry->d_lock);
if (dentry->d_lockref.count > 1) {
dentry->d_lockref.count--;
spin_unlock(&dentry->d_lock);
}
/*
* If we weren't the last ref, we're done.
*/
if (ret)
}
/*
* If we weren't the last ref, we're done.
*/
if (ret)
/*
* Careful, careful. The reference count went down
/*
* Careful, careful. The reference count went down
/* Nothing to do? Dropping the reference was all we needed? */
if (d_flags == (DCACHE_REFERENCED | DCACHE_LRU_LIST) && !d_unhashed(dentry))
/* Nothing to do? Dropping the reference was all we needed? */
if (d_flags == (DCACHE_REFERENCED | DCACHE_LRU_LIST) && !d_unhashed(dentry))
/*
* Not the fast normal case? Get the lock. We've already decremented
/*
* Not the fast normal case? Get the lock. We've already decremented
*/
if (dentry->d_lockref.count) {
spin_unlock(&dentry->d_lock);
*/
if (dentry->d_lockref.count) {
spin_unlock(&dentry->d_lock);
* set it to 1.
*/
dentry->d_lockref.count = 1;
* set it to 1.
*/
dentry->d_lockref.count = 1;