Re: [PATCH net] net: bridge: Do not offload IGMP/MLD messages

[Joseph Huang]

On 7/2/2025 4:41 AM, Tobias Waldekranz wrote:
On tis, jul 01, 2025 at 15:36, Joseph Huang <joseph.hu...@garmin.com> wrote:
Do not offload IGMP/MLD messages as it could lead to IGMP/MLD Reports
being unintentionally flooded to Hosts. Instead, let the bridge decide
where to send these IGMP/MLD messages.

Hi Joseph,

Do I understand the situation correctly that this is the case where the
local host is sending out reports in response to a remote querier?

         mcast-listener-process (IP_ADD_MEMBERSHIP)
            \
            br0
           /   \
        swp1   swp2
          |     |
    QUERIER     SOME-OTHER-HOST

So in the above setup, br0 will want to br_forward() reports for
mcast-listener-process's group(s) via swp1 to QUERIER; but since the
source hwdom is 0, the report is eligible for tx offloading, and is
flooded by hardware to both swp1 and swp2, reaching SOME-OTHER-HOST as
well?

That's exactly what's happening with my setup.

Also, IIRC, the querier port (a.k.a. mrouter port) is not offloaded to 
the switch (at least not for DSA switches). So depending on the 
multicast_flood setting on the (querier) port, the Reports may not even 
reach the querier if they are tx offloaded.

Thanks,
Joseph