svn commit: r324992 - head/sys/ufs/ffs

Mark Johnston markj at FreeBSD.org
Wed Oct 25 17:20:19 UTC 2017


Author: markj
Date: Wed Oct 25 17:20:18 2017
New Revision: 324992
URL: https://svnweb.freebsd.org/changeset/base/324992

Log:
  Make drain_output() use bufobj_wwait().
  
  No functional change intended.
  
  Reviewed by:	kib
  MFC after:	1 week
  Differential Revision:	https://reviews.freebsd.org/D12790

Modified:
  head/sys/ufs/ffs/ffs_softdep.c

Modified: head/sys/ufs/ffs/ffs_softdep.c
==============================================================================
--- head/sys/ufs/ffs/ffs_softdep.c	Wed Oct 25 16:01:19 2017	(r324991)
+++ head/sys/ufs/ffs/ffs_softdep.c	Wed Oct 25 17:20:18 2017	(r324992)
@@ -14289,25 +14289,14 @@ softdep_get_depcounts(struct mount *mp,
 
 /*
  * Wait for pending output on a vnode to complete.
- * Must be called with vnode lock and interlock locked.
- *
- * XXX: Should just be a call to bufobj_wwait().
  */
 static void
 drain_output(vp)
 	struct vnode *vp;
 {
-	struct bufobj *bo;
 
-	bo = &vp->v_bufobj;
 	ASSERT_VOP_LOCKED(vp, "drain_output");
-	ASSERT_BO_WLOCKED(bo);
-
-	while (bo->bo_numoutput) {
-		bo->bo_flag |= BO_WWAIT;
-		msleep((caddr_t)&bo->bo_numoutput,
-		    BO_LOCKPTR(bo), PRIBIO + 1, "drainvp", 0);
-	}
+	(void)bufobj_wwait(&vp->v_bufobj, 0, 0);
 }
 
 /*


More information about the svn-src-head mailing list