LINUX: consolidate duplicate code in canonical_dentry 91/12791/5
authorMark Vitale <mvitale@sinenomine.net>
Thu, 30 Nov 2017 21:51:32 +0000 (16:51 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Tue, 2 Jan 2018 02:06:07 +0000 (21:06 -0500)
The two stanzas for HAVE_DCACHE_LOCK are now identical;
remove the preprocessor conditionals and duplicate code.

No functional change should be incurred by this commit.

Change-Id: I15cd4631d1932dcfb920313acb82fcbe570087e8
Reviewed-on: https://gerrit.openafs.org/12791
Reviewed-by: Andrew Deason <adeason@dson.org>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/afs/LINUX/osi_vnodeops.c

index a9394ee..d8dfd93 100644 (file)
@@ -917,17 +917,10 @@ canonical_dentry(struct inode *ip)
 
     vcp->target_link = ret;
 
-# ifdef HAVE_DCACHE_LOCK
     if (ret) {
        afs_linux_dget(ret);
     }
     afs_d_alias_unlock(ip);
-# else
-    if (ret) {
-       afs_linux_dget(ret);
-    }
-    afs_d_alias_unlock(ip);
-# endif
 
     return ret;
 }