fold swapping ->d_name.hash into switch_names()
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 24 Sep 2014 19:27:39 +0000 (12:27 -0700)
committerAl Viro <viro@zeniv.linux.org.uk>
Sat, 27 Sep 2014 19:59:11 +0000 (15:59 -0400)
and do it along with ->d_name.len there

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/dcache.c

index 92f7d76db498723cc30a3bd2a3b398c0932e556f..7599d35b4ae558be1064dde50ad32295914443bc 100644 (file)
@@ -2412,7 +2412,7 @@ static void switch_names(struct dentry *dentry, struct dentry *target)
                        }
                }
        }
-       swap(dentry->d_name.len, target->d_name.len);
+       swap(dentry->d_name.hash_len, target->d_name.hash_len);
 }
 
 static void dentry_lock_for_move(struct dentry *dentry, struct dentry *target)
@@ -2510,7 +2510,6 @@ static void __d_move(struct dentry *dentry, struct dentry *target,
 
        /* Switch the names.. */
        switch_names(dentry, target);
-       swap(dentry->d_name.hash, target->d_name.hash);
 
        /* ... and switch them in the tree */
        if (IS_ROOT(dentry)) {