OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_14-8-g5f5d5c4

Gerrit Code Review gerrit@openafs.org
Thu, 20 Nov 2025 15:51:23 -0500


The following commit has been merged in the openafs-stable-1_8_x branch:
commit c379ff006d8b7db425f7648321c549ab24919d92
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Thu Oct 23 15:15:38 2025 -0600

    linux: remove implied def HAVE_LINUX_FILEMAP_GET_FOLIO
    
    Commit:
      Linux: Use struct kiocb * for aops write_begin/end (1ad827ded0ed9)
    added 2 implied defines in osi_vnodeops, one for
    HAVE_LINUX_WRITE_BEGIN_END_FOLIO and one for HAVE_LINUX_FILEMAP_GET_FOLIO.
    The HAVE_LINUX_FILEMAP_GET_FOLIO was not needed, and causes a build
    failure:
    
      osi_vnodeops.c:4056:10: error: "HAVE_LINUX_FILEMAP_GET_FOLIO" redefined [-Werror]
       4056 | # define HAVE_LINUX_FILEMAP_GET_FOLIO
            |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    This is because the configure test for __filemap_get_folio() succeeds
    (linux 6.15 and later) and the test for write_begin/end functions take a
    kiocb also succeeds (6.17 or later).
    
    Remove the unnecessary define.
    
    Reviewed-on: https://gerrit.openafs.org/16589
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    (cherry picked from commit 503ef823970695eb646a323f38b9c07d52971e69)
    
    Change-Id: I6a5768ac8e8f7cce79c5fbd67d5dcebfb4aead98
    Reviewed-on: https://gerrit.openafs.org/16604
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/LINUX/osi_vnodeops.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
OpenAFS Master Repository