OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_11-19-g53752b0

Gerrit Code Review gerrit@openafs.org
Sun, 12 May 2024 22:46:25 -0400


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 53752b01bc734f4bd5f5da24ac116c7fcb2ac07f
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Mon Feb 12 09:43:29 2024 -0700

    afs: Remove SRXAFSCB* protos from afs_prototypes.h
    
    The prototypes for the afscbint RPC functions (SRXAFSCB_*) implemented
    in afs_callback.c are defined in 2 locations, afs_protypes.h and
    afscbint.h (which is generated by rxgen).
    
    Remove the protoypes for the SRXAFSCB_* functions from afs_prototypes.h
    as they are redundant and are currently out of sync (the prototypes
    for SRXAFSCB_GetCellByNum() and SRXAFSCB_TellMeAboutYourself() are
    not present in afs_prototypes.h).  Also remove a redundant prototype for
    afs_RXCallBackServer() which was prototyped both before and in the
    middle of the SRXAFSCB* prototypes.
    
    Since afs_callback.c only pulled in afs_prototypes.h and not afscbint.h,
    add an include for afscbint.h to ensure that the all the prototypes for
    the SRXAFSCB_* functions are present.
    
    The 2 functions listed above are flagged due to missing prototypes when
    building against a Linux 6.8 kernel (which sets the
    -Wmissing-declarations and -Wmissing-prototypes compiler flags as
    default). Linux 6.8 commit:
     'Makefile.extrawarn: turn on missing-prototypes globally' (0fcb70851f).
    
    When building against a kernel with CONFIG_WERROR=y, the build fails.
    
    Reviewed-on: https://gerrit.openafs.org/15644
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Tested-by: Benjamin Kaduk <kaduk@mit.edu>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit fae4003b403420d0b548920c7837a2d2b6e432ae)
    
    Change-Id: I7d435455fc60cc10200ee91d0d76534367ab5883
    Reviewed-on: https://gerrit.openafs.org/15697
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/afs_callback.c   |  2 ++
 src/afs/afs_prototypes.h | 37 -------------------------------------
 2 files changed, 2 insertions(+), 37 deletions(-)

-- 
OpenAFS Master Repository