d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

- Add a missing vn_start_write() to flushbufqueues(). This could have

Browse Source 
caused snapshot related problems.
 - The vp can not be NULL here or we would panic in vfs_bio_awrite().  Stop
   confusing the logic by checking for it in several places.

Submitted by:	kirk and then rototilled by me to remove vp == NULL checks.
This commit is contained in:
jeff 2003-10-05 22:16:08 +00:00
parent 11b574e32b
commit 6be3f23255
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
sys/kern/vfs_bio.c
View File

@ -2135,6 +2135,7 @@ flushbufqueues(int flushdeps)
{
struct thread *td = curthread;
struct vnode *vp;
struct mount *mp;
struct buf *bp;
int hasdeps;
@ -2176,12 +2177,16 @@ flushbufqueues(int flushdeps)
* of vnode followed by buf lock. This is ok because
* the NOWAIT will prevent deadlock.
*/
if ((vp = bp->b_vp) == NULL ||
vn_lock(vp, LK_EXCLUSIVE | LK_NOWAIT, td) == 0) {
vp = bp->b_vp;
if (vn_start_write(vp, &mp, V_NOWAIT) != 0) {
BUF_UNLOCK(bp);
continue;
}
if (vn_lock(vp, LK_EXCLUSIVE | LK_NOWAIT, td) == 0) {
mtx_unlock(&bqlock);
vfs_bio_awrite(bp);
if (vp != NULL)
VOP_UNLOCK(vp, 0, td);
vn_finished_write(mp);
VOP_UNLOCK(vp, 0, td);
flushwithdeps += hasdeps;
return (1);
}