From 6a22a2f4b7aa0d960cea17b419003986f6184d58 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. Change-Id: Ied5653b6f01b78525091d8bf09bdc454002eedc0 Reviewed-on: http://gerrit.openafs.org/4797 Reviewed-by: Derrick Brashear Tested-by: Jeffrey Altman Reviewed-by: Jeffrey Altman --- 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 b3a6f5a2..89e6df1 100644 --- a/src/rx/rx.c +++ b/src/rx/rx.c @@ -2994,11 +2994,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 ec2a833..48e8e3a 100644 --- a/src/rx/rx.h +++ b/src/rx/rx.h @@ -246,7 +246,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 */ -- 1.9.4