[B.A.T.M.A.N.] [PATCHv14 0/4] Multicast optimizations for bridges
linus.luessing at c0d3.blue
Tue May 10 02:03:11 CEST 2016
On Thu, May 05, 2016 at 11:15:28PM +0200, Sven Eckelmann wrote:
> On Thursday 05 May 2016 17:07:01 Linus Lüssing wrote:
> > This patchset can be found in the current linus/multicast-bridge
> > branch.
> Please check the attached results of the daily build test scripts.
Just wanted to note that there seem to be some false positives in
"[net/batman-adv/multicast.c:1181]: (style) The function 'batadv_mcast_flags_seq_print_text' is never used."
-> used in debugfs.c
Also the header suggestions seem a bit unreliable to me,
the following removals are false positives, I think:
* header removals in compat-sources/*
-> breaks compiling
* linux/icmp6.h in multicast.c
* linux/ipv6.h in multicast.c
* net/addrconf.h in multicast.c
* net/ip.h in multicast.c
* net/ipv6.h in multicast.c
Moving the following includes upwards seems unnecessary:
* linux/if_bridge.h in multicast.c
* linux/igmp.h in multicast.c
The following removals seem correct \o/:
* linux/in6.h in multicast.c
* net/mld.h in multicast.c
Additionally, the following include seems to be missing but wasn't
caught by the scripts:
* net/if_inet6.h in multicast.c
Let me know whether this seems about right. Or whether I am
missing #includes in my->head :).
PS: I also do not understand the sparse complaints regarding
ip_eth_mc_map(). The according include is currently there, seems
to compile fine.
More information about the B.A.T.M.A.N