On 23/02/2022 19:24, Joachim Wiberg wrote: > This patch expands on the earlier work on layer-2 mdb entries by adding > support for host entries. Due to the fact that host joined entries do > not have any flag field, we infer the permanent flag when reporting the > entries to userspace, which otherwise would be listed as 'temp'. > > Before patch: > > ~# bridge mdb add dev br0 port br0 grp 01:00:00:c0:ff:ee permanent > Error: bridge: Flags are not allowed for host groups. > ~# bridge mdb add dev br0 port br0 grp 01:00:00:c0:ff:ee > Error: bridge: Only permanent L2 entries allowed. > > After patch: > > ~# bridge mdb add dev br0 port br0 grp 01:00:00:c0:ff:ee permanent > ~# bridge mdb show > dev br0 port br0 grp 01:00:00:c0:ff:ee permanent vid 1 > > Signed-off-by: Joachim Wiberg <[email protected]> > --- > net/bridge/br_mdb.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) >
It would be nice to add a selftest for L2 entries. You can send it as a follow-up. The patch looks good to me. Acked-by: Nikolay Aleksandrov <[email protected]>
