fff-babeld: add MTU option to babelpeers #125

Open
jkimmel wants to merge 2 commits from jkimmel/firmware:babelpeermtu into master
Owner

Currently the MTU of a babelpeers is not configurable, nor set to a
default value. If present, the mtu option is now copied to the
interface section, otherwise it is set to a default value of 1500.

Warning for VLAN Interfaces:

For MTUs larger than 1500 to take effect, the parent interface also
needs adjusting. E.g. setting eth0.100 to an MTU larger than the MTU
of eth0 will not work.

At this point, manually add the following section to
/etc/config/network:

config interface
	option ifname 'eth0'
	option mtu '1570'

Beware that this will cause any child interface of eth0 to have the
same large MTU.

Signed-off-by: Johannes Kimmel fff@bareminimum.eu

Currently the MTU of a babelpeers is not configurable, nor set to a default value. If present, the `mtu` option is now copied to the interface section, otherwise it is set to a default value of 1500. Warning for VLAN Interfaces: For MTUs larger than 1500 to take effect, the parent interface also needs adjusting. E.g. setting `eth0.100` to an MTU larger than the MTU of `eth0` will not work. At this point, manually add the following section to `/etc/config/network`: ``` config interface option ifname 'eth0' option mtu '1570' ``` Beware that this will cause any child interface of `eth0` to have the same large MTU. Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>
jkimmel added the
layer3
packages/fff
RFC
labels 2021-02-17 06:47:08 +01:00
jkimmel added 2 commits 2021-02-17 06:47:08 +01:00
057541129d fff-babeld: clean up type/rxcost option handling
Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>
f333c3c822 fff-babeld: add MTU option to babelpeers
Currently the MTU of a babelpeers is not configurable, nor set to a
default value. If present, the `mtu` option is now copied to the
interface section, otherwise it is set to a default value of 1500.

Warning for VLAN Interfaces:

For MTUs larger than 1500 to take effect, the parent interface also
needs adjusting. E.g. setting `eth0.100` to an MTU larger than the MTU
of `eth0` will not work.

At this point, manually add the following section to
`/etc/config/network`:

```
config interface
	option ifname 'eth0'
	option mtu '1570'
```

Beware that this will cause any child interface of `eth0` to have the
same large MTU.

Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>
jkimmel added a new dependency 2021-02-17 07:17:05 +01:00
fbl requested changes 2021-03-02 00:21:59 +01:00
fbl left a comment
Owner

Das was im ersten Commit gemacht wird, sollte man für alle derartige Vorkommen (speziell in /etc/layer3.d) machen. Gefällt mir gut.

Statt die MTU explizit auf 1500 zu setzen würde ich hier gerne dabei bleiben die MTU im Default-Fall nicht explizit zu setzen, wodurch sie dann beim Linux Default bleibt. Man kann babelpeer nämlich nicht nur für Ethernet-Interfaces her nehmen, sondern für beliebige Interfaces, die per Default auch etwas anderes als 1500 MTU haben könnten.

Das was im ersten Commit gemacht wird, sollte man für alle derartige Vorkommen (speziell in /etc/layer3.d) machen. Gefällt mir gut. Statt die MTU explizit auf 1500 zu setzen würde ich hier gerne dabei bleiben die MTU im Default-Fall nicht explizit zu setzen, wodurch sie dann beim Linux Default bleibt. Man kann babelpeer nämlich nicht nur für Ethernet-Interfaces her nehmen, sondern für beliebige Interfaces, die per Default auch etwas anderes als 1500 MTU haben könnten.
jkimmel removed a dependency 2024-03-12 12:31:40 +01:00
This pull request has changes conflicting with the target branch.
  • src/packages/fff/fff-babeld/Makefile
  • src/packages/fff/fff-babeld/files/etc/layer3.d/40-babel
Sign in to join this conversation.
No description provided.