From: Andrew Deason Date: Tue, 8 Jan 2013 23:50:57 +0000 (-0600) Subject: SOLARIS: Avoid open count cleanup for Solaris 11 X-Git-Tag: upstream/1.6.2_pre3^2~1 X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=42257f27e69b7ff3f4c0d72a916423280dfd93e0;p=packages%2Fo%2Fopenafs.git SOLARIS: Avoid open count cleanup for Solaris 11 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 Reviewed-by: Derrick Brashear (cherry picked from commit 08a4b05c27964dd06d7922c134dc2dccc062b379) Change-Id: Ic689ac75fd618945ae7a6e101f29c3610d414603 Reviewed-on: http://gerrit.openafs.org/8897 Tested-by: BuildBot Reviewed-by: Paul Smeddle Reviewed-by: Stephan Wiesand --- diff --git a/src/afs/SOLARIS/osi_vnodeops.c b/src/afs/SOLARIS/osi_vnodeops.c index c74b87bcd..f2fe46d3c 100644 --- a/src/afs/SOLARIS/osi_vnodeops.c +++ b/src/afs/SOLARIS/osi_vnodeops.c @@ -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);