From: Andrew Lunn <andrew@lunn.ch>
To: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Cc: Tobias Waldekranz <tobias@waldekranz.com>,
Network Development <netdev@vger.kernel.org>,
Vivien Didelot <vivien.didelot@gmail.com>,
Horatiu Vultur <horatiu.vultur@microchip.com>,
Vladimir Oltean <olteanv@gmail.com>
Subject: Re: commit 4c7ea3c0791e (net: dsa: mv88e6xxx: disable SA learning for DSA and CPU ports)
Date: Mon, 18 Jan 2021 18:50:45 +0100 [thread overview]
Message-ID: <YAXKdWL9CdplNrtm@lunn.ch> (raw)
In-Reply-To: <af05538b-7b64-e115-6960-0df8e503dde3@prevas.dk>
> I suppose the real solution is having userspace do some "bridge mdb add"
> yoga, but since no code currently uses
> MV88E6XXX_G1_ATU_DATA_STATE_MC_STATIC_DA_MGMT, I don't think there's any
> way to actually achieve this. And I have no idea how to represent the
> requirement that "frames with this multicast DA are only to be directed
> at the CPU" in a hardware-agnostic way.
The switchdev interface for this exists, because there can be
multicast listeners on the bridge. When they join a group, they ask
the switch to put in a HOST MDB, which should cause the traffic for
the group to be sent to the host. What you don't have is the
exclusivity. If there is an IGMP report for the DA received on another
port, IGMP snooping will add an MDB entry to forward traffic out that
port.
Andrew
next prev parent reply other threads:[~2021-01-18 17:52 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-14 13:49 commit 4c7ea3c0791e (net: dsa: mv88e6xxx: disable SA learning for DSA and CPU ports) Rasmus Villemoes
2021-01-16 1:42 ` Tobias Waldekranz
2021-01-18 15:41 ` Rasmus Villemoes
2021-01-18 17:50 ` Andrew Lunn [this message]
2021-01-18 18:30 ` Tobias Waldekranz
2021-01-18 19:01 ` Andrew Lunn
2021-01-18 19:10 ` Tobias Waldekranz
2021-01-18 21:19 ` Vladimir Oltean
2021-01-18 22:07 ` Rasmus Villemoes
2021-01-18 23:08 ` Tobias Waldekranz
2021-01-19 9:15 ` Horatiu Vultur
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=YAXKdWL9CdplNrtm@lunn.ch \
--to=andrew@lunn.ch \
--cc=horatiu.vultur@microchip.com \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--cc=rasmus.villemoes@prevas.dk \
--cc=tobias@waldekranz.com \
--cc=vivien.didelot@gmail.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).