Do not define AFS_SYSCALL for ppc64le_linux26 92/13592/2
authorAndrew Deason <adeason@sinenomine.net>
Thu, 16 May 2019 21:12:47 +0000 (16:12 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 7 Jun 2019 06:44:30 +0000 (02:44 -0400)
AFS_SYSCALL is defined to the syscall number we can use for a certain
platform (for pioctls and other AFS-specific kernel calls). On many
modern platforms, such as Linux, we don't use direct syscalls anymore,
instead routing our AFS-specific syscalls through an ioctl, and
AFS_SYSCALL is just used as a fallback for compatibility for older
OpenAFS releases that might still be using the syscall.

For new platforms, we have no need for this compatibility code path,
since there is no existing code we might need to be compatible with.
We should avoid defining AFS_SYSCALL for those, so we can avoid
manually-issuing syscalls in more cases. The ppc64le_linux26 platform
is a very new platform (introduced in 191e18eb "Open ppc64le_linux
sysname space"), and so should not have AFS_SYSCALL defined.

So, remove AFS_SYSCALL from ppc64le_linux26's param.h.

Change-Id: I7811831b05a17c9428556aca49681cd544da4ff1
Reviewed-on: https://gerrit.openafs.org/13592
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Tested-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/config/param.ppc64le_linux26.h

index d877bb1..9c8b822 100644 (file)
@@ -30,6 +30,5 @@
 /* Machine / Operating system information */
 #define SYS_NAME       "ppc64le_linux26"
 #define SYS_NAME_ID    SYS_NAME_ID_ppc64le_linux26
-#define AFS_SYSCALL    137
 
 #endif /* AFS_PARAM_H */