Author: kib
Date: Tue Sep  8 14:43:42 2009
New Revision: 196974
URL: http://svn.freebsd.org/changeset/base/196974

Log:
  MFC r196692:
  Make the mnt_writeopcount and mnt_secondary_writes counters,
  used by the suspension code, not greater then mnt_ref reference
  counter value.
  
  MFC r196733:
  Fix mount reference leak when V_XSLEEP is specified to vn_start_write().
  
  Approved by:  re (kensmith)

Modified:
  stable/8/sys/   (props changed)
  stable/8/sys/amd64/include/xen/   (props changed)
  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
  stable/8/sys/contrib/dev/acpica/   (props changed)
  stable/8/sys/contrib/pf/   (props changed)
  stable/8/sys/dev/xen/xenpci/   (props changed)
  stable/8/sys/kern/vfs_vnops.c

Modified: stable/8/sys/kern/vfs_vnops.c
==============================================================================
--- stable/8/sys/kern/vfs_vnops.c       Tue Sep  8 14:19:14 2009        
(r196973)
+++ stable/8/sys/kern/vfs_vnops.c       Tue Sep  8 14:43:42 2009        
(r196974)
@@ -999,7 +999,8 @@ vn_start_write(vp, mpp, flags)
                goto unlock;
        mp->mnt_writeopcount++;
 unlock:
-       MNT_REL(mp);
+       if (error != 0 || (flags & V_XSLEEP) != 0)
+               MNT_REL(mp);
        MNT_IUNLOCK(mp);
        return (error);
 }
@@ -1049,7 +1050,6 @@ vn_start_secondary_write(vp, mpp, flags)
        if ((mp->mnt_kern_flag & (MNTK_SUSPENDED | MNTK_SUSPEND2)) == 0) {
                mp->mnt_secondary_writes++;
                mp->mnt_secondary_accwrites++;
-               MNT_REL(mp);
                MNT_IUNLOCK(mp);
                return (0);
        }
@@ -1081,6 +1081,7 @@ vn_finished_write(mp)
        if (mp == NULL)
                return;
        MNT_ILOCK(mp);
+       MNT_REL(mp);
        mp->mnt_writeopcount--;
        if (mp->mnt_writeopcount < 0)
                panic("vn_finished_write: neg cnt");
@@ -1103,6 +1104,7 @@ vn_finished_secondary_write(mp)
        if (mp == NULL)
                return;
        MNT_ILOCK(mp);
+       MNT_REL(mp);
        mp->mnt_secondary_writes--;
        if (mp->mnt_secondary_writes < 0)
                panic("vn_finished_secondary_write: neg cnt");
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to