From: Chas Williams (CONTRACTOR) <chas@cmf.nrl.navy.mil>
Date: Wed, 20 Oct 2010 13:06:35 +0000 (-0400)
Subject: rx: remove spurious compare for maxDgramPackets
X-Git-Tag: openafs-devel-1_5_78~56
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=5396d713327509d8bbf413be3db1c7da60f097e7;p=packages%2Fo%2Fopenafs.git

rx: remove spurious compare for maxDgramPackets

maxDgramPackets is initially assigned this value after correcting
for the wire endian.  This compare is harmless on little endian
since the network endian value will typically be huge and redundant
on big endian machines.

Reviewed-on: http://gerrit.openafs.org/3015
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
(cherry picked from commit da400a24a7c9d5ac1595f10e0c76df5a461aa5dc)
Change-Id: I3e8c77e30021055192951765bb823b635775e128
Reviewed-on: http://gerrit.openafs.org/3114
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/rx/rx.c b/src/rx/rx.c
index 3befc6376..03d44771a 100644
--- a/src/rx/rx.c
+++ b/src/rx/rx.c
@@ -4243,7 +4243,6 @@ rxi_ReceiveAckPacket(struct rx_call *call, struct rx_packet *np,
 	    maxDgramPackets = MIN(maxDgramPackets, rxi_nDgramPackets);
 	    maxDgramPackets =
 		MIN(maxDgramPackets, (int)(peer->ifDgramPackets));
-	    maxDgramPackets = MIN(maxDgramPackets, tSize);
 	    if (maxDgramPackets > 1) {
 		peer->maxDgramPackets = maxDgramPackets;
 		call->MTU = RX_JUMBOBUFFERSIZE + RX_HEADER_SIZE;