diff options
| author | Jeff Roberson <jeff@FreeBSD.org> | 2003-03-13 07:04:11 +0000 |
|---|---|---|
| committer | Jeff Roberson <jeff@FreeBSD.org> | 2003-03-13 07:04:11 +0000 |
| commit | 619bddc702d7de3de4d606e2afc81804369d67e7 (patch) | |
| tree | 5c8950e330aa692f14fbaf472ca9949158bf99b2 /sys/nfsclient | |
| parent | 542bd65fcb546ee9ae0f4262b64e365f25636a74 (diff) | |
| download | src-619bddc702d7de3de4d606e2afc81804369d67e7.tar.gz src-619bddc702d7de3de4d606e2afc81804369d67e7.zip | |
- Lock the buf before inspecting its contents.
Notes
Notes:
svn path=/head/; revision=112178
Diffstat (limited to 'sys/nfsclient')
| -rw-r--r-- | sys/nfsclient/nfs_vnops.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/sys/nfsclient/nfs_vnops.c b/sys/nfsclient/nfs_vnops.c index 2a137b77f627..253f0c745a64 100644 --- a/sys/nfsclient/nfs_vnops.c +++ b/sys/nfsclient/nfs_vnops.c @@ -2658,18 +2658,16 @@ again: for (bp = TAILQ_FIRST(&vp->v_dirtyblkhd); bp; bp = nbp) { if (bvecpos >= bvecsize) break; - if ((bp->b_flags & (B_DELWRI | B_NEEDCOMMIT)) != - (B_DELWRI | B_NEEDCOMMIT)) { + if (BUF_LOCK(bp, LK_EXCLUSIVE | LK_NOWAIT, NULL)) { nbp = TAILQ_NEXT(bp, b_vnbufs); continue; } - if (BUF_LOCK(bp, - LK_EXCLUSIVE | LK_NOWAIT | LK_INTERLOCK, - VI_MTX(vp))) { - VI_LOCK(vp); + if ((bp->b_flags & (B_DELWRI | B_NEEDCOMMIT)) != + (B_DELWRI | B_NEEDCOMMIT)) { nbp = TAILQ_NEXT(bp, b_vnbufs); continue; } + VI_UNLOCK(vp); bremfree(bp); /* * Work out if all buffers are using the same cred |