afs: Fix a few ARCH/osi_vcache.c style errors 99/13699/4
authorAndrew Deason <adeason@sinenomine.net>
Mon, 15 Jul 2019 21:24:10 +0000 (16:24 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Mon, 19 Aug 2019 06:45:34 +0000 (02:45 -0400)
Most of the ARCH/osi_vcache.c implementations were defining functions
like:

void
osi_foo(args) {
    /* impl */
}

But our prevailing style is:

void
osi_foo(args)
{
    /* impl */
}

Fix them to follow our prevailing style, and fix a couple of the more
obvious errors with identation and goto label.

Change-Id: Ie752ee67aa6acfec3bf9a28d7da41151f95fbbf6
Reviewed-on: https://gerrit.openafs.org/13699
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/afs/AIX/osi_vcache.c
src/afs/DARWIN/osi_vcache.c
src/afs/FBSD/osi_vcache.c
src/afs/HPUX/osi_vcache.c
src/afs/IRIX/osi_vcache.c
src/afs/LINUX/osi_vcache.c
src/afs/NBSD/osi_vcache.c
src/afs/OBSD/osi_vcache.c
src/afs/SOLARIS/osi_vcache.c
src/afs/UKERNEL/osi_vcache.c

index b61d466..5dea565 100644 (file)
 extern struct vnodeops *afs_ops;
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
-     int code;
-     if (!VREFCOUNT_GT(avc,0)
-         && avc->opens == 0 && (avc->f.states & CUnlinkedDel) == 0) {
-        code = afs_FlushVCache(avc, slept);
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
+    int code;
+    if (!VREFCOUNT_GT(avc,0)
+       && avc->opens == 0 && (avc->f.states & CUnlinkedDel) == 0) {
+       code = afs_FlushVCache(avc, slept);
        if (code == 0)
            return 1;
-     }
-     return 0;
+    }
+    return 0;
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *tvc;
 
     tvc = afs_osi_Alloc(sizeof(struct vcache));
@@ -42,7 +44,8 @@ osi_NewVnode(void) {
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 
 #ifdef AFS_AIX32_ENV
@@ -57,10 +60,13 @@ osi_PrePopulateVCache(struct vcache *avc) {
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) { }
+osi_AttachVnode(struct vcache *avc, int seq)
+{
+}
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     avc->v.v_op = afs_ops;
 
     avc->v.v_vfsp = afs_globalVFS;
index 54c6e77..fd47d55 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"        /*AFS-based standard headers */
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *tvc;
 
     tvc = afs_osi_Alloc(sizeof(struct vcache));
@@ -27,7 +28,8 @@ osi_NewVnode(void) {
 
 #if defined(AFS_DARWIN80_ENV)
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     *slept = 0;
 
     /* we ignore defersleep, as we *always* need to sleep */
@@ -62,7 +64,8 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 }
 #else
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     if (!VREFCOUNT_GT(avc,0)
         || ((VREFCOUNT(avc) == 1) && (UBCINFOEXISTS(AFSTOV(avc))))
         && avc->opens == 0 && (avc->f.states & CUnlinkedDel) == 0)
@@ -83,12 +86,14 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 #endif /* AFS_DARWIN80_ENV */
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) {
+osi_AttachVnode(struct vcache *avc, int seq)
+{
     ReleaseWriteLock(&afs_xvcache);
     AFS_GUNLOCK();
     afs_darwin_getnewvnode(avc);  /* includes one refcount */
@@ -102,7 +107,8 @@ osi_AttachVnode(struct vcache *avc, int seq) {
 }
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
 #if !defined(AFS_DARWIN80_ENV)
    avc->v->v_mount = afs_globalVFS;
    vSetType(avc, VREG);
index 69ac7f8..f6a2c6d 100644 (file)
@@ -54,7 +54,8 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *tvc;
 
     tvc = afs_osi_Alloc(sizeof(struct vcache));
@@ -64,12 +65,14 @@ osi_NewVnode(void) {
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) {
+osi_AttachVnode(struct vcache *avc, int seq)
+{
     struct vnode *vp;
 
     ReleaseWriteLock(&afs_xvcache);
@@ -103,7 +106,8 @@ osi_AttachVnode(struct vcache *avc, int seq) {
 }
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     avc->v->v_mount = afs_globalVFS;
     vSetType(avc, VREG);
 }
index cf669a2..5a1df46 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"        /*AFS-based standard headers */
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     int code;
 
     /* we can't control whether we sleep */
@@ -28,25 +29,28 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     return afs_osi_Alloc(sizeof(struct vcache));
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 
     avc->flushDV.low = avc->flushDV.high = AFS_MAXDV;
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) {
+osi_AttachVnode(struct vcache *avc, int seq)
+{
 }
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     AFSTOV(avc)->v_op = afs_ops;
     avc->v.v_vfsp = afs_globalVFS;
     vSetType(avc, VREG);
 }
-
index e273aa5..93d6c03 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"        /*AFS-based standard headers */
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
      int code;
      /* we can't control whether we sleep */
      if (!VREFCOUNT_GT(avc,0)
@@ -29,7 +30,8 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 extern char *makesname();
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *avc;
     char name[METER_NAMSZ];
 
@@ -52,16 +54,20 @@ osi_NewVnode(void) {
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     avc->uncred = 0;
     memset(&(avc->f), 0, sizeof(struct fvcache));
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) { }
+osi_AttachVnode(struct vcache *avc, int seq)
+{
+}
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     memset(&(avc->vc_bhv_desc), 0, sizeof(avc->vc_bhv_desc));
     bhv_desc_init(&(avc->vc_bhv_desc), avc, avc, &Afs_vnodeops);
 
index 3b82cc6..d3e7ac5 100644 (file)
@@ -25,7 +25,7 @@ TryEvictDirDentries(struct inode *inode)
 
     afs_d_alias_lock(inode);
 
-restart:
+ restart:
 #if defined(D_ALIAS_IS_HLIST)
 # if defined(HLIST_ITERATOR_NO_NODE)
     hlist_for_each_entry(dentry, &inode->i_dentry, d_alias) {
@@ -88,7 +88,7 @@ restart:
     }
     afs_d_alias_unlock(inode);
 
-inuse:
+ inuse:
     return;
 }
 
@@ -232,7 +232,7 @@ osi_ResetRootVCache(afs_uint32 volid)
 
     AFS_RELE(root);
     afs_globalVp = vcp;
-out:
+ out:
     crfree(credp);
     afs_DestroyReq(treq);
 }
index 6ceefc9..faf0657 100644 (file)
@@ -73,4 +73,3 @@ osi_PostPopulateVCache(struct vcache *avc)
     AFSTOV(avc)->v_mount = afs_globalVFS;
     vSetType(avc, VREG);
 }
-
index f6e4fca..c7899f9 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"       /*AFS-based standard headers */
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     *slept = 0;
 
     if (!VREFCOUNT_GT(avc,0)
@@ -34,7 +35,8 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *tvc;
 
     tvc = afs_osi_Alloc(sizeof(struct vcache));
@@ -44,12 +46,14 @@ osi_NewVnode(void) {
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) {
+osi_AttachVnode(struct vcache *avc, int seq)
+{
     ReleaseWriteLock(&afs_xvcache);
     AFS_GUNLOCK();
     afs_obsd_getnewvnode(avc); /* includes one refcount */
@@ -59,7 +63,8 @@ osi_AttachVnode(struct vcache *avc, int seq) {
 }
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     AFSTOV(avc)->v_mount = afs_globalVFS;
     vSetType(avc, VREG);
 }
index 78a1e20..27525f7 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"        /*AFS-based standard headers */
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     int code;
 
     if (!VREFCOUNT_GT(avc,0)
@@ -27,7 +28,8 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     struct vcache *avc;
 
     avc = afs_osi_Alloc(sizeof(struct vcache));
@@ -36,7 +38,8 @@ osi_NewVnode(void) {
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 
     QInit(&avc->multiPage);
@@ -78,7 +81,8 @@ osi_AttachVnode(struct vcache *avc, int seq)
 }
 
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     AFSTOV(avc)->v_op = afs_ops;
     AFSTOV(avc)->v_vfsp = afs_globalVFS;
     vSetType(avc, VREG);
index dce051e..0a2b9e3 100644 (file)
@@ -14,7 +14,8 @@
 #include "afsincludes.h"        /*AFS-based standard headers */
 
 int
-osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
+osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep)
+{
     int code;
 
     if (!VREFCOUNT_GT(avc,0)
@@ -27,21 +28,26 @@ osi_TryEvictVCache(struct vcache *avc, int *slept, int defersleep) {
 }
 
 struct vcache *
-osi_NewVnode(void) {
+osi_NewVnode(void)
+{
     return afs_osi_Alloc(sizeof(struct vcache));
 }
 
 void
-osi_PrePopulateVCache(struct vcache *avc) {
+osi_PrePopulateVCache(struct vcache *avc)
+{
     memset(avc, 0, sizeof(struct vcache));
 }
 
 void
-osi_AttachVnode(struct vcache *avc, int seq) { }
+osi_AttachVnode(struct vcache *avc, int seq)
+{
+}
 
 extern struct vnodeops *afs_ops;
 void
-osi_PostPopulateVCache(struct vcache *avc) {
+osi_PostPopulateVCache(struct vcache *avc)
+{
     AFSTOV(avc)->v_vfsp = afs_globalVFS;
     AFSTOV(avc)->v_op = afs_ops;
     vSetType(avc, VREG);