On Monday, 26 December 2022 17:15:50 CET Linus Lüssing wrote: > The multicast code to send a multicast packet via multiple batman-adv > unicast packets is not only capable of sending to multiple but also to a > single node. Therefore we can safely remove the old, specialized, now > redundant multicast-to-single-unicast code. > > The only functional change of this simplification is that the edge case > of allowing a multicast packet with an unsnoopable destination address > (224.0.0.0/24 or ff02::1) where only a single node has signaled interest > in it via the batman-adv want-all-unsnoopables multicast flag is now > transmitted via a batman-adv broadcast instead of a batman-adv unicast > packet. Maintaining this edge case feature does not seem worth the extra > lines of code and people should just not expect to be able to snoop and > optimize such unsnoopable multicast addresses when bridges are involved. > > While at it also renaming a few items in the batadv_forw_mode enum to > prepare for the new batman-adv multicast packet type. > > Signed-off-by: Linus Lüssing > --- > net/batman-adv/multicast.c | 248 ++------------------------------ > net/batman-adv/multicast.h | 38 +---- > net/batman-adv/soft-interface.c | 25 ++-- > 3 files changed, 33 insertions(+), 278 deletions(-) Output of different failed tests ================================ ecsv/pu: headers ---------------- diff --git a/net/batman-adv/multicast.c b/net/batman-adv/multicast.c index 89ba1093..6f387e61 100644 --- a/net/batman-adv/multicast.c +++ b/net/batman-adv/multicast.c @@ -26,7 +26,6 @@ #include #include #include -#include #include #include #include diff --git a/net/batman-adv/soft-interface.c b/net/batman-adv/soft-interface.c index c2fa0ef5..cd04ab27 100644 --- a/net/batman-adv/soft-interface.c +++ b/net/batman-adv/soft-interface.c @@ -48,7 +48,6 @@ #include "hard-interface.h" #include "multicast.h" #include "network-coding.h" -#include "originator.h" #include "send.h" #include "translation-table.h"