diff options
| author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2018-08-02 02:39:05 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2018-08-05 21:52:44 +0200 |
| commit | 7964410fcf135d7e76deef4e475816ec02482f7b (patch) | |
| tree | c8274087ac8bfa6678a4e6c3acd5886a6e728f68 /fs/dcache.c | |
| parent | fold generic_readlink() into its only caller (diff) | |
| download | linux-7964410fcf135d7e76deef4e475816ec02482f7b.tar.xz linux-7964410fcf135d7e76deef4e475816ec02482f7b.zip | |
fs: dcache: Use true and false for boolean values
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>
Diffstat (limited to 'fs/dcache.c')
| -rw-r--r-- | fs/dcache.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 0e8e5de3c48a..6fd5c1aa4620 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -732,16 +732,16 @@ static inline bool fast_dput(struct dentry *dentry) if (dentry->d_lockref.count > 1) { dentry->d_lockref.count--; spin_unlock(&dentry->d_lock); - return 1; + return true; } - return 0; + return false; } /* * If we weren't the last ref, we're done. */ if (ret) - return 1; + return true; /* * Careful, careful. The reference count went down @@ -770,7 +770,7 @@ static inline bool fast_dput(struct dentry *dentry) /* Nothing to do? Dropping the reference was all we needed? */ if (d_flags == (DCACHE_REFERENCED | DCACHE_LRU_LIST) && !d_unhashed(dentry)) - return 1; + return true; /* * Not the fast normal case? Get the lock. We've already decremented @@ -787,7 +787,7 @@ static inline bool fast_dput(struct dentry *dentry) */ if (dentry->d_lockref.count) { spin_unlock(&dentry->d_lock); - return 1; + return true; } /* @@ -796,7 +796,7 @@ static inline bool fast_dput(struct dentry *dentry) * set it to 1. */ dentry->d_lockref.count = 1; - return 0; + return false; } |