Reconciliation of src/{afs,vol}/voldefs.h 05/5405/8
authorSteve Simmons <scs@umich.edu>
Tue, 13 Sep 2011 17:41:19 +0000 (13:41 -0400)
committerBenjamin Kaduk <kaduk@mit.edu>
Wed, 4 May 2016 05:40:17 +0000 (01:40 -0400)
Bring these two files back into synchronization. Fix
possible bug on very old SysV hosts where volume
header file extension could be handled inconsistently.
Overall differences reduced by about 50%. HPUX/AIX
differences now correctly managed in both versions.
Comment formats and whitespace in both modified to
remove differences and follow openafs standards.

Change-Id: I8fdf9941a0ee6ad7a091be38740bc2796f2b1d18
Reviewed-on: https://gerrit.openafs.org/5405
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Chas Williams <3chas3@gmail.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/afs/voldefs.h
src/vol/voldefs.h

index df6c7a2..5ce3934 100644 (file)
@@ -8,14 +8,14 @@
  */
 
 /*
-       System:         VICE-TWO
-       Module:         voldefs.h
-       Institution:    The Information Technology Center, Carnegie-Mellon University
-
+ * System:             VICE-TWO
+ * Module:             voldefs.h
+ * Institution:        The Information Technology Center, Carnegie-Mellon University
  */
 
 /* If you add volume types here, be sure to check the definition of
-   volumeWriteable in volume.h */
+ * volumeWriteable in volume.h
+ */
 
 #define readwriteVolume                RWVOL
 #define readonlyVolume         ROVOL
 #define ROVOL                  1
 #define BACKVOL                        2
 
-/* All volumes will have a volume header name in this format */
-#define VFORMAT "V%010lu.vol"
+/*
+ * All volumes will have a volume header file with the filename
+ * in one of these two formats. The difference is largely historical.
+ * Early versions of System V Unix had severe limitations on maximum
+ * filename length, and the '.vol' extension had to be shortened to
+ * allow for the maximum volume name to fit. Those restrictions no
+ * longer apply, but the descendants of those legacy AIX and HPUX
+ * servers still use the shorter name.
+ *
+ * If you change VHDREXT, you must change VFORMAT as well. Don't try
+ * this with token pasting in cpp; it's just too inconsistent in
+ * across C compilers.
+ *
+ * Note that <afs/param.h> must have been included before we get here.
+ */
+
+#if defined(AFS_AIX_ENV) || defined(AFS_HPUX_ENV)
+#define VHDREXT        ".vl"
+#define VFORMAT        "V%010" AFS_VOLID_FMT ".vl"
+#else
+#define VHDREXT        ".vol"
+#define VFORMAT        "V%010" AFS_VOLID_FMT ".vol"
+#endif
+
 #define VMAXPATHLEN 64         /* Maximum length (including null) of a volume
-                                * external path name */
+                                * external path name
+                                */
index aa9d25d..57bebe6 100644 (file)
@@ -8,14 +8,14 @@
  */
 
 /*
-       System:         VICE-TWO
-       Module:         voldefs.h
-       Institution:    The Information Technology Center, Carnegie-Mellon University
-
+ * System:             VICE-TWO
+ * Module:             voldefs.h
+ * Institution:        The Information Technology Center, Carnegie-Mellon University
  */
 
 /* If you add volume types here, be sure to check the definition of
-   volumeWriteable in volume.h */
+ * volumeWriteable in volume.h
+ */
 
 #define readwriteVolume                RWVOL
 #define readonlyVolume         ROVOL
 #define BACKVOL                        2
 #define RWREPL                 3
 
-#define VOLMAXTYPES             4   /* _current_ max number of types */
+#define VOLMAXTYPES             4   /* current max number of types */
 
 /* the maximum number of volumes in a volume group that we can handle */
+
 #define VOL_VG_MAX_VOLS 20
 
-/* how many times to retry if we raced the fileserver restarting, when trying
- * to checkout/lock a volume */
+/* How many times to retry if we raced the fileserver restarting, when trying
+ * to check out or lock a volume. */
+
 #define VOL_MAX_CHECKOUT_RETRIES 10
 
-/* maximum numbe of Vice partitions */
-#define        VOLMAXPARTS     255
+/* maximum number of vice partitions */
+#define VOLMAXPARTS    255
 
 #define VFORMATDIGITS 10
 
-/* All volumes will have a volume header name in this format */
-#if    defined(AFS_AIX_ENV) || defined(AFS_HPUX_ENV)
-/* Note that <afs/param.h> must have been included before we get here... */
-#define        VFORMAT "V%010" AFS_VOLID_FMT ".vl"     /* Sys5's filename length limitation hits us again */
-#define        VHDREXT ".vl"
+/*
+ * All volumes will have a volume header file with the filename
+ * in one of these two formats. The difference is largely historical.
+ * Early versions of System V Unix had severe limitations on maximum
+ * filename length, and the '.vol' extension had to be shortened to
+ * allow for the maximum volume name to fit. Those restrictions no
+ * longer apply, but the descendants of those legacy AIX and HPUX
+ * servers still use the shorter name.
+ *
+ * If you change VHDREXT, you must change VFORMAT as well. Don't try
+ * this with token pasting in cpp; it's just too inconsistent in
+ * across C compilers.
+ *
+ * Note that <afs/param.h> must have been included before we get here.
+ */
+
+#if defined(AFS_AIX_ENV) || defined(AFS_HPUX_ENV)
+#define VHDREXT        ".vl"
+#define        VFORMAT "V%010" AFS_VOLID_FMT ".vl"
 #else
-#define VFORMAT "V%010" AFS_VOLID_FMT ".vol"
 #define        VHDREXT ".vol"
+#define VFORMAT "V%010" AFS_VOLID_FMT ".vol"
 #endif
 #define        VHDRNAMELEN (VFORMATDIGITS + 1 + sizeof(VHDREXT) - 1) /* must match VFORMAT */
-#define VMAXPATHLEN 64         /* Maximum length (including null) of a volume
-                                * external path name */
+
+/* Maximum length (including trailing NUL) of a volume external path name. */
+#define VMAXPATHLEN 64
 
 #if defined(AFS_NAMEI_ENV) && !defined(AFS_NT40_ENV)
+
 /* INODEDIR holds all the inodes. Since it's name does not begin with "V"
  * and it's created when the first volume is created, linear directory
  * searches will find the directory early. If only I had needed this before
  * the NT server went beta, it could be used there as well.
  */
+
 #define INODEDIR "AFSIDat"
 #define INODEDIRLEN (sizeof(INODEDIR)-1)
 #endif
 
 /* Pathname for the maximum volume id ever created by this server */
+
 #define MAXVOLIDPATH   "/vice/vol/maxvolid"
 
-/* Pathname for server id definitions--the server id is used to allocate volume numbers */
+/* Pathname for server id definitions. The server id is used to
+ * allocate volume numbers. */
+
 #define SERVERLISTPATH "/vice/db/servers"