OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_13_2-39-gfe009bb

Gerrit Code Review gerrit@openafs.org
Thu, 24 Apr 2025 12:40:43 -0400


The following commit has been merged in the openafs-stable-1_8_x branch:
commit fe009bb0537c80074b4a56ce871e9043216baf32
Author: Mark Vitale <mvitale@sinenomine.net>
Date:   Mon Aug 10 13:28:19 2020 -0400

    viced: Ignore on-disk fsstate HashTable data
    
    Currently, if the fileserver tries to restore fsstate.dat and the
    included FE hashtable does not match our in-memory size (FEHASH_SIZE),
    we'll discard the entire state file. This is unnecessary, since we
    could just add the FEs to our hashtable ourselves; the on-disk
    hashtable data is not required.
    
    Furthermore, loading the hashtable from disk isn't even terribly
    efficient when the hashtable sizes do match, since any indices need to
    be converted from the on-disk values to in-memory values
    (fe_OldToNew). So if we're processing the FE hashtable indices anyway,
    we might as well just add them to the hashtable like normal, and
    ignore the values on disk.
    
    So, ignore loading the "fehash" data from the fsstate file, and just
    add file entries to our hashtable as we read them in. This ends up
    being slightly faster, simpler, and more flexible, since we don't need
    to worry about reflecting any changes to our hashtable in the
    fsstate.dat file.
    
    At least for now, we still write our hashtable data to fsstate.dat, in
    case other utilities (or older fileservers) need it.
    
    [adeason@sinenomine.net: Converted to always ignore on-disk hashtable
    data.]
    
    Reviewed-on: https://gerrit.openafs.org/14738
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    (cherry picked from commit f3629f87daa0c9067fbeb66d48ba1deb6fafac06)
    
    Change-Id: Icea91fe2ea266ae9fb53beb12dd7b664a4b30cb9
    Reviewed-on: https://gerrit.openafs.org/16022
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/viced/callback.c | 89 +++++++++++++---------------------------------------
 1 file changed, 21 insertions(+), 68 deletions(-)

-- 
OpenAFS Master Repository