On Mon, Aug 21, 2017 at 09:43:15PM -0700, Roopa Prabhu wrote: > > This is relevant to the discussion because it's a feature which is > > non-obvious (to me) on how to do with the VXLAN model of having an > > entirely separate FDB. Meanwhile, with this architecture, the proof of > > concept / hack is coming in at a measly cost of: > > 8 files changed, 176 insertions(+), 15 deletions(-) > > David, what is special about the vpls igmp/mld snooping code ?...do > you have to snoop vpls attrs ?. It just needs to snoop which tunnel endpoint[s] a multicast group is subscribed on. > in the vxlan model.., the vxlan driver can snoop its own attrs eg > vxlan-id, remote dst etc. > and the pkt is passed up to the bridge where it will hit the normal > bridge igmp/mpld snooping code. > can you pls elaborate ? Yes, that's exactly what the hack I have is doing, it's snooping the driver-specific attrs into a metadata_dst, passing it up to the bridge, which puts the metadata_dst on the MDB entry. What I'm arguing against is duplicating the entire MDB into all of the drivers. The snooping machinery is a pretty complex piece of code which holds information as a cartesian combination of MAC * VLAN * Group. It has several configuration knobs and can even send its own packets. It's complex enough to have been the subject of several CVEs over its history (e.g. CVE-2013-4129, CVE-2013-2636.) > keeping vpls specific code and api in a separate vpls driver allows > for cleanly extending it in the future. That's exactly my point: it's not VPLS specific. It's exactly the same functionality for VXLAN, VPLS, NVGRE and 802.11. The functionality is "remember the tunnel endpoint." It's the same bits as lwtunnel uses, which is why the 2nd patch in this series reuses the existing lwtunnel netlink encapsulation functions. -David
