OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-307-ge84193c

Gerrit Code Review gerrit@openafs.org
Tue, 26 Oct 2010 08:09:43 -0700 (PDT)


The following commit has been merged in the master branch:
commit e84193ca2a9cef5a13403d291435eb5ad47f5b41
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Mon Oct 25 09:16:09 2010 +0100

    rx: Fix resend accounting
    
    rxi_Start flagged itself as 'resending' whenever it flushed the
    transmit queue due to a resend event. However, it would flush the
    entire transmit queue at this point, rather than only transmitting
    packets that require a resend. When running with large window sizes
    this results an a large number of packets erroneously being marked
    as resent.
    
    Instead, let SendXmitList decide whether a packet is being
    retransmitted by using the presence of a serial number. This takes
    advantage of the fact that a retransmitted packet must be the only
    entry in a packet list - we just flag the packet list, instead of
    having to maintain counters for each individual packet.
    
    Change-Id: Id8e87b8efa87f5dfec2e51b7983a9fd6b1a5c4c5
    Reviewed-on: http://gerrit.openafs.org/3135
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/rx/rx.c |   40 +++++++++++++++++++++-------------------
 1 files changed, 21 insertions(+), 19 deletions(-)

-- 
OpenAFS Master Repository