From ef78554c5d07dd13b83be5d706483094e907b207 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Fri, 5 Nov 2010 00:23:23 +0000 Subject: [PATCH] rx: Don't maintain maxSerial There were no users within the code of the rx connection maxSerial element, and maintaining it required locking on a critical path. So, get rid of it. Reviewed-on: http://gerrit.openafs.org/4797 Reviewed-by: Derrick Brashear Tested-by: Jeffrey Altman Reviewed-by: Jeffrey Altman (cherry picked from commit 6a22a2f4b7aa0d960cea17b419003986f6184d58) Change-Id: I465e73b7cc7c57b384cd2652bbc8825a898b8bbc Reviewed-on: http://gerrit.openafs.org/4930 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear --- src/rx/rx.c | 5 ----- src/rx/rx.h | 1 - 2 files changed, 6 deletions(-) diff --git a/src/rx/rx.c b/src/rx/rx.c index 45f274374..b71370aab 100644 --- a/src/rx/rx.c +++ b/src/rx/rx.c @@ -3175,11 +3175,6 @@ rxi_ReceivePacket(struct rx_packet *np, osi_socket socket, return np; } - MUTEX_ENTER(&conn->conn_data_lock); - if (conn->maxSerial < np->header.serial) - conn->maxSerial = np->header.serial; - MUTEX_EXIT(&conn->conn_data_lock); - /* If the connection is in an error state, send an abort packet and ignore * the incoming packet */ if (conn->error) { diff --git a/src/rx/rx.h b/src/rx/rx.h index 957ade6b6..7f5b1403f 100644 --- a/src/rx/rx.h +++ b/src/rx/rx.h @@ -249,7 +249,6 @@ struct rx_connection { * call slot, or 0 if the slot is not busy */ afs_uint32 serial; /* Next outgoing packet serial number */ afs_uint32 lastSerial; /* # of last packet received, for computing skew */ - afs_int32 maxSerial; /* largest serial number seen on incoming packets */ afs_int32 lastPacketSize; /* last >max attempt */ afs_int32 lastPacketSizeSeq; /* seq number of attempt */ afs_int32 lastPingSize; /* last MTU ping attempt */ -- 2.39.5