diff options
author | rwatson <rwatson@FreeBSD.org> | 2008-02-14 00:55:03 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2008-02-14 00:55:03 +0000 |
commit | 021108eeb6017989f537321dcd7b581714149497 (patch) | |
tree | 15d683ccbf11d22ee1e7ed2f17d09a053b8d0471 /sys/fs | |
parent | 0cf10cfc004db23371ad6b56068c2063b664e5d0 (diff) | |
download | FreeBSD-src-021108eeb6017989f537321dcd7b581714149497.zip FreeBSD-src-021108eeb6017989f537321dcd7b581714149497.tar.gz |
Remove debugging code under OLD_DIAGNOSTIC; this is all >10 years old and
hasn't been used in that time.
MFC after: 1 month
Diffstat (limited to 'sys/fs')
-rw-r--r-- | sys/fs/coda/coda_psdev.c | 4 | ||||
-rw-r--r-- | sys/fs/coda/coda_vnops.c | 31 |
2 files changed, 3 insertions, 32 deletions
diff --git a/sys/fs/coda/coda_psdev.c b/sys/fs/coda/coda_psdev.c index 27ef739..a133860 100644 --- a/sys/fs/coda/coda_psdev.c +++ b/sys/fs/coda/coda_psdev.c @@ -239,10 +239,6 @@ vc_read(struct cdev *dev, struct uio *uiop, int flag) myprintf(("vcread: error (%d) on uiomove\n", error)); error = EINVAL; } -#ifdef OLD_DIAGNOSTIC - if (vmp->vm_chain.forw == 0 || vmp->vm_chain.back == 0) - panic("vc_read: bad chain"); -#endif REMQUE(vmp->vm_chain); /* diff --git a/sys/fs/coda/coda_vnops.c b/sys/fs/coda/coda_vnops.c index cc8f812..d6e3429 100644 --- a/sys/fs/coda/coda_vnops.c +++ b/sys/fs/coda/coda_vnops.c @@ -841,15 +841,8 @@ coda_inactive(struct vop_inactive_args *ap) printf("coda_inactive: cp->ovp != NULL use %d: vp " "%p, cp %p\n", vrefcnt(vp), vp, cp); #endif - } else { -#ifdef OLD_DIAGNOSTIC - if (vrefcnt(CTOV(cp))) - panic("coda_inactive: nonzero reference count"); - if (cp->c_ovp != NULL) - panic("coda_inactive: cp->ovp != NULL"); -#endif + } else vgone(vp); - } MARK_INT_SAT(CODA_INACTIVE_STATS); return (0); } @@ -1080,10 +1073,6 @@ coda_create(struct vop_create_args *ap) cache_enter(dvp, *vpp, cnp); if (cnp->cn_flags & LOCKLEAF) vn_lock(*ap->a_vpp, LK_EXCLUSIVE | LK_RETRY); -#ifdef OLD_DIAGNOSTIC - else - printf("coda_create: LOCKLEAF not set!\n"); -#endif } else if (error == ENOENT) { /* * XXXRW: We only enter a negative entry if ENOENT is @@ -1214,16 +1203,6 @@ coda_rename(struct vop_rename_args *ap) MARK_ENTRY(CODA_RENAME_STATS); /* - * Hmmm. The vnodes are already looked up. Perhaps they are locked? - * This could be Bad. XXX - */ -#ifdef OLD_DIAGNOSTIC - if ((fcnp->cn_cred != tcnp->cn_cred) || - (fcnp->cn_thread != tcnp->cn_thread)) - panic("coda_rename: component names don't agree"); -#endif - - /* * Check for rename involving control object. */ if (IS_CTL_NAME(odvp, fnm, flen) || IS_CTL_NAME(ndvp, tnm, tlen)) { @@ -1572,12 +1551,8 @@ coda_reclaim(struct vop_reclaim_args *ap) } #endif } else { -#ifdef OLD_DIAGNOSTIC - if (vrefcnt(vp) != 0) - print("coda_reclaim: pushing active %p\n", vp); - if (VTOC(vp)->c_ovp) - panic("coda_reclaim: c_ovp not void"); -#endif + if (prtactive && vp->v_usecount != 0) + vprint("coda_reclaim: pushing active", vp); } cache_purge(vp); coda_free(VTOC(vp)); |