OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_18a-20-g16c68a7
Gerrit Code Review
gerrit@openafs.org
Thu, 8 Nov 2012 06:29:29 -0800 (PST)
The following commit has been merged in the openafs-devel-1_7_x branch:
commit 27acd671c85e1fc84a000c8c26c20284980ad7f5
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Fri Oct 26 14:55:02 2012 +0100
rx: Remove duplicate out of order ACK check
Once we've moved the congestion window, there's no going back. So
any ACK packets that attempt to move the window backwards by including
a 'firstPacket' value earlier than the current window position must
be ignored.
However, we check (and ignore) these packets twice. Once in
rxi_ReceivePacket, which only checks in the client side case, and again
in rxi_ReceiveAckPacket, which has a more complete check that runs for
both client and server connections.
Remove the identical check from rxi_ReceivePacket in a continuing effort
to clean up this bit of code.
Reviewed-on: http://gerrit.openafs.org/8299
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
(cherry picked from commit 8d359e6dff5317698597e77f0a1dd5ba2bfb569a)
Change-Id: I29c72136d02a16381e33d8836f35528ba7b1f055
Reviewed-on: http://gerrit.openafs.org/8396
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/rx/rx.c | 25 +------------------------
1 files changed, 1 insertions(+), 24 deletions(-)
--
OpenAFS Master Repository