Hi Nik,
awesome to see this patchset! :-) I've begun setting things up here
for testing. Just have a question about this:
On Thu, Aug 26, 2021 at 16:05, Nikolay Aleksandrov [off-list ref] wrote:
Add control and dump support for the global mcast_igmp_version option
which controls the IGMP version on the vlan (default 2).
Why is the default IGMPv2? Since we support IGMPv3, surely that should
be the default, with fallback to IGMPv2 when we detect end devices that
don't support v3?
The snooping RFC refers back to the IGMPv3 RFC
https://datatracker.ietf.org/doc/html/rfc3376#section-7
I noticed the default for MLD is also set to the older version v1, and
I'm guessing there's a reasoning behind both that I haven't yet grasped.
Best regards
/Joachim