diff options
| author | David Howells <dhowells@redhat.com> | 2018-05-10 23:26:00 +0100 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2018-05-10 23:26:00 +0100 |
| commit | c54e43d752c7187595c8c62a231e0b0d53c7fded (patch) | |
| tree | 3e813d2af3416daf069285dbaefced6e2cc89982 /net/rxrpc/ar-internal.h | |
| parent | 4a026da91caaa36004a53a844dd00959370ea8fc (diff) | |
| download | linux-c54e43d752c7187595c8c62a231e0b0d53c7fded.tar.gz linux-c54e43d752c7187595c8c62a231e0b0d53c7fded.tar.bz2 linux-c54e43d752c7187595c8c62a231e0b0d53c7fded.zip | |
rxrpc: Fix missing start of call timeout
The expect_rx_by call timeout is supposed to be set when a call is started
to indicate that we need to receive a packet by that point. This is
currently put back every time we receive a packet, but it isn't started
when we first send a packet. Without this, the call may wait forever if
the server doesn't deign to reply.
Fix this by setting the timeout upon a successful UDP sendmsg call for the
first DATA packet. The timeout is initiated only for initial transmission
and not for subsequent retries as we don't want the retry mechanism to
extend the timeout indefinitely.
Fixes: a158bdd3247b ("rxrpc: Fix call timeouts")
Reported-by: Marc Dionne <marc.dionne@auristor.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'net/rxrpc/ar-internal.h')
| -rw-r--r-- | net/rxrpc/ar-internal.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/rxrpc/ar-internal.h b/net/rxrpc/ar-internal.h index 90d7079e0aa9..19975d2ca9a2 100644 --- a/net/rxrpc/ar-internal.h +++ b/net/rxrpc/ar-internal.h @@ -476,6 +476,7 @@ enum rxrpc_call_flag { RXRPC_CALL_SEND_PING, /* A ping will need to be sent */ RXRPC_CALL_PINGING, /* Ping in process */ RXRPC_CALL_RETRANS_TIMEOUT, /* Retransmission due to timeout occurred */ + RXRPC_CALL_BEGAN_RX_TIMER, /* We began the expect_rx_by timer */ }; /* |