OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-323-g35c8c1b

Gerrit Code Review gerrit@openafs.org
Wed, 6 Mar 2024 23:37:31 -0500


The following commit has been merged in the master branch:
commit 35c8c1bf0b1cb48178f676ba5bcf16ad59c5a33b
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Tue Mar 5 08:37:47 2024 -0700

    Linux 6.8: Remove ctl_table sentinels
    
    The Linux 6.8 commit 'sysctl: Remove the now superfluous sentinel
    elements from ctl_table array' (c8a65501d3) was a clean up commit
    that removed the sentinel entry in the ctl_table array (e.g. the
    "null" entry at the end of the table).
    
    As of Linux 6.8, including the sentinel entry (.procname =) in the
    ctl_table is unnecessary, but doesn't yet break anything.  But it is
    likely that including the sentinel will start to cause runtime errors in
    future Linux versions very soon, so avoid the sentinel when we can, to
    avoid possible problems in the future.
    
    Define a new macro that can be used as the last entry of a ctl_table
    that will either add a "null" entry, or nothing.
    
    There is not a specific build test we can use within configure, so we
    must explicitly test the Linux version to decide if we need to use a
    sentinel or not when defining the macro.  We are selecting 6.8 to match
    the version where the Linux kernel is removing the sentinels from the in
    kernel filesystems.
    
    Note: See the Linux merge commits 'Merge tag 'sysctl-6.8-rc1' of
    git://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux' (a05aea98d4)
    for more details behind the staged removal of the sentinels in the
    ctl_table structures and the potential future change for removing the
    actual check for the sentinel within the Linux kernel.
    
    Change-Id: I0f50872e7d08dc198584623337909a180002ab30
    Reviewed-on: https://gerrit.openafs.org/15645
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/LINUX/osi_sysctl.c | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

-- 
OpenAFS Master Repository