Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Colin Foster <colin.foster@in-advantage.com>
To:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     Vladimir Oltean <vladimir.oltean@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>
Subject: [PATCH v1 net 0/1] net: mscc: ocelot: broadcast storm fixup
Date:   Wed, 22 Sep 2021 19:03:37 -0700
Message-Id: <20210923020338.1945812-1-colin.foster@in-advantage.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?KC8/wc8gaeWhgHfxhyRO2iaqaDOHIVXAZLwZjrSyZBat8XtWiVRa1We/Vz8E?=
 =?us-ascii?Q?oRTCFQ4mlQbaeU9Arpx3F8190tF/QxeRlrAUVftereEO7lnZLwUDgcLHGWQt?=
 =?us-ascii?Q?zdLv65BNVXN14+b7wyP0qrNl+fMxYBuEneFDISPdK8NezrYrYLlFjoxtsz01?=
 =?us-ascii?Q?6I0ec4WvQhMBXAYhxD3WYLxg3ZmTXm3ZKa9GQO6MWms2diAjyD2825kzCdsv?=
 =?us-ascii?Q?pyZu7vDSyotJ9PAaTART5n4DjojNozTXhfuNpntjIHh2J2uBVK8G/+6hgUKQ?=
 =?us-ascii?Q?i61rTUEqFllw6gm8LEIXEfA9GxV5J7fannoAHplHBnVNEIz3EY6uhCIEFk5y?=
 =?us-ascii?Q?3OvbTBkdMIRGDTlVoclDP/JwA3388l2iYmSi287ZdMNvoe5xJlKd2XfWb9rK?=
 =?us-ascii?Q?51ABrxkazgIqqRlHCm/29OEL8/ir66uMW4uu0whXirBTo4OMbNFYHcR47Z2a?=
 =?us-ascii?Q?e5b8iSu3900MOkzVnAHvUmqe64hTwwf8rJDWHPODiGWwrSsLI7RgJ6ao7qyS?=
 =?us-ascii?Q?WpXCDyETknWPjuXgBh5gPIbybGk1ftC4Mb3TzikjWEEZ82UsNWSxxB4prNCz?=
 =?us-ascii?Q?r8yxGXUwCf1NPvpypLZ68MJWP9RQzOt4lZx/hm+Xz6T7Urz5ZmA49NQnoKe0?=
 =?us-ascii?Q?2umE7sGWwh3kNxeM5jKEiOGnFgswkF/eK7IFFSW8x72KFdo9b2azWrgzIo7H?=
 =?us-ascii?Q?cGF6JVq7c9FpVQmKjmnq8Co7Ei2RRZhX5Ba84owx1t0gZGJH4Uy5eppWb54z?=
 =?us-ascii?Q?oiqUF5hGRECcStuASGwknpcp5Alv2qzhezAbzC4jJ70STD3/LA9BbssJgjRs?=
 =?us-ascii?Q?dpgrFFLagaf1FM7P0O6/ZEVFYzws/quI971HR8vza4+kVJd7MlFyJRa+2YPF?=
 =?us-ascii?Q?TnKgibxLlXHc90lJRUM4C/LgdRwA+QX0MPqGKQ5FlQfoe3aHFYCNe3ol1JHu?=
 =?us-ascii?Q?0BQfw1yccUaNZBbObFwNdA8fGHTIZUxv2eiXgoTj4Q+hTDM/rQSu9eBk8hQl?=
 =?us-ascii?Q?h6sa01zkWYJJy/e3X2GZ/jqE1h+x2C9i06NDsSRQGBECtL9e9bFB/cbvH4yH?=
 =?us-ascii?Q?L37LRaEEvkYyTK9kPOqKZxhBTrQjTSpnPHtUm4bF5ZvD0RxxCt6CgTfOK6JA?=
 =?us-ascii?Q?I4roYO5m3Vfi3gIa7G1x2pr5FC9d2FB42PFrRwP+/WHK5qWHpQdeseAj+6Kz?=
 =?us-ascii?Q?GWIVaa8a2ZmTCXJvsE/R5/TtT+jWvOmKvo5I4szqwgjGZma2hF2UlHNSmlMH?=
 =?us-ascii?Q?OwWuGKhmVd10+UJsGYh4h4uBOGZMN8fV7kKUOyhsaxAb5VErDuWmY2Z/UKeC?=
 =?us-ascii?Q?rrZIFSx6ZhUOT19qKAj43BLB?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: abe42623-15f7-4504-a852-08d97e3660a7
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Sep 2021 02:03:46.7188
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ZnpdB5RneAAXISjBYj1s9zIEyQ7ySYL5Be399I9ffDO4e6LzZY25NMbpvFXGT8KYTMqFpC6+aybjOjR9bJfVYKtr+sUifsyFR6e9Eo+FXJg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1934
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Ocelot ports would still forward out ethernet broadcasts when they were
in the LEARNING or BLOCKING state. This is due to the
ocelot_get_bridge_fwd_mask, which would tell disabled ports to forward
packets out all FORWARDING ports. Broadcast storms would insue.

This patch restores the functionality of disabling forwarding for ports 
that aren't in the FORWARDING state. No more broadcast storms.

Tested and verified on an in-development driver, and Vladimir has done
independent testing and verification on supported hardware.


Vladimir Oltean (1):
  net: mscc: ocelot: fix forwarding from BLOCKING ports remaining
    enabled

 drivers/net/ethernet/mscc/ocelot.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

-- 
2.25.1