summaryrefslogtreecommitdiffstats
path: root/sys/ufs
diff options
context:
space:
mode:
authortrasz <trasz@FreeBSD.org>2008-12-16 21:13:11 +0000
committertrasz <trasz@FreeBSD.org>2008-12-16 21:13:11 +0000
commitb2515a861b17c67bef0ad8eb3e85cdaffe242388 (patch)
tree69e7de47346c349ddf6f303959a812bf891ea077 /sys/ufs
parent4039bb7c824968c46ec054cb3ea586cc81de6623 (diff)
downloadFreeBSD-src-b2515a861b17c67bef0ad8eb3e85cdaffe242388.zip
FreeBSD-src-b2515a861b17c67bef0ad8eb3e85cdaffe242388.tar.gz
According to phk@, VOP_STRATEGY should never, _ever_, return
anything other than 0. Make it so. This fixes "panic: VOP_STRATEGY failed bp=0xc320dd90 vp=0xc3b9f648", encountered when writing to an orphaned filesystem. Reason for the panic was the following assert: KASSERT(i == 0, ("VOP_STRATEGY failed bp=%p vp=%p", bp, bp->b_vp)); at vfs_bio:bufstrategy(). Reviewed by: scottl, phk Approved by: rwatson (mentor) Sponsored by: FreeBSD Foundation
Diffstat (limited to 'sys/ufs')
-rw-r--r--sys/ufs/ufs/ufs_vnops.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/ufs/ufs/ufs_vnops.c b/sys/ufs/ufs/ufs_vnops.c
index e5047e9..7b639e8 100644
--- a/sys/ufs/ufs/ufs_vnops.c
+++ b/sys/ufs/ufs/ufs_vnops.c
@@ -2013,7 +2013,7 @@ ufs_strategy(ap)
bp->b_error = error;
bp->b_ioflags |= BIO_ERROR;
bufdone(bp);
- return (error);
+ return (0);
}
if ((long)bp->b_blkno == -1)
vfs_bio_clrbuf(bp);
OpenPOWER on IntegriCloud