OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-devel-1_5_77-128-gdab406b

Gerrit Code Review gerrit@openafs.org
Tue, 26 Oct 2010 17:27:24 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit dab406baf28c19710b4a451ae2516dfa1ebeee29
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Mon Oct 25 20:50:29 2010 +0100

    rx: Fix starting of transmit timers
    
    The code used to start the transmit timer once for every set of packets
    that it sends. However, these packets might be sent individually or in
    clumps, with blocking for sendmsg, and on peer->lock, between each set
    of packet sends. This has the effect of, even on a very stable network,
    producing a high degree of variation in RTTs and timeouts. This is a
    particular issue where the connection size is larger, as the number of
    packets being sent individually under the one timer grows too.
    
    Fix this by moving timer initialisation to SendList. This already takes
    the peer lock, so obtain the timeout value here too. This means that
    each jumbo gram, or individual packet (where jumbograms are disabled)
    is sent with its own start time, and stabilises RTTs.
    
    (cherry picked from commit e7066a9c30047e787d72f02dd1c28e5e473e494b)
    Reviewed-on: http://gerrit.openafs.org/3137
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    
    Change-Id: I64edd961e7e9cd615e07cc3f32d9bf71f74c8c8e
    Reviewed-on: http://gerrit.openafs.org/3155
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/rx/rx.c |   61 ++++++++++++++++++++++------------------------------------
 1 files changed, 23 insertions(+), 38 deletions(-)

-- 
OpenAFS Master Repository