Back out proc locking to protect p_ucred for obtaining additional

references along with the actual obtaining of additional references.

1 files changed, 2 insertions, 14 deletions

diff --git a/sys/coda/coda_vfsops.c b/sys/coda/coda_vfsops.c
index 0708181..0496ea5 100644
--- a/sys/coda/coda_vfsops.c
+++ b/sys/coda/coda_vfsops.c
@@ -292,7 +292,6 @@ coda_root(vfsp, vpp)
     int error;
     struct proc *p = curproc;    /* XXX - bnoble */
     ViceFid VFid;
-    struct ucred* uc;
 
     ENTRY;
     MARK_ENTRY(CODA_ROOT_STATS);
@@ -316,12 +315,7 @@ coda_root(vfsp, vpp)
 	    }
     }
 
-    PROC_LOCK(p);
-    uc = p->p_ucred;
-    crhold(uc);
-    PROC_UNLOCK(p);
-    error = venus_root(vftomi(vfsp), uc, p, &VFid);
-    crfree(uc);
+    error = venus_root(vftomi(vfsp), p->p_ucred, p, &VFid);
 
     if (!error) {
 	/*
@@ -447,7 +441,6 @@ coda_fhtovp(vfsp, fhp, nam, vpp, exflagsp, creadanonp)
     int error;
     struct proc *p = curproc; /* XXX -mach */
     ViceFid VFid;
-    struct ucred *uc;
     int vtype;
 
     ENTRY;
@@ -461,12 +454,7 @@ coda_fhtovp(vfsp, fhp, nam, vpp, exflagsp, creadanonp)
 	return(0);
     }
     
-    PROC_LOCK(p);
-    uc = p->p_ucred;
-    crhold(uc);
-    PROC_UNLOCK(p);
-    error = venus_fhtovp(vftomi(vfsp), &cfid->cfid_fid, uc, p, &VFid, &vtype);
-    crfree(uc);
+    error = venus_fhtovp(vftomi(vfsp), &cfid->cfid_fid, p->p_ucred, p, &VFid, &vtype);
     
     if (error) {
 	CODADEBUG(CODA_VGET, myprintf(("vget error %d\n",error));)