diff options
| author | Nayana Bidari <nybidari@google.com> | 2021-10-07 16:48:28 -0700 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2021-10-07 16:51:06 -0700 |
| commit | 487651ac46f302592ccffc9e5a4336a331010e42 (patch) | |
| tree | 3175111e0821c590f48fb4d4f469c5d7d2bf1de6 /pkg/tcpip/stack/tcp.go | |
| parent | 0743a862e5e90368291ad4ef9061f3c4ca3a065f (diff) | |
Add a new metric to detect the number of spurious loss recoveries.
- Implements RFC 3522 (Eifel detection algorithm) to detect if the connection
entered loss recovery unnecessarily.
- Added a new metric to count the total number of spurious loss recoveries.
- Added tests to verify the new metric.
PiperOrigin-RevId: 401637359
Diffstat (limited to 'pkg/tcpip/stack/tcp.go')
| -rw-r--r-- | pkg/tcpip/stack/tcp.go | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/pkg/tcpip/stack/tcp.go b/pkg/tcpip/stack/tcp.go index dc7289441..a941091b0 100644 --- a/pkg/tcpip/stack/tcp.go +++ b/pkg/tcpip/stack/tcp.go @@ -289,6 +289,12 @@ type TCPSenderState struct { // RACKState holds the state related to RACK loss detection algorithm. RACKState TCPRACKState + + // RetransmitTS records the timestamp used to detect spurious recovery. + RetransmitTS uint32 + + // SpuriousRecovery indicates if the sender entered recovery spuriously. + SpuriousRecovery bool } // TCPSACKInfo holds TCP SACK related information for a given TCP endpoint. |