macos: don't attempt finalize fixup on root vnode

because of how the root fid is created we can end up being dumb.
turns out we never want to bypass doing the full pass for root anyway
so just force fixup to not happen.

Reviewed-on: http://gerrit.openafs.org/5095
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit f8ff2591f73401e6d9b6def7630f428569e2f3a6)

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

diff --git a/src/afs/DARWIN/osi_vnodeops.c b/src/afs/DARWIN/osi_vnodeops.c
index 8dc8b85ef910d1c7f265363ad35990af1e5cc68d..e16505a0bf09d78eec94188891372835d55df53d 100644 (file)
--- a/src/afs/DARWIN/osi_vnodeops.c
+++ b/src/afs/DARWIN/osi_vnodeops.c
@@ -2164,8 +2164,8 @@ afs_darwin_finalizevnode(struct vcache *avc, struct vnode *dvp,
     ovp = AFSTOV(avc);
 
     /* if the type changed, we still need to do a fixup, for bulkstat */
-    if (vnode_vtype(ovp) == avc->f.m.Type && !(avc->f.states & CDeadVnode)
-       && vnode_vtype(ovp) != VNON)
+    if (!isroot && (vnode_vtype(ovp) == avc->f.m.Type &&
+                   !(avc->f.states & CDeadVnode) && vnode_vtype(ovp) != VNON)
     {
        AFS_GUNLOCK();
 #if 0 /* unsupported */