jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-18 23:10:58 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

rx: Tidy up rxi_CheckCall()'s mtuout handling

Browse Source 
We don't actually do anything that matters if lastPacketSizeSeq
is set and lastPacketSize is zero, so zero both when we're cleaning
up.

lastPacketSize and lastPacketSizeSeq are set together in
rxi_SendPacket (and rxi_SendPacketList), when we are sending a packet
larger than the current estimate of the peer's maxPacketSize.

The two fields are checked together during ack processing, but
rxi_CheckCall() only checks lastPacketSize, ignoring lastPacketSizeSeq.

Change-Id: I4e52bed0900b5551859200699f114f5d5a61581c
Reviewed-on: http://gerrit.openafs.org/11633
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Daria Brashear <shadow@your-file-system.com>
This commit is contained in:
Benjamin Kaduk 2014-12-14 16:13:39 -05:00
parent 20d5760fe9
commit 244b82d24c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/rx/rx.c
View File

@ -6308,7 +6308,7 @@ mtuout:
rxi_SetPeerMtu(conn->peer, 0, 0, newmtu); rxi_SetPeerMtu(conn->peer, 0, 0, newmtu);
/* clean up */ /* clean up */
conn->lastPacketSize = 0; conn->lastPacketSize = conn->lastPacketSizeSeq = 0;
/* needed so ResetCall doesn't clobber us. */ /* needed so ResetCall doesn't clobber us. */
call->MTU = conn->peer->ifMTU; call->MTU = conn->peer->ifMTU;