diff options
| author | mckusick <mckusick@FreeBSD.org> | 2000-01-13 20:03:22 +0000 |
|---|---|---|
| committer | mckusick <mckusick@FreeBSD.org> | 2000-01-13 20:03:22 +0000 |
| commit | 7eac0e762afc8308262687ae9f51e41336831d8d (patch) | |
| tree | aa014fa104b9af60ce5bbb13f3ae1e54cfcf9b6f /sys | |
| parent | e22c8a3445990960db9634c58b4bb5d1f1752c19 (diff) | |
| download | FreeBSD-src-7eac0e762afc8308262687ae9f51e41336831d8d.zip FreeBSD-src-7eac0e762afc8308262687ae9f51e41336831d8d.tar.gz | |
Confirming Peter's fix (locking 101: release the lock before you go
to sleep). Locking 101, part 2: do not look at buffer contents after
you have been asleep. There is no telling what wonderous changes may
have occurred.
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/contrib/softupdates/ffs_softdep.c | 2 | ||||
| -rw-r--r-- | sys/ufs/ffs/ffs_softdep.c | 2 |
2 files changed, 0 insertions, 4 deletions
diff --git a/sys/contrib/softupdates/ffs_softdep.c b/sys/contrib/softupdates/ffs_softdep.c index c4cd06d..b0e7d04 100644 --- a/sys/contrib/softupdates/ffs_softdep.c +++ b/sys/contrib/softupdates/ffs_softdep.c @@ -4566,8 +4566,6 @@ getdirtybuf(bpp, waitfor) FREE_LOCK_INTERLOCKED(&lk); tsleep(&bp->b_xflags, PRIBIO, "getbuf", 0); ACQUIRE_LOCK_INTERLOCKED(&lk); - if (bp->b_xflags & BX_BKGRDINPROG) - panic("getdirtybuf: still writing"); continue; } if (waitfor != MNT_WAIT) diff --git a/sys/ufs/ffs/ffs_softdep.c b/sys/ufs/ffs/ffs_softdep.c index c4cd06d..b0e7d04 100644 --- a/sys/ufs/ffs/ffs_softdep.c +++ b/sys/ufs/ffs/ffs_softdep.c @@ -4566,8 +4566,6 @@ getdirtybuf(bpp, waitfor) FREE_LOCK_INTERLOCKED(&lk); tsleep(&bp->b_xflags, PRIBIO, "getbuf", 0); ACQUIRE_LOCK_INTERLOCKED(&lk); - if (bp->b_xflags & BX_BKGRDINPROG) - panic("getdirtybuf: still writing"); continue; } if (waitfor != MNT_WAIT) |
