| Message ID | 20190814144024.9710-1-nikolay@cumulusnetworks.com |
|---|---|
| Headers | show |
| Series | net: bridge: mdb: allow dump/add/del of host-joined entries | expand |
On 8/14/19 5:40 PM, Nikolay Aleksandrov wrote: > Hi, > This set makes the bridge dump host-joined mdb entries, they should be > treated as normal entries since they take a slot and are aging out. > We already have notifications for them but we couldn't dump them until > now so they remained hidden. We dump them similar to how they're > notified, in order to keep user-space compatibility with the dumped > objects (e.g. iproute2 dumps mdbs in a format which can be fed into > add/del commands) we allow host-joined groups also to be added/deleted via > mdb commands. That can later be used for L2 mcast MAC manipulation as > was recently discussed. Note that iproute2 changes are not necessary, > this set will work with the current user-space mdb code. > > Patch 01 - a trivial comment move > Patch 02 - factors out the mdb filling code so it can be > re-used for the host-joined entries > Patch 03 - dumps host-joined entries > Patch 04 - allows manipulation of host-joined entries via standard mdb > calls > > Thanks, > Nik > > Nikolay Aleksandrov (4): > net: bridge: mdb: move vlan comments > net: bridge: mdb: factor out mdb filling > net: bridge: mdb: dump host-joined entries as well > net: bridge: mdb: allow add/delete for host-joined groups > > net/bridge/br_mdb.c | 171 +++++++++++++++++++++++++------------- > net/bridge/br_multicast.c | 24 ++++-- > net/bridge/br_private.h | 2 + > 3 files changed, 133 insertions(+), 64 deletions(-) > Self-NAK There's a double notification sent for manual add/del of host groups. It's a trivial fix, I'll spin v2 later after running more tests.
Hi, This set makes the bridge dump host-joined mdb entries, they should be treated as normal entries since they take a slot and are aging out. We already have notifications for them but we couldn't dump them until now so they remained hidden. We dump them similar to how they're notified, in order to keep user-space compatibility with the dumped objects (e.g. iproute2 dumps mdbs in a format which can be fed into add/del commands) we allow host-joined groups also to be added/deleted via mdb commands. That can later be used for L2 mcast MAC manipulation as was recently discussed. Note that iproute2 changes are not necessary, this set will work with the current user-space mdb code. Patch 01 - a trivial comment move Patch 02 - factors out the mdb filling code so it can be re-used for the host-joined entries Patch 03 - dumps host-joined entries Patch 04 - allows manipulation of host-joined entries via standard mdb calls Thanks, Nik Nikolay Aleksandrov (4): net: bridge: mdb: move vlan comments net: bridge: mdb: factor out mdb filling net: bridge: mdb: dump host-joined entries as well net: bridge: mdb: allow add/delete for host-joined groups net/bridge/br_mdb.c | 171 +++++++++++++++++++++++++------------- net/bridge/br_multicast.c | 24 ++++-- net/bridge/br_private.h | 2 + 3 files changed, 133 insertions(+), 64 deletions(-)