Retire AFS_MOUNT_AFS 23/14323/4
authorAndrew Deason <adeason@sinenomine.net>
Wed, 26 Aug 2020 18:54:00 +0000 (13:54 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 5 Feb 2021 05:26:02 +0000 (00:26 -0500)
commit6b96a49eb6268adf9fc7e077fe849af7802a1575
tree5ce4ce71f7146d6ff05f1fd2f513176b08a37a89
parente0e0b3cea6305cdbccc71039a05d6121c32c51cf
Retire AFS_MOUNT_AFS

Currently, the AFS_MOUNT_AFS #define is used to mean two completely
different things:

- The string "afs", corresponding to the first argument to mount(2) on
  many platforms and some related calls inside libafs (e.g.
  getnewvnode() on FBSD).

- An integer identifying the AFS filesystem (e.g. gfsadd() on AIX).

Depending on the platform and the build context (UKERNEL vs KERNEL),
AFS_MOUNT_AFS gets defined to one of those two things. This is very
confusing, and has led to mistakes in the past, such as those fixed in
commit 446457a1 (afs: Set AFS_VFSFSID to a numerical value).

To avoid such confusion, get rid of AFS_MOUNT_AFS completely, and
replace it with two new symbols:

- AFS_MOUNT_STR, the string "afs".

- AFS_FSNO, the integer given to gfsadd() et al.

When AFS_MOUNT_AFS is split this way, AFS_MOUNT_STR then is always
defined to the same value, so remove it from the param.h files for our
platforms. Instead, define it in afs.h for libafs use, and in
afsd_kernel.c (the only place outside of src/afs that uses it).

Also remove the logic for conditionally defining MOUNT_AFS from the
param.h files, moving the logic to the same locations as
AFS_MOUNT_STR.

Note that this commit removes the numeric definition for AFS_MOUNT_AFS
in param.sgi_65.h (aka AFS_FSNO). We never actually used this value,
since AFS_FSNO is not used on IRIX; instead, we tend to use the
'afs_fstype' global instead of a constant number.

Change-Id: I6cbf051dc938cd1c456cbe236c0afe99a3c3dd87
Reviewed-on: https://gerrit.openafs.org/14323
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
80 files changed:
src/afs/AIX/osi_config.c
src/afs/AIX/osi_vnodeops.c
src/afs/IRIX/osi_vfsops.c
src/afs/NBSD/osi_vfsops.c
src/afs/afs.h
src/afsd/afsd_kernel.c
src/config/param.alpha_nbsd15.h
src/config/param.alpha_nbsd16.h
src/config/param.arm_darwin_100.h
src/config/param.generic_fbsd.h
src/config/param.hp_ux102.h
src/config/param.hp_ux110.h
src/config/param.hp_ux11i.h
src/config/param.i386_dfbsd_23.h
src/config/param.i386_nbsd15.h
src/config/param.i386_nbsd16.h
src/config/param.linux26.h
src/config/param.nbsd15.h
src/config/param.nbsd16.h
src/config/param.nbsd20.h
src/config/param.nbsd21.h
src/config/param.nbsd30.h
src/config/param.nbsd40.h
src/config/param.nbsd50.h
src/config/param.nbsd60.h
src/config/param.nbsd70.h
src/config/param.obsd31.h
src/config/param.obsd32.h
src/config/param.obsd33.h
src/config/param.obsd34.h
src/config/param.obsd35.h
src/config/param.obsd36.h
src/config/param.obsd37.h
src/config/param.obsd38.h
src/config/param.obsd39.h
src/config/param.obsd40.h
src/config/param.obsd41.h
src/config/param.obsd42.h
src/config/param.obsd43.h
src/config/param.obsd44.h
src/config/param.obsd45.h
src/config/param.obsd46.h
src/config/param.obsd47.h
src/config/param.obsd48.h
src/config/param.obsd49.h
src/config/param.obsd50.h
src/config/param.obsd51.h
src/config/param.obsd52.h
src/config/param.obsd53.h
src/config/param.obsd54.h
src/config/param.ppc_darwin_70.h
src/config/param.ppc_darwin_80.h
src/config/param.ppc_darwin_90.h
src/config/param.ppc_nbsd16.h
src/config/param.rs_aix42.h
src/config/param.rs_aix51.h
src/config/param.rs_aix52.h
src/config/param.rs_aix53.h
src/config/param.rs_aix61.h
src/config/param.sgi_65.h
src/config/param.sun4x_510.h
src/config/param.sun4x_511.h
src/config/param.sun4x_58.h
src/config/param.sun4x_59.h
src/config/param.sunx86_510.h
src/config/param.sunx86_511.h
src/config/param.sunx86_58.h
src/config/param.sunx86_59.h
src/config/param.x86_darwin_100.h
src/config/param.x86_darwin_110.h
src/config/param.x86_darwin_120.h
src/config/param.x86_darwin_130.h
src/config/param.x86_darwin_140.h
src/config/param.x86_darwin_150.h
src/config/param.x86_darwin_160.h
src/config/param.x86_darwin_170.h
src/config/param.x86_darwin_180.h
src/config/param.x86_darwin_190.h
src/config/param.x86_darwin_80.h
src/config/param.x86_darwin_90.h