]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
SOLARIS: Avoid open count cleanup for Solaris 11
authorAndrew Deason <adeason@sinenomine.net>
Tue, 8 Jan 2013 23:50:57 +0000 (17:50 -0600)
committerStephan Wiesand <stephan.wiesand@desy.de>
Thu, 10 Jan 2013 16:05:10 +0000 (08:05 -0800)
The comments in here no longer apply to Solaris, as of OpenSolaris
commit 11736:63a134e1f09c by Donghai Qiao (4492533 Filesystems may
need VOP_CLOSE() for executables following a VOP_OPEN()). This means
that this workaround should no longer be necessary for any Solaris 11
release, any illumos release, and anything else based off of
OpenSolaris. So, stop doing it.

Thanks to Frank Batschulat for pointing this out, and providing all of
the details.

Reviewed-on: http://gerrit.openafs.org/8895
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit 08a4b05c27964dd06d7922c134dc2dccc062b379)

Change-Id: Ic689ac75fd618945ae7a6e101f29c3610d414603
Reviewed-on: http://gerrit.openafs.org/8897
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Paul Smeddle <paul.smeddle@gmail.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/afs/SOLARIS/osi_vnodeops.c

index c74b87bcdb4741b2f2c3af3a4d068e42781ecfde..f2fe46d3c2e1caa392a1dffc4ea6e785eb34c044 100644 (file)
@@ -1832,6 +1832,7 @@ afs_inactive(struct vcache *avc, afs_ucred_t *acred)
     }
     mutex_exit(&vp->v_lock);
 
+#ifndef AFS_SUN511_ENV
     /*
      * Solaris calls VOP_OPEN on exec, but doesn't call VOP_CLOSE when
      * the executable exits.  So we clean up the open count here.
@@ -1842,6 +1843,7 @@ afs_inactive(struct vcache *avc, afs_ucred_t *acred)
      */
     if (avc->opens > 0 && avc->mvstat == 0 && !(avc->f.states & CCore))
        avc->opens = avc->execsOrWriters = 0;
+#endif
 
     afs_InactiveVCache(avc, acred);