Re: [B.A.T.M.A.N.] [PATCH 1/1] batman-adv: Always flood IGMP/MLD reports

["Linus Lüssing" <linus.luessing@c0d3.blue>]

On Mon, Jun 29, 2015 at 04:52:48AM +0200, Linus Lüssing wrote:
> The alternative to bumping the version number would be to introduce
> another flag into the mcast tvlv and keep counters and lists similar
> to the num_disabled or want_* ones. So maybe about 80 more lines of
> code and additional state and complexity to maintain.

Another alternative I just had to think about would be to register
both a v1 and v2 TVLV container while only registering a handler
for v2. That'd add just two or four extra lines of code with the same
result. The disadvantage would be a few bytes extra overhead for
dragging a v1 TVLV in the OGMs along.

Actually, I think I'd kinda like that as it has a reasonable
benefit-to-cost ratio. What do others think?

(And maybe such legacy multicast v1 TVLV could then safely be
obsoleted/removed in four years or after evaluating whether someone
still needs it.)