diff options
author | Artur Grabowski <art@cvs.openbsd.org> | 1999-01-10 21:12:09 +0000 |
---|---|---|
committer | Artur Grabowski <art@cvs.openbsd.org> | 1999-01-10 21:12:09 +0000 |
commit | 4d70f72cd3a27a14bff429292e9a5b2932e76d12 (patch) | |
tree | 6a2b2a1af9e20d9fa17c974ef6075e98316bfd97 /sys/ufs/mfs | |
parent | 052b43163f6e0a3b05a14c19b4e3f3de80fe7162 (diff) |
fix panic messages and put some code in DIAGNOSTIC
Diffstat (limited to 'sys/ufs/mfs')
-rw-r--r-- | sys/ufs/mfs/mfs_vnops.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/sys/ufs/mfs/mfs_vnops.c b/sys/ufs/mfs/mfs_vnops.c index 689df7e9265..47772fb45cb 100644 --- a/sys/ufs/mfs/mfs_vnops.c +++ b/sys/ufs/mfs/mfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mfs_vnops.c,v 1.7 1997/11/06 05:59:24 csapuntz Exp $ */ +/* $OpenBSD: mfs_vnops.c,v 1.8 1999/01/10 21:12:08 art Exp $ */ /* $NetBSD: mfs_vnops.c,v 1.8 1996/03/17 02:16:32 christos Exp $ */ /* @@ -125,10 +125,12 @@ mfs_open(v) struct proc *a_p; } */ *ap = v; +#ifdef DIAGNOSTIC if (ap->a_vp->v_type != VBLK) { - panic("mfs_ioctl not VBLK"); + panic("mfs_open not VBLK"); /* NOTREACHED */ } +#endif return (0); } @@ -169,8 +171,10 @@ mfs_strategy(v) struct vnode *vp; struct proc *p = curproc; /* XXX */ +#ifdef DIAGNOSTIC if (!vfinddev(bp->b_dev, VBLK, &vp) || vp->v_usecount == 0) panic("mfs_strategy: bad dev"); +#endif mfsp = VTOMFS(vp); /* check for mini-root access */ if (mfsp->mfs_pid == 0) { @@ -272,14 +276,18 @@ mfs_close(v) */ if ((error = vinvalbuf(vp, 1, ap->a_cred, ap->a_p, 0, 0)) != 0) return (error); +#ifdef DIAGNOSTIC /* * There should be no way to have any more uses of this * vnode, so if we find any other uses, it is a panic. */ if (vp->v_usecount > 1) printf("mfs_close: ref count %d > 1\n", vp->v_usecount); + if (mfsp->mfs_buflist) + printf("mfs_close: dirty buffers\n"); if (vp->v_usecount > 1 || mfsp->mfs_buflist) panic("mfs_close"); +#endif /* * Send a request to the filesystem server to exit. */ @@ -302,9 +310,11 @@ mfs_inactive(v) } */ *ap = v; register struct mfsnode *mfsp = VTOMFS(ap->a_vp); +#ifdef DIAGNOSTIC if (mfsp->mfs_buflist && mfsp->mfs_buflist != (struct buf *)(-1)) panic("mfs_inactive: not inactive (mfs_buflist %p)", mfsp->mfs_buflist); +#endif VOP_UNLOCK(ap->a_vp, 0, ap->a_p); return (0); } |