From: Ben Kaduk <kaduk@mit.edu>
Date: Wed, 30 Jun 2010 04:29:41 +0000 (-0400)
Subject: FBSD: in reclaim, print the failed vnode
X-Git-Tag: openafs-devel-1_5_75~41
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=b1566e42a69935467a5869391ed20860d416fcdd;p=packages%2Fo%2Fopenafs.git

FBSD: in reclaim, print the failed vnode

If we fail to flush a vnode, in addition to printing a diagnostic
message, also print information about the vnode; fs getfid can
frequently confirm which vnode(s) are problematic, which may be
useful in debugging the failure.

Change-Id: Ia6e93f436f9dcd2d634c46856ede5effbf79e9cb
Reviewed-on: http://gerrit.openafs.org/2296
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/FBSD/osi_vnodeops.c b/src/afs/FBSD/osi_vnodeops.c
index cfa59ff25..8d88aef29 100644
--- a/src/afs/FBSD/osi_vnodeops.c
+++ b/src/afs/FBSD/osi_vnodeops.c
@@ -1438,8 +1438,10 @@ afs_vop_reclaim(struct vop_reclaim_args *ap)
      * XXX Pretend it worked, to prevent panic on shutdown
      * Garrett, please fix - Jim Rees
      */
-    if (code)
-	printf("afs_vop_reclaim: afs_FlushVCache failed code %d\n", code);
+    if (code) {
+	printf("afs_vop_reclaim: afs_FlushVCache failed code %d vnode\n", code);
+	VOP_PRINT(vp);
+    }
 
     /* basically, it must not fail */
     vnode_destroy_vobject(vp);