search

bisdn / basebox

A tiny OpenFlow controller for OF-DPA switches.
Mozilla Public License 2.0
45 stars 9 forks source link

nl_bridge: take over fdb entries instead of creating them #366

Closed KanjiMonster closed 2 years ago

KanjiMonster commented 2 years ago

A recent fix for the kernel allows fdb entries to be switched to/from ext_lern, so instead of inspecting every packet on reception, let the kernel learn and take over fdb entries as ncessary.

Signed-off-by: Jonas Gorski jonas.gorski@bisdn.de

Description

Instead of trying to parse every incoming packet and try to learn the neighbors, just pass them on to the kernel and let it parse them, then handle the new neighbor notification.

This allows dropping the special handling in the RX path for bridges.

Motivation and Context

Adding support for Broadcom KNET intertfaces means we will not be able to inspect every packet, so we need to instead take over learned neighbors.

How Has This Been Tested?

Running a pipeline on switches, but should be tested more.

KanjiMonster commented 2 years ago

I'm not 100% sure about the vxlan changes (if they are enough), but the test succeeds with it.