diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2013-10-25 16:41:01 -0400 |
|---|---|---|
| committer | Jiri Slaby <jslaby@suse.cz> | 2014-09-15 18:50:43 +0200 |
| commit | c9258b208f7e5ad213449a83468d3be2ad09d266 (patch) | |
| tree | aa8ebea55648638917d63bb0115f7dd2053965d5 /fs | |
| parent | 70279f38568703022d4790d382d1f6462f8b6383 (diff) | |
dcache.c: get rid of pointless macros
commit 482db9066199813d6b999b65a3171afdbec040b6 upstream.
D_HASH{MASK,BITS} are used once each, both in the same function (d_hash()).
At this point they are actively misguiding - they imply that values are
compiler constants, which is no longer true.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/dcache.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 8ef74f3d8fe5..5859bc5c981d 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -125,8 +125,6 @@ static inline void done_seqretry(seqlock_t *lock, int seq) * This hash-function tries to avoid losing too many bits of hash * information, yet avoid using a prime hash-size or similar. */ -#define D_HASHBITS d_hash_shift -#define D_HASHMASK d_hash_mask static unsigned int d_hash_mask __read_mostly; static unsigned int d_hash_shift __read_mostly; @@ -137,8 +135,8 @@ static inline struct hlist_bl_head *d_hash(const struct dentry *parent, unsigned int hash) { hash += (unsigned long) parent / L1_CACHE_BYTES; - hash = hash + (hash >> D_HASHBITS); - return dentry_hashtable + (hash & D_HASHMASK); + hash = hash + (hash >> d_hash_shift); + return dentry_hashtable + (hash & d_hash_mask); } /* Statistics gathering. */ |