All of lore.kernel.org
 help / color / mirror / Atom feed
* [MPTCP] [RFC PATCH 0/6] mptcp: add and use wmem accounting for rtx queue
@ 2019-11-08 22:09 Florian Westphal
  0 siblings, 0 replies; only message in thread
From: Florian Westphal @ 2019-11-08 22:09 UTC (permalink / raw)
  To: mptcp

[-- Attachment #1: Type: text/plain, Size: 1805 bytes --]

This series adds wmem accounting and makes mptcp sockets block once
the sndbuf size is reached.  rtx cleanup is changed to wakeup any
blocked mptcp socket once enough wmem is available again.

sndbuf is set to the maximum allowed size of the tcp wmem.
There is no auto-tuning, I think we can add that later on as an
enhancement.

This is an RFC series in order to prevent the patches from getting
applied, I will send a v2 as soon as the open questions are sorted
out.  Everthing continues to work for me.

I've tested these patches with an 4k-sized sndbuf, which results
in frequent wmem blocks and mptcp connection recovers once the acks
come in.

1.  (see patch 5): add a new work queue or re-use existing one?
2.  Should all patches get squashed or should there be a
    "add wmem accouting" patch?  A mix of both?

2, 3 and 4 would be squash candidates for example even if we add
a extra "wmem patch".

I will send the next iteration of the scheduler patches after the above
is sorted out.
The scheduler patches have no major changes aside from addressing a
comment from Paolo.

Meanwhile I am working on removal of __tcp_poll and on make the mptcp
socket conn list store tcp_sk structs rather than require a 'struct socket'.

Florian Westphal (6):
      mptcp: add wmem_queued accounting
      mptcp: allow partial cleaning of rtx head dfrag
      subflow: store known available bytes in subflow
      options: ack pending sequence
      mptcp: add ack work queue
      sendmsg: block until mptcp sk is writeable

 net/mptcp/options.c  | 13 ++++++--
 net/mptcp/protocol.c | 86 +++++++++++++++++++++++++++++++++++++++++++++++-----
 net/mptcp/protocol.h |  5 +--
 net/mptcp/subflow.c  | 18 ++++++++---
 4 files changed, 105 insertions(+), 17 deletions(-)


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2019-11-08 22:09 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-08 22:09 [MPTCP] [RFC PATCH 0/6] mptcp: add and use wmem accounting for rtx queue Florian Westphal

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.