batman-adv: decrease maximum fragment size
authorMatthias Schiffer <mschiffer@universe-factory.net>
Wed, 22 Feb 2017 16:25:41 +0000 (17:25 +0100)
committerSimon Wunderlich <sw@simonwunderlich.de>
Tue, 23 May 2017 10:09:13 +0000 (12:09 +0200)
commit8ea026b1601c68fd133c3f3b0350dd1cd59ae352
tree300ab68dcf3b90914f5e9472b28f2418360380f1
parentb1d2cf3de334d84812ea2a2ab287867f07c3ef26
batman-adv: decrease maximum fragment size

With this patch the maximum fragment size is reduced from 1400 to 1280
bytes.

Fragmentation v2 correctly uses the smaller of 1400 and the interface
MTU, thus generally supporting interfaces with an MTU < 1400 bytes, too.

However, currently "Fragmentation v2" does not support re-fragmentation.
Which means that once a packet is split into two packets of 1400 + x
bytes for instance and the next hop provides an interface with an even
smaller MTU of 1280 bytes, then the larger fragment is lost.

A maximum fragment size of 1280 bytes is a safer option as this is the
minimum MTU required by IPv6, making interfaces with an MTU < 1280
rather exotic.

Regarding performance, this should have no negative impact on unicast
traffic: Having some more bytes in the smaller and some less in the
larger does not change the sum of both fragments.

Concerning TT, choosing 1280 bytes fragments might result in more TT
messages than necessary when a large network is bridged into batman-adv.
However, the TT overhead in general is marginal due to its reactive
nature, therefore such a performance impact on TT should not be
noticeable for a user.

Cc: Matthias Schiffer <mschiffer@universe-factory.net>
[linus.luessing@c0d3.blue: Added commit message]
Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
net/batman-adv/main.h