diff options
author | mckusick <mckusick@FreeBSD.org> | 2018-02-02 00:52:29 +0000 |
---|---|---|
committer | mckusick <mckusick@FreeBSD.org> | 2018-02-02 00:52:29 +0000 |
commit | 330151004a6f6a1c10107a928648ccc380eafe71 (patch) | |
tree | 880d430f79d1375033bbb5ac1206949a4b5c590d | |
parent | da03586749f36d6ca4908c00e9d4490a8c22a5cc (diff) | |
download | FreeBSD-src-330151004a6f6a1c10107a928648ccc380eafe71.zip FreeBSD-src-330151004a6f6a1c10107a928648ccc380eafe71.tar.gz |
MFC of 328444.
Eliminate "fsync: giving up on dirty" messages.
-rw-r--r-- | sys/kern/vfs_default.c | 23 |
1 files changed, 17 insertions, 6 deletions
diff --git a/sys/kern/vfs_default.c b/sys/kern/vfs_default.c index 21c4a71..9075e2c 100644 --- a/sys/kern/vfs_default.c +++ b/sys/kern/vfs_default.c @@ -629,13 +629,21 @@ vop_stdfsync(ap) struct thread *a_td; } */ *ap; { - struct vnode *vp = ap->a_vp; - struct buf *bp; + struct vnode *vp; + struct buf *bp, *nbp; struct bufobj *bo; - struct buf *nbp; - int error = 0; - int maxretry = 1000; /* large, arbitrarily chosen */ + struct mount *mp; + int error, maxretry; + error = 0; + maxretry = 10000; /* large, arbitrarily chosen */ + vp = ap->a_vp; + mp = NULL; + if (vp->v_type == VCHR) { + VI_LOCK(vp); + mp = vp->v_rdev->si_mountpt; + VI_UNLOCK(vp); + } bo = &vp->v_bufobj; BO_LOCK(bo); loop1: @@ -678,6 +686,8 @@ loop2: bremfree(bp); bawrite(bp); } + if (maxretry < 1000) + pause("dirty", hz < 1000 ? 1 : hz / 1000); BO_LOCK(bo); goto loop2; } @@ -699,7 +709,8 @@ loop2: TAILQ_FOREACH(bp, &bo->bo_dirty.bv_hd, b_bobufs) if ((error = bp->b_error) == 0) continue; - if (error == 0 && --maxretry >= 0) + if ((mp != NULL && mp->mnt_secondary_writes > 0) || + (error == 0 && --maxretry >= 0)) goto loop1; error = EAGAIN; } |