Original cover letter:
Currently, the following case doesn't use DCTCP, even if it should:
- responder has f.e. cubic as system wide default
- 'ip route congctl dctcp $src' was set
Then, DCTCP is NOT used if a DCTCP sender attempts to connect from a
host in the $src range: ECT(0) is set, but listen_sk is not dctcp, so
we fail the INET_ECN_is_not_ect sanity check.
We also have to examine the dst used for the SYN/ACK reply to make
this case work.
In order to minimize additional cost, store the 'ecn is must have'
information is the dst_features field.
The set targets -next instead of -net since this doesn't seem to be a
serious bug and to give the change more soak time until it hits linus
tree.
v1 -> v2:
- Addressed Dave's feedback, not exposing any bits to user space
- Added patch 3 to reject incorrect configurations
- Rest as is, rebased and retested
Thanks!
Daniel Borkmann (3):
net: fib6: reduce identation in ip6_convert_metrics
fib, fib6: reject invalid feature bits
tcp: use dctcp if enabled on the route to the initiator
Florian Westphal (1):
net: fib: move metrics parsing to a helper
include/net/dst.h | 6 ++++
include/net/tcp.h | 2 +-
include/uapi/linux/rtnetlink.h | 11 +++---
net/core/rtnetlink.c | 6 ++++
net/ipv4/fib_semantics.c | 77 ++++++++++++++++++++++++++----------------
net/ipv4/tcp_cong.c | 9 +++--
net/ipv4/tcp_input.c | 7 ++--
net/ipv6/route.c | 39 ++++++++++++---------
8 files changed, 101 insertions(+), 56 deletions(-)
--
1.9.3
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
