mptcp: remove tx_pending_data
authorFlorian Westphal <fw@strlen.de>
Fri, 24 Sep 2021 21:12:37 +0000 (14:12 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sat, 25 Sep 2021 10:36:51 +0000 (11:36 +0100)
commit9e65b6a5aaa3236488b4f4e3e8b914d73124a5a5
tree56cc227a68d6ef5b3e0e4b3f12231a21fada3e4a
parent765ff425528f309b166978c4b295eb47ebefd47a
mptcp: remove tx_pending_data

The update on recovery is not correct.

msk->tx_pending_data += msk->snd_nxt - rtx_head->data_seq;

will update tx_pending_data multiple times when a subflow is declared
stale while earlier recovery is still in progress.
This means that tx_pending_data will still be positive even after
all data as has been transmitted.

Rather than fix it, remove this field: there are no consumers.
The outstanding data byte count can be computed either via

 "msk->write_seq - rtx_head->data_seq" or
 "msk->write_seq - msk->snd_una".

The latter is more recent/accurate estimate as rtx_head adjustment
is deferred until mptcp lock can be acquired.

Acked-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/mptcp/protocol.c
net/mptcp/protocol.h