diff options
| author |   David Howells <dhowells@redhat.com> | 2020-08-20 15:13:00 +0100 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2020-08-20 18:21:28 +0100 |
| commit | 1d4adfaf65746203861c72d9d78de349eb97d528 (patch) | |
| tree | 324c9592cfe0c5c63a2fb2d29d74db421cab1a88 /fs/afs | |
| parent | rxrpc: Fix loss of RTT samples due to interposed ACK (diff) | |
| download | linux-dev-1d4adfaf65746203861c72d9d78de349eb97d528.tar.xz linux-dev-1d4adfaf65746203861c72d9d78de349eb97d528.zip | |
rxrpc: Make rxrpc_kernel_get_srtt() indicate validity
Fix rxrpc_kernel_get_srtt() to indicate the validity of the returned
smoothed RTT. If we haven't had any valid samples yet, the SRTT isn't
useful.
Fixes: c410bf01933e ("rxrpc: Fix the excessive initial retransmission timeout")
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs')
| -rw-r--r-- | fs/afs/fs_probe.c | 4 | ||||
| -rw-r--r-- | fs/afs/vl_probe.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/fs/afs/fs_probe.c b/fs/afs/fs_probe.c index 5d9ef517cf81..e7e98ad63a91 100644 --- a/fs/afs/fs_probe.c +++ b/fs/afs/fs_probe.c @@ -161,8 +161,8 @@ responded: } } - rtt_us = rxrpc_kernel_get_srtt(call->net->socket, call->rxcall); - if (rtt_us < server->probe.rtt) { + if (rxrpc_kernel_get_srtt(call->net->socket, call->rxcall, &rtt_us) && + rtt_us < server->probe.rtt) { server->probe.rtt = rtt_us; server->rtt = rtt_us; alist->preferred = index; diff --git a/fs/afs/vl_probe.c b/fs/afs/vl_probe.c index e3aa013c2177..081b7e5b13f5 100644 --- a/fs/afs/vl_probe.c +++ b/fs/afs/vl_probe.c @@ -92,8 +92,8 @@ responded: } } - rtt_us = rxrpc_kernel_get_srtt(call->net->socket, call->rxcall); - if (rtt_us < server->probe.rtt) { + if (rxrpc_kernel_get_srtt(call->net->socket, call->rxcall, &rtt_us) && + rtt_us < server->probe.rtt) { server->probe.rtt = rtt_us; alist->preferred = index; have_result = true; |