]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
LINUX: consolidate duplicate code in canonical_dentry
authorMark Vitale <mvitale@sinenomine.net>
Thu, 30 Nov 2017 21:51:32 +0000 (16:51 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Wed, 3 Jan 2018 05:44:10 +0000 (00:44 -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.

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>
(cherry picked from commit 0678ad26b6069040a6ea86866fb59ef5968ea343)

Change-Id: If0f9516201cea747a753db04ba2d0e2cac69971b
Reviewed-on: https://gerrit.openafs.org/12849
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/afs/LINUX/osi_vnodeops.c

index a9394ee3760c64e8bbaf20c5e76f98084f95e7a5..d8dfd93e9a5172789ba73a86062c3c1b5c33993a 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;
 }