diff options
author | phk <phk@FreeBSD.org> | 2004-10-27 06:48:21 +0000 |
---|---|---|
committer | phk <phk@FreeBSD.org> | 2004-10-27 06:48:21 +0000 |
commit | f3795e5212484aa4f59911ad7498e0d0d27e6275 (patch) | |
tree | 59e4b9cc1138a6536a4c0b9b13ba5009fc6189c9 /sys/fs | |
parent | e5378319aa408712dac3cd6643d8b4f8008cbed2 (diff) | |
download | FreeBSD-src-f3795e5212484aa4f59911ad7498e0d0d27e6275.zip FreeBSD-src-f3795e5212484aa4f59911ad7498e0d0d27e6275.tar.gz |
Eliminate unnecessary KASSERTs.
Don't use bp->b_vp in VOP_STRATEGY: the vnode is passed in as an argument.
Diffstat (limited to 'sys/fs')
-rw-r--r-- | sys/fs/cd9660/cd9660_vnops.c | 4 | ||||
-rw-r--r-- | sys/fs/hpfs/hpfs_vnops.c | 2 | ||||
-rw-r--r-- | sys/fs/msdosfs/msdosfs_vnops.c | 6 | ||||
-rw-r--r-- | sys/fs/ntfs/ntfs_vnops.c | 4 | ||||
-rw-r--r-- | sys/fs/udf/udf_vnops.c | 2 | ||||
-rw-r--r-- | sys/fs/unionfs/union_vnops.c | 6 |
6 files changed, 6 insertions, 18 deletions
diff --git a/sys/fs/cd9660/cd9660_vnops.c b/sys/fs/cd9660/cd9660_vnops.c index afc6a2e..5407dff 100644 --- a/sys/fs/cd9660/cd9660_vnops.c +++ b/sys/fs/cd9660/cd9660_vnops.c @@ -714,11 +714,9 @@ cd9660_strategy(ap) } */ *ap; { struct buf *bp = ap->a_bp; - struct vnode *vp = bp->b_vp; + struct vnode *vp = ap->a_vp; struct iso_node *ip; - KASSERT(ap->a_vp == ap->a_bp->b_vp, ("%s(%p != %p)", - __func__, ap->a_vp, ap->a_bp->b_vp)); ip = VTOI(vp); if (vp->v_type == VBLK || vp->v_type == VCHR) panic("cd9660_strategy: spec"); diff --git a/sys/fs/hpfs/hpfs_vnops.c b/sys/fs/hpfs/hpfs_vnops.c index 6f91358..7831aed 100644 --- a/sys/fs/hpfs/hpfs_vnops.c +++ b/sys/fs/hpfs/hpfs_vnops.c @@ -650,8 +650,6 @@ hpfs_strategy(ap) daddr_t blkno; int error; - KASSERT(ap->a_vp == ap->a_bp->b_vp, ("%s(%p != %p)", - __func__, ap->a_vp, ap->a_bp->b_vp)); dprintf(("hpfs_strategy(): \n")); if (vp->v_type == VBLK || vp->v_type == VCHR) diff --git a/sys/fs/msdosfs/msdosfs_vnops.c b/sys/fs/msdosfs/msdosfs_vnops.c index 9e32989..207e6d8 100644 --- a/sys/fs/msdosfs/msdosfs_vnops.c +++ b/sys/fs/msdosfs/msdosfs_vnops.c @@ -1772,15 +1772,11 @@ msdosfs_strategy(ap) } */ *ap; { struct buf *bp = ap->a_bp; - struct denode *dep = VTODE(bp->b_vp); struct vnode *vp; + struct denode *dep = VTODE(ap->a_vp); int error = 0; daddr_t blkno; - KASSERT(ap->a_vp == ap->a_bp->b_vp, ("%s(%p != %p)", - __func__, ap->a_vp, ap->a_bp->b_vp)); - if (bp->b_vp->v_type == VBLK || bp->b_vp->v_type == VCHR) - panic("msdosfs_strategy: spec"); /* * If we don't already know the filesystem relative block number * then get it using pcbmap(). If pcbmap() returns the block diff --git a/sys/fs/ntfs/ntfs_vnops.c b/sys/fs/ntfs/ntfs_vnops.c index 1ee98a5..8a8f422 100644 --- a/sys/fs/ntfs/ntfs_vnops.c +++ b/sys/fs/ntfs/ntfs_vnops.c @@ -270,14 +270,12 @@ ntfs_strategy(ap) } */ *ap; { register struct buf *bp = ap->a_bp; - register struct vnode *vp = bp->b_vp; + register struct vnode *vp = ap->a_vp; register struct fnode *fp = VTOF(vp); register struct ntnode *ip = FTONT(fp); struct ntfsmount *ntmp = ip->i_mp; int error; - KASSERT(ap->a_vp == ap->a_bp->b_vp, ("%s(%p != %p)", - __func__, ap->a_vp, ap->a_bp->b_vp)); dprintf(("ntfs_strategy: offset: %d, blkno: %d, lblkno: %d\n", (u_int32_t)bp->b_offset,(u_int32_t)bp->b_blkno, (u_int32_t)bp->b_lblkno)); diff --git a/sys/fs/udf/udf_vnops.c b/sys/fs/udf/udf_vnops.c index d26fddc..d2ca71f 100644 --- a/sys/fs/udf/udf_vnops.c +++ b/sys/fs/udf/udf_vnops.c @@ -831,7 +831,7 @@ udf_strategy(struct vop_strategy_args *a) int maxsize; bp = a->a_bp; - vp = bp->b_vp; + vp = a->a_vp; node = VTON(vp); KASSERT(a->a_vp == a->a_bp->b_vp, ("%s(%p != %p)", diff --git a/sys/fs/unionfs/union_vnops.c b/sys/fs/unionfs/union_vnops.c index 6544674..c092db9 100644 --- a/sys/fs/unionfs/union_vnops.c +++ b/sys/fs/unionfs/union_vnops.c @@ -1842,15 +1842,13 @@ union_strategy(ap) } */ *ap; { struct buf *bp = ap->a_bp; - struct vnode *othervp = OTHERVP(bp->b_vp); + struct vnode *othervp = OTHERVP(ap->a_vp); - KASSERT(ap->a_vp == ap->a_bp->b_vp, ("%s(%p != %p)", - __func__, ap->a_vp, ap->a_bp->b_vp)); #ifdef DIAGNOSTIC if (othervp == NULLVP) panic("union_strategy: nil vp"); if ((bp->b_iocmd == BIO_WRITE) && - (othervp == LOWERVP(bp->b_vp))) + (othervp == LOWERVP(ap->a_vp))) panic("union_strategy: writing to lowervp"); #endif return (VOP_STRATEGY(othervp, bp)); |