OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_22-1-g618d1a0

Gerrit Code Review gerrit@openafs.org
Wed, 20 Dec 2017 07:58:31 -0500


The following commit has been merged in the openafs-stable-1_6_x branch:
commit 618d1a0b2fe3b607fbcf5e07e22a6117e503e83f
Author: Damien Diederen <ddiederen@sinenomine.net>
Date:   Mon Sep 18 11:59:40 2017 +0200

    Linux: Test for __vfs_write rather than __vfs_read
    
    The following commit:
    
        commit eb031849d52e61d24ba54e9d27553189ff328174
        Author: Christoph Hellwig <hch@lst.de>
        Date:   Fri Sep 1 17:39:23 2017 +0200
    
            fs: unexport __vfs_read/__vfs_write
    
    unexports both __vfs_read and __vfs_write, but keeps the former in
    fs.h--as it is is still being used by another part of the tree.
    
    This situation results in a false positive in our Autoconf check,
    which does not see the export statements, and ends up marking the
    corresponding API as available.
    
    That, in turn, causes some code which assumes symmetry with
    __vfs_write to fail to compile.
    
    Switch to testing for __vfs_write, which correctly marks the API as
    unavailable.
    
    Reviewed-on: https://gerrit.openafs.org/12728
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 929e77a886fc9853ee292ba1aa52a920c454e94b)
    
    Reviewed-on: https://gerrit.openafs.org/12736
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit c42a25d28fbcc76fdcac2b5f29704f8f1b353b45)
    
    Change-Id: I27c1c67bed6ebb922389ebb70cbb7fc238378b58
    Reviewed-on: https://gerrit.openafs.org/12758
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 acinclude.m4                 |    4 ++--
 src/afs/LINUX/osi_compat.h   |    4 ++--
 src/afs/LINUX/osi_vnodeops.c |    2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

-- 
OpenAFS Master Repository