Hey Linus,
thanks for working and integrating this patchset!
On Sat, May 11, 2013 at 07:23:24PM +0200, Linus Lüssing wrote:
This set of patches is the first one for a more efficient, group aware multicast forwarding infrastructure in batman-adv.
This initial set mainly consists of the integration of announcing the location of multicast listeners via the translation table mechanism to be able to find out which batman-adv nodes are actually interested in certain multicast traffic. As well as the signalizing of this announcement capability via a new multicast TVLV.
Finally some basic multicast forwarding opitimizations are introduced: If all nodes signalized the MLA capability then link-local IPv6 traffic will be dropped if there is no interested listener or gets forwarded via a batman-adv unicast packet if there is just one node interested in the data.
I've seen that you've only implemented that for IPv6, would it be possible to use it for IPv4 too in the current state? (I use some media streaming thingy at home where I could test ;])
For now, these optimizations only apply if all nodes in the mesh have no bridge interface on top their bat0 interface. However making it possible with bridges, too, and other features are on the roadmap. See the according wiki page for details [1].
So from what I understand this means:
* multicast support is working for non-bridged interfaces for now, because we lack proper multicast router discovery support (MRD) in the bridge code --> you told me someone is working on that, or you'll pick up to get this done?
* the current optimization is to handle multicast like: * no listener - drop * one listener - unicast * more listener - broadcast --> I think that is a good and simple optimization. :) The "tracker" packets can come later (as shown in your roadmap). This might require to announce that tracker packets are supported in the TVLV?
Would you consider Antonios comments and update your patchset? I would like to test it in the next days ...
Thanks, Simon