OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-65-g7739d1e

Gerrit Code Review gerrit@openafs.org
Thu, 5 Aug 2021 11:43:29 -0400


The following commit has been merged in the master branch:
commit 7739d1e967177686e91a6ea86c73b404515a1c0d
Author: Jeffrey Altman <jaltman@auristor.com>
Date:   Fri Oct 2 19:51:06 2020 -0400

    rx: compare RX_ACK_TYPE_ACK as a bit-field
    
    The rx_ackPacket.acks array (the SACK table) consists of up to 255
    octets.  Each octet stores either the value zero (RX_ACK_TYPE_NACK)
    or one (RX_ACK_TYPE_ACK).  Effectively only bit-zero of each octet
    is used.
    
    The rx_ackPacket.acks array cannot be enlarged but one possible
    method of encoding the ACK/NACK state for packets when the
    window size is greater than 255 is to use bits 1-7 of each
    octet.
    
    This change alters the test for ACK vs NACK to be a bit comparison
    instead of a equality comparison.  This change permits RX to be
    compatible with any future use of bits 1-7.
    
    No peer that treats the SACK table as bytes can ever send more
    than 255 packets regardless of the advertised receive window.
    Therefore, existing peers will never receive a SACK table with
    more than 255 packets worth of bits.
    
    Change-Id: I4dfcdc9ea18e060eeb257832297f557b7109e93a
    Reviewed-on: https://gerrit.openafs.org/14465
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/rx/rx.c |   12 +++++++-----
 src/rx/rx.h |   25 +++++++++++++++++++------
 2 files changed, 26 insertions(+), 11 deletions(-)

-- 
OpenAFS Master Repository