On Thu, Sep 02, 2021 at 01:36:19PM +0800, DENG Qingfang wrote: > On Fri, Aug 27, 2021 at 12:19:23PM +0200, Greg KH wrote: > > On Tue, Aug 24, 2021 at 11:57:53PM +0800, DENG Qingfang wrote: > > > Standalone ports should have address learning disabled, according to > > > the documentation: > > > https://www.kernel.org/doc/html/v5.14-rc7/networking/dsa/dsa.html#bridge-layer > > > dsa_switch_ops on 5.10 or earlier does not have .port_bridge_flags > > > function so it has to be done differently. > > > > > > I've identified an issue related to this. > > > > What issue is that? Where was it reported? > > See Florian's message here > https://lore.kernel.org/stable/20210317003549.3964522-2-f.fainelli@xxxxxxxxx/ THat is just the patch changelog text, or is it unique to this stable-only patch? It is not obvious at all. > > > > 2. A partial backport of this patch? > > > > > > The other part does not actually fix anything. > > > > Then why is it not ok to just take the whole thing? > > > > When backporting not-identical-patches, something almost always goes > > wrong, so we prefer to take the original commit when ever possible. > > Okay. MDB and tag ops can be backported as is, and broadcast/multicast > flooding can be implemented in .port_egress_floods. So what are we supposed to do here? totally confused, greg k-h