From: Joachim Wiberg <troglobit@gmail.com>
To: Nikolay Aleksandrov <nikolay@nvidia.com>,
Horatiu Vultur <horatiu.vultur@microchip.com>,
davem@davemloft.net, kuba@kernel.org, roopa@nvidia.com,
allan.nielsen@microchip.com, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, bridge@lists.linux-foundation.org
Subject: Re: [RFC PATCH v2] net: bridge: igmp: Extend IGMP query to be per vlan
Date: Fri, 22 Jan 2021 17:05:57 +0100 [thread overview]
Message-ID: <87czxxvtwa.fsf@gmail.com> (raw)
In-Reply-To: <b54644f6-b559-b13b-adf8-d95f7b2a6885@nvidia.com>
On Mon, Jan 18, 2021 at 13:53, Nikolay Aleksandrov <nikolay@nvidia.com> wrote:
> On 16/01/2021 17:39, Joachim Wiberg wrote:
>> We have discussed at length at work if an IGMP snooping implementation
>> really belongs in the bridge, or if it's better suited as a user space
>> daemon? Similar to what was decided for RSTP/MSTP support, i.e., the
>> bridge only has STP and RSTP/MSTP is handled by mstpd[2].
>>
>> Most of what's required for a user space implementation is available,
>> but it would've been nice if a single AF_PACKET socket on br0 could be
>> used to catch what brport (ifindex) a query or report comes in on. As
>> it is now that information is lost/replaced with the ifindex of br0.
>> And then there's the issue of detecting and forwarding to a multicast
>> routing daemon on top of br0. That br0 is not a brport in the MDB, or
>> that host_joined cannot be set/seen with iproute2 is quite limiting.
>> These issues can of course be addressed, but are they of interest to
>> the community at large?
>>
>> [1]: https://lore.kernel.org/netdev/20180418120713.GA10742@troglobit/
>> [2]: https://github.com/mstpd/mstpd
Ni Nik,
> I actually had started implementing IGMPv3/MLDv2 as a user-space daemon part of
> FRRouting (since it already has a lot of the required infra to talk to the kernel).
> It also has IGMPv3/MLDv2 support within pimd, so a lot of code can be shared.
Interesting! Glad to hear other people have had similar ideas :)
> Obviously there are pros and cons to each choice, but I'd be interested to see a
> full user-space implementation. I decided to make the kernel support more complete
> since it already did IGMPv2 and so stopped with the new FRR daemon.
Yeah it's difficult to find the right cut-off for when it'll be more
useful to do it all in userspace. For us I think it was the combination
of having many VLANs and wanting full querier support, i.e., not having
any multicast router available. When we had to go dumpster diving for
useful IP address in IGMP queries on higher-level VLAN interfaces.
> If needed I'd be happy to help with the kernel support for a new
> user-space daemon, and also can contribute to the daemon itself if
> time permits.
That's good to know. I think I'll start breathing life into a small
IGMP-only (for now) userspace daemon and see where it leads. I need
it both for work and for all the various multicast routing projects
I maintain on my spare time.
Would it be OK to send questions regarding issues interfacing with the
bridge and updates/progress to this list for such a project?
Best regards
/Joachim
prev parent reply other threads:[~2021-01-22 16:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-12 13:59 [RFC PATCH v2] net: bridge: igmp: Extend IGMP query to be per vlan Horatiu Vultur
2021-01-13 12:15 ` Nikolay Aleksandrov
2021-01-13 17:02 ` Horatiu Vultur
2021-01-16 15:39 ` Joachim Wiberg
2021-01-18 11:53 ` Nikolay Aleksandrov
2021-01-22 16:05 ` Joachim Wiberg [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87czxxvtwa.fsf@gmail.com \
--to=troglobit@gmail.com \
--cc=allan.nielsen@microchip.com \
--cc=bridge@lists.linux-foundation.org \
--cc=davem@davemloft.net \
--cc=horatiu.vultur@microchip.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nikolay@nvidia.com \
--cc=roopa@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).