OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-200-gd9232fe

Gerrit Code Review gerrit@openafs.org
Thu, 18 Aug 2022 11:32:52 -0400


The following commit has been merged in the master branch:
commit d9232fedec06a6fcd953cceb73c54e48a638575a
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Fri Jun 3 13:22:19 2022 -0600

    afs: Cleanup AFS_S390X_ENV statement
    
    The commit 'Change AFS*_LINUXnn_ENV to AFS*_LINUX_ENV' (6329a523f6)
    mechanically replaced the multiple versions of the AFS*_LINUXnn_ENV
    definitions with non-versioned definitions.  A follow-on commit 'Cleanup
    AFS_*LINUX_ENV usage' (cbc18e4b3) refactored code to handle cleaning up
    redundant statements or removing dead code.
    
    The clean up in cbc18e4b3 missed a block of preprocessor statements in
    osi_probe.c which defined the macros _SS(x) and _SX(x).
    
    Prior to the commit 6329a523f6, these macros performed bit shifting or
    masking for the S390X LINUX20 environment.  For S390X LINUX24 or other
    platforms the macros just emitted their parameter.
    
    With commit 6329a523f6 the statement:
       #if defined(AFS_S390X_LINUX20_ENV) && !defined(AFS_S390X_LINUX24_ENV)
    was replaced with (which dropped support for S390X LINUX20):
       #if defined(AFS_S390X_LINUX_ENV) && !defined(AFS_S390X_LINUX_ENV)
    
    After cleaning up the conditional, the result leaves the definitions for
    _SS and _SX as:
       #define _SS(x) (x)
       #define _SX(x) (x)
    
    Which allows further cleanup by removing these macros and just using
    their arguments directly.
    
    Change-Id: I7b56c01ffd59e5f0220bf2a29e92a5cd41977b04
    Reviewed-on: https://gerrit.openafs.org/14993
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/LINUX/osi_probe.c | 28 ++++++++++------------------
 1 file changed, 10 insertions(+), 18 deletions(-)

-- 
OpenAFS Master Repository