]> git.baikalelectronics.ru Git - kernel.git/commit
sctp: align MTU to a word
authorMarcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Sat, 19 Mar 2016 15:17:20 +0000 (12:17 -0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 20 Mar 2016 20:31:12 +0000 (16:31 -0400)
commitcfc17ce3cf1df72dce08dc688d45347c3a4c8a49
tree681827ec62422bf2f212913f9a696db89eaa72a6
parenta11595cef6bfb83a684862c939b6b902e41d1233
sctp: align MTU to a word

SCTP is a protocol that is aligned to a word (4 bytes). Thus using bare
MTU can sometimes return values that are not aligned, like for loopback,
which is 65536 but ipv4_mtu() limits that to 65535. This mis-alignment
will cause the last non-aligned bytes to never be used and can cause
issues with congestion control.

So it's better to just consider a lower MTU and keep congestion control
calcs saner as they are based on PMTU.

Same applies to icmp frag needed messages, which is also fixed by this
patch.

One other effect of this is the inability to send MTU-sized packet
without queueing or fragmentation and without hitting Nagle. As the
check performed at sctp_packet_can_append_data():

if (chunk->skb->len + q->out_qlen >= transport->pathmtu - packet->overhead)
/* Enough data queued to fill a packet */
return SCTP_XMIT_OK;

with the above example of MTU, if there are no other messages queued,
one cannot send a packet that just fits one packet (65532 bytes) and
without causing DATA chunk fragmentation or a delay.

v2:
 - Added WORD_TRUNC macro

Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/sctp/sctp.h
net/sctp/associola.c
net/sctp/input.c
net/sctp/transport.c