client-dir-package-interface-update-20041106
[openafs.git] / src / afs / VNOPS / afs_vnop_rename.c
index dd332a5..394f114 100644 (file)
 #include <afsconfig.h>
 #include "afs/param.h"
 
-RCSID("$Header$");
+RCSID
+    ("$Header$");
 
 #include "afs/sysincludes.h"   /* Standard vendor system headers */
 #include "afsincludes.h"       /* Afs-based standard headers */
-#include "afs/afs_stats.h" /* statistics */
+#include "afs/afs_stats.h"     /* statistics */
 #include "afs/afs_cbqueue.h"
 #include "afs/nfsclient.h"
 #include "afs/afs_osidnlc.h"
@@ -31,8 +32,9 @@ extern afs_rwlock_t afs_xcbhash;
 /* Note that we don't set CDirty here, this is OK because the rename
  * RPC is called synchronously. */
 
-int afsrename(struct vcache *aodp, char *aname1, struct vcache *andp, 
-       char *aname2, struct AFS_UCRED *acred, struct vrequest *areq)
+int
+afsrename(struct vcache *aodp, char *aname1, struct vcache *andp,
+         char *aname2, struct AFS_UCRED *acred, struct vrequest *areq)
 {
     register struct conn *tc;
     register afs_int32 code;
@@ -44,10 +46,9 @@ int afsrename(struct vcache *aodp, char *aname1, struct vcache *andp,
     struct dcache *tdc1, *tdc2;
     struct AFSFetchStatus OutOldDirStatus, OutNewDirStatus;
     struct AFSVolSync tsync;
-    XSTATS_DECLS
-
+    XSTATS_DECLS;
     AFS_STATCNT(afs_rename);
-    afs_Trace4(afs_iclSetp, CM_TRACE_RENAME, ICL_TYPE_POINTER, aodp, 
+    afs_Trace4(afs_iclSetp, CM_TRACE_RENAME, ICL_TYPE_POINTER, aodp,
               ICL_TYPE_STRING, aname1, ICL_TYPE_POINTER, andp,
               ICL_TYPE_STRING, aname2);
 
@@ -57,14 +58,17 @@ int afsrename(struct vcache *aodp, char *aname1, struct vcache *andp,
     }
 
     /* verify the latest versions of the stat cache entries */
-tagain:
+  tagain:
     code = afs_VerifyVCache(aodp, areq);
-    if (code) goto done;
+    if (code)
+       goto done;
     code = afs_VerifyVCache(andp, areq);
-    if (code) goto done;
-    
+    if (code)
+       goto done;
+
     /* lock in appropriate order, after some checks */
-    if (aodp->fid.Cell != andp->fid.Cell || aodp->fid.Fid.Volume != andp->fid.Fid.Volume) {
+    if (aodp->fid.Cell != andp->fid.Cell
+       || aodp->fid.Fid.Volume != andp->fid.Fid.Volume) {
        code = EXDEV;
        goto done;
     }
@@ -78,7 +82,7 @@ tagain:
            code = 0;
            goto done;
        }
-       ObtainWriteLock(&andp->lock,147);
+       ObtainWriteLock(&andp->lock, 147);
        tdc1 = afs_GetDCache(aodp, (afs_size_t) 0, areq, &offset, &len, 0);
        if (!tdc1) {
            code = ENOENT;
@@ -86,38 +90,37 @@ tagain:
            ObtainWriteLock(&tdc1->lock, 643);
        }
        tdc2 = tdc1;
-       oneDir = 1;         /* only one dude locked */
-    }
-    else if ((andp->states & CRO) || (aodp->states & CRO)) {
-       code = EROFS;
-       goto done;
-     }
-    else if (andp->fid.Fid.Vnode < aodp->fid.Fid.Vnode) {
-       ObtainWriteLock(&andp->lock,148);       /* lock smaller one first */
-       ObtainWriteLock(&aodp->lock,149);
-       tdc2 = afs_FindDCache(andp, (afs_size_t)0);
-       if (tdc2) ObtainWriteLock(&tdc2->lock, 644);
+       oneDir = 1;             /* only one dude locked */
+    } else if ((andp->states & CRO) || (aodp->states & CRO)) {
+       code = EROFS;
+       goto done;
+    } else if (andp->fid.Fid.Vnode < aodp->fid.Fid.Vnode) {
+       ObtainWriteLock(&andp->lock, 148);      /* lock smaller one first */
+       ObtainWriteLock(&aodp->lock, 149);
+       tdc2 = afs_FindDCache(andp, (afs_size_t) 0);
+       if (tdc2)
+           ObtainWriteLock(&tdc2->lock, 644);
        tdc1 = afs_GetDCache(aodp, (afs_size_t) 0, areq, &offset, &len, 0);
        if (tdc1)
            ObtainWriteLock(&tdc1->lock, 645);
        else
            code = ENOENT;
-    }
-    else {
-       ObtainWriteLock(&aodp->lock,150);       /* lock smaller one first */
-       ObtainWriteLock(&andp->lock,557);
+    } else {
+       ObtainWriteLock(&aodp->lock, 150);      /* lock smaller one first */
+       ObtainWriteLock(&andp->lock, 557);
        tdc1 = afs_GetDCache(aodp, (afs_size_t) 0, areq, &offset, &len, 0);
        if (tdc1)
            ObtainWriteLock(&tdc1->lock, 646);
        else
            code = ENOENT;
-       tdc2 = afs_FindDCache(andp, (afs_size_t)0);
-       if (tdc2) ObtainWriteLock(&tdc2->lock, 647);
+       tdc2 = afs_FindDCache(andp, (afs_size_t) 0);
+       if (tdc2)
+           ObtainWriteLock(&tdc2->lock, 647);
     }
 
-    osi_dnlc_remove (aodp, aname1, 0);
-    osi_dnlc_remove (andp, aname2, 0);
-    afs_symhint_inval(aodp); 
+    osi_dnlc_remove(aodp, aname1, 0);
+    osi_dnlc_remove(andp, aname2, 0);
+    afs_symhint_inval(aodp);
     afs_symhint_inval(andp);
 
     /*
@@ -143,7 +146,7 @@ tagain:
     }
 
     if (code == 0)
-       code = afs_dir_Lookup(&tdc1->f.inode, aname1, &fileFid.Fid);
+       code = afs_dir_Lookup(tdc1, aname1, &fileFid.Fid);
     if (code) {
        if (tdc1) {
            ReleaseWriteLock(&tdc1->lock);
@@ -164,26 +167,28 @@ tagain:
     do {
        tc = afs_Conn(&aodp->fid, areq, SHARED_LOCK);
        if (tc) {
-          XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_RENAME);
-         RX_AFS_GUNLOCK();
-         code = RXAFS_Rename(tc->id, (struct AFSFid *) &aodp->fid.Fid, aname1,
-                               (struct AFSFid *) &andp->fid.Fid, aname2,
-                               &OutOldDirStatus, &OutNewDirStatus, &tsync);
-         RX_AFS_GLOCK();
-          XSTATS_END_TIME;
-       } else code = -1;
+           XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_RENAME);
+           RX_AFS_GUNLOCK();
+           code =
+               RXAFS_Rename(tc->id, (struct AFSFid *)&aodp->fid.Fid, aname1,
+                            (struct AFSFid *)&andp->fid.Fid, aname2,
+                            &OutOldDirStatus, &OutNewDirStatus, &tsync);
+           RX_AFS_GLOCK();
+           XSTATS_END_TIME;
+       } else
+           code = -1;
+
+    } while (afs_Analyze
+            (tc, code, &andp->fid, areq, AFS_STATS_FS_RPCIDX_RENAME,
+             SHARED_LOCK, NULL));
 
-    } while
-       (afs_Analyze(tc, code, &andp->fid, areq,
-                    AFS_STATS_FS_RPCIDX_RENAME, SHARED_LOCK, NULL));
+    returnCode = code;         /* remember for later */
 
-    returnCode = code;     /* remember for later */
-    
     /* Now we try to do things locally.  This is really loathsome code. */
     unlinkFid.Fid.Vnode = 0;
     if (code == 0) {
        /*  In any event, we don't really care if the data (tdc2) is not
-           in the cache; if it isn't, we won't do the update locally.  */
+        * in the cache; if it isn't, we won't do the update locally.  */
        /* see if version numbers increased properly */
        doLocally = 1;
        if (oneDir) {
@@ -191,8 +196,7 @@ tagain:
            if (!afs_LocalHero(aodp, tdc1, &OutOldDirStatus, 1)) {
                doLocally = 0;
            }
-       }
-       else {
+       } else {
            /* two separate dirs, each increasing by 1 */
            if (!afs_LocalHero(aodp, tdc1, &OutOldDirStatus, 1))
                doLocally = 0;
@@ -201,37 +205,38 @@ tagain:
            if (!doLocally) {
                if (tdc1) {
                    ZapDCE(tdc1);
-                   DZap(&tdc1->f.inode);
+                   DZap(tdc1);
                }
                if (tdc2) {
                    ZapDCE(tdc2);
-                   DZap(&tdc2->f.inode);
+                   DZap(tdc2);
                }
            }
        }
        /* now really do the work */
        if (doLocally) {
            /* first lookup the fid of the dude we're moving */
-           code = afs_dir_Lookup(&tdc1->f.inode, aname1, &fileFid.Fid);
+           code = afs_dir_Lookup(tdc1, aname1, &fileFid.Fid);
            if (code == 0) {
                /* delete the source */
-               code = afs_dir_Delete(&tdc1->f.inode, aname1);
+               code = afs_dir_Delete(tdc1, aname1);
            }
            /* first see if target is there */
-           if (code == 0 &&
-               afs_dir_Lookup(&tdc2->f.inode, aname2, &unlinkFid.Fid) == 0) {
+           if (code == 0
+               && afs_dir_Lookup(tdc2, aname2,
+                                 &unlinkFid.Fid) == 0) {
                /* target already exists, and will be unlinked by server */
-               code = afs_dir_Delete(&tdc2->f.inode, aname2);
+               code = afs_dir_Delete(tdc2, aname2);
            }
            if (code == 0) {
-               code = afs_dir_Create(&tdc2->f.inode, aname2, &fileFid.Fid);
+               code = afs_dir_Create(tdc2, aname2, &fileFid.Fid);
            }
            if (code != 0) {
                ZapDCE(tdc1);
-               DZap(&tdc1->f.inode);
+               DZap(tdc1);
                if (!oneDir) {
                    ZapDCE(tdc2);
-                   DZap(&tdc2->f.inode);
+                   DZap(tdc2);
                }
            }
        }
@@ -241,19 +246,18 @@ tagain:
        if (!oneDir)
            andp->m.LinkCount = OutNewDirStatus.LinkCount;
 
-    }
-    else {     /* operation failed (code != 0) */
-       if (code < 0) { 
+    } else {                   /* operation failed (code != 0) */
+       if (code < 0) {
            /* if failed, server might have done something anyway, and 
             * assume that we know about it */
-         ObtainWriteLock(&afs_xcbhash, 498);
-         afs_DequeueCallback(aodp);
-         afs_DequeueCallback(andp);
-         andp->states &= ~CStatd;
-         aodp->states &= ~CStatd;
-         ReleaseWriteLock(&afs_xcbhash);
-         osi_dnlc_purgedp(andp);
-         osi_dnlc_purgedp(aodp);
+           ObtainWriteLock(&afs_xcbhash, 498);
+           afs_DequeueCallback(aodp);
+           afs_DequeueCallback(andp);
+           andp->states &= ~CStatd;
+           aodp->states &= ~CStatd;
+           ReleaseWriteLock(&afs_xcbhash);
+           osi_dnlc_purgedp(andp);
+           osi_dnlc_purgedp(aodp);
        }
     }
 
@@ -269,20 +273,21 @@ tagain:
     }
 
     ReleaseWriteLock(&aodp->lock);
-    if (!oneDir) ReleaseWriteLock(&andp->lock);
-    
+    if (!oneDir)
+       ReleaseWriteLock(&andp->lock);
+
     if (returnCode) {
        code = returnCode;
        goto done;
     }
 
     /* now, some more details.  if unlinkFid.Fid.Vnode then we should decrement
-       the link count on this file.  Note that if fileFid is a dir, then we don't
-       have to invalidate its ".." entry, since its DataVersion # should have
-       changed. However, interface is not good enough to tell us the
-       *file*'s new DataVersion, so we're stuck.  Our hack: delete mark
-       the data as having an "unknown" version (effectively discarding the ".."
-       entry */
+     * the link count on this file.  Note that if fileFid is a dir, then we don't
+     * have to invalidate its ".." entry, since its DataVersion # should have
+     * changed. However, interface is not good enough to tell us the
+     * *file*'s new DataVersion, so we're stuck.  Our hack: delete mark
+     * the data as having an "unknown" version (effectively discarding the ".."
+     * entry */
     if (unlinkFid.Fid.Vnode) {
        unlinkFid.Fid.Volume = aodp->fid.Fid.Volume;
        unlinkFid.Cell = aodp->fid.Cell;
@@ -290,26 +295,27 @@ tagain:
        if (!unlinkFid.Fid.Unique) {
            tvc = afs_LookupVCache(&unlinkFid, areq, NULL, aodp, aname1);
        }
-       if (!tvc) /* lookup failed or wasn't called */
-          tvc = afs_GetVCache(&unlinkFid, areq, NULL, NULL);
+       if (!tvc)               /* lookup failed or wasn't called */
+           tvc = afs_GetVCache(&unlinkFid, areq, NULL, NULL);
 
        if (tvc) {
 #if    defined(AFS_SUN_ENV) || defined(AFS_ALPHA_ENV) || defined(AFS_SUN5_ENV)
            afs_BozonLock(&tvc->pvnLock, tvc);  /* Since afs_TryToSmush will do a pvn_vptrunc */
 #endif
-           ObtainWriteLock(&tvc->lock,151);
+           ObtainWriteLock(&tvc->lock, 151);
            tvc->m.LinkCount--;
-           tvc->states &= ~CUnique;            /* For the dfs xlator */
+           tvc->states &= ~CUnique;    /* For the dfs xlator */
            if (tvc->m.LinkCount == 0 && !osi_Active(tvc)) {
                /* if this was last guy (probably) discard from cache.
-                 * We have to be careful to not get rid of the stat
-                 * information, since otherwise operations will start
-                 * failing even if the file was still open (or
-                 * otherwise active), and the server no longer has the
-                 * info.  If the file still has valid links, we'll get
-                 * a break-callback msg from the server, so it doesn't
-                 * matter that we don't discard the status info */
-               if (!AFS_NFSXLATORREQ(acred)) afs_TryToSmush(tvc, acred, 0);
+                * We have to be careful to not get rid of the stat
+                * information, since otherwise operations will start
+                * failing even if the file was still open (or
+                * otherwise active), and the server no longer has the
+                * info.  If the file still has valid links, we'll get
+                * a break-callback msg from the server, so it doesn't
+                * matter that we don't discard the status info */
+               if (!AFS_NFSXLATORREQ(acred))
+                   afs_TryToSmush(tvc, acred, 0);
            }
            ReleaseWriteLock(&tvc->lock);
 #if    defined(AFS_SUN_ENV) || defined(AFS_ALPHA_ENV)  || defined(AFS_SUN5_ENV)
@@ -326,14 +332,14 @@ tagain:
        if (!fileFid.Fid.Unique)
            tvc = afs_LookupVCache(&fileFid, areq, NULL, andp, aname2);
        else
-           tvc = afs_GetVCache(&fileFid, areq, NULL, (struct vcache*)0);
+           tvc = afs_GetVCache(&fileFid, areq, NULL, (struct vcache *)0);
        if (tvc && (vType(tvc) == VDIR)) {
-           ObtainWriteLock(&tvc->lock,152);
-           tdc1 = afs_FindDCache(tvc, (afs_size_t)0);
+           ObtainWriteLock(&tvc->lock, 152);
+           tdc1 = afs_FindDCache(tvc, (afs_size_t) 0);
            if (tdc1) {
                ObtainWriteLock(&tdc1->lock, 648);
                ZapDCE(tdc1);   /* mark as unknown */
-               DZap(&tdc1->f.inode);
+               DZap(tdc1);
                ReleaseWriteLock(&tdc1->lock);
                afs_PutDCache(tdc1);    /* put it back */
            }
@@ -348,46 +354,52 @@ tagain:
        }
     }
     code = returnCode;
-done:
+  done:
     return code;
 }
 
+int
 #ifdef AFS_OSF_ENV
 afs_rename(fndp, tndp)
-    struct nameidata *fndp, *tndp; {
+     struct nameidata *fndp, *tndp;
+{
     struct vcache *aodp = VTOAFS(fndp->ni_dvp);
     char *aname1 = fndp->ni_dent.d_name;
     struct vcache *andp = VTOAFS(tndp->ni_dvp);
     char *aname2 = tndp->ni_dent.d_name;
     struct ucred *acred = tndp->ni_cred;
-#else  /* AFS_OSF_ENV */
+#else /* AFS_OSF_ENV */
 #if defined(AFS_SGI_ENV)
 afs_rename(OSI_VC_ARG(aodp), aname1, andp, aname2, npnp, acred)
     struct pathname *npnp;
 #else
 afs_rename(OSI_VC_ARG(aodp), aname1, andp, aname2, acred)
 #endif
-    OSI_VC_DECL(aodp);
-    struct vcache *andp;
-    char *aname1, *aname2;
-    struct AFS_UCRED *acred; {
+     OSI_VC_DECL(aodp);
+     struct vcache *andp;
+     char *aname1, *aname2;
+     struct AFS_UCRED *acred;
+{
 #endif
     register afs_int32 code;
     struct afs_fakestat_state ofakestate;
     struct afs_fakestat_state nfakestate;
     struct vrequest treq;
-    OSI_VC_CONVERT(aodp)
+    OSI_VC_CONVERT(aodp);
 
     code = afs_InitReq(&treq, acred);
-    if (code) return code;
+    if (code)
+       return code;
     afs_InitFakeStat(&ofakestate);
     afs_InitFakeStat(&nfakestate);
     code = afs_EvalFakeStat(&aodp, &ofakestate, &treq);
-    if (code) goto done;
+    if (code)
+       goto done;
     code = afs_EvalFakeStat(&andp, &nfakestate, &treq);
-    if (code) goto done;
+    if (code)
+       goto done;
     code = afsrename(aodp, aname1, andp, aname2, acred, &treq);
-done:
+  done:
     afs_PutFakeStat(&ofakestate);
     afs_PutFakeStat(&nfakestate);
 #ifdef AFS_OSF_ENV
@@ -397,7 +409,7 @@ done:
     }
     AFS_RELE(fndp->ni_dvp);
     AFS_RELE(fndp->ni_vp);
-#endif /* AFS_OSF_ENV */
+#endif /* AFS_OSF_ENV */
     code = afs_CheckCode(code, &treq, 25);
     return code;
 }