From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A29F0C6FA8A for ; Sun, 11 Sep 2022 09:24:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229862AbiIKJYB (ORCPT ); Sun, 11 Sep 2022 05:24:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47408 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229570AbiIKJYA (ORCPT ); Sun, 11 Sep 2022 05:24:00 -0400 Received: from mailout-taastrup.gigahost.dk (mailout-taastrup.gigahost.dk [46.183.139.199]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71C8F3E75A; Sun, 11 Sep 2022 02:23:58 -0700 (PDT) Received: from mailout.gigahost.dk (mailout.gigahost.dk [89.186.169.112]) by mailout-taastrup.gigahost.dk (Postfix) with ESMTP id 0ECF71883A1B; Sun, 11 Sep 2022 09:23:56 +0000 (UTC) Received: from smtp.gigahost.dk (smtp.gigahost.dk [89.186.169.109]) by mailout.gigahost.dk (Postfix) with ESMTP id EC96825032B7; Sun, 11 Sep 2022 09:23:55 +0000 (UTC) Received: by smtp.gigahost.dk (Postfix, from userid 1000) id E32589EC0002; Sun, 11 Sep 2022 09:23:55 +0000 (UTC) X-Screener-Id: 413d8c6ce5bf6eab4824d0abaab02863e8e3f662 MIME-Version: 1.0 Date: Sun, 11 Sep 2022 11:23:55 +0200 From: netdev@kapio-technology.com To: Vladimir Oltean Cc: Ido Schimmel , davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, Florian Fainelli , Andrew Lunn , Vivien Didelot , Eric Dumazet , Paolo Abeni , Kurt Kanzenbach , Hauke Mehrtens , Woojung Huh , UNGLinuxDriver@microchip.com, Sean Wang , Landen Chao , DENG Qingfang , Matthias Brugger , Claudiu Manoil , Alexandre Belloni , Jiri Pirko , Ivan Vecera , Roopa Prabhu , Nikolay Aleksandrov , Shuah Khan , Christian Marangi , Daniel Borkmann , Yuwei Wang , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, bridge@lists.linux-foundation.org, linux-kselftest@vger.kernel.org Subject: Re: [PATCH v5 net-next 6/6] selftests: forwarding: add test of MAC-Auth Bypass to locked port tests In-Reply-To: <20220911001346.qno33l47i6nvgiwy@skbuf> References: <69db7606896c77924c11a6c175c4b1a6@kapio-technology.com> <5cee059b65f6f7671e099150f9da79c1@kapio-technology.com> <8dfc9b525f084fa5ad55019f4418a35e@kapio-technology.com> <20220908112044.czjh3xkzb4r27ohq@skbuf> <152c0ceadefbd742331c340bec2f50c0@kapio-technology.com> <20220911001346.qno33l47i6nvgiwy@skbuf> User-Agent: Gigahost Webmail Message-ID: <15ee472a68beca4a151118179da5e663@kapio-technology.com> X-Sender: netdev@kapio-technology.com Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On 2022-09-11 02:13, Vladimir Oltean wrote: > On Fri, Sep 09, 2022 at 03:11:56PM +0200, netdev@kapio-technology.com > wrote: >> > > > On Wed, Sep 07, 2022 at 11:10:07PM +0200, netdev@kapio-technology.com wrote: >> > > > > I am at the blackhole driver implementation now, as I suppose that the >> > > > > iproute2 command should work with the mv88e6xxx driver when adding blackhole >> > > > > entries (with a added selftest)? >> > > > > I decided to add the blackhole feature as new ops for drivers with functions >> > > > > blackhole_fdb_add() and blackhole_fdb_del(). Do you agree with that approach? >> > > > >> > > > I assume you are talking about extending 'dsa_switch_ops'? >> > > >> > > Yes, that is the idea. >> > > >> > > > If so, it's up to the DSA maintainers to decide. >> > >> > What will be the usefulness of adding a blackhole FDB entry from user >> > space? >> >> With the software bridge it could be used to signal a untrusted host >> in >> connection with a locked port entry attempt. I don't see so much use >> other >> that test purposes with the driver though. > > Not a huge selling point, to be honest. Can't the blackhole flag remain > settable only in the device -> bridge direction, with user space just > reading it? That is possible, but it would of course not make sense to have selftests of the feature as that would not work unless there is a driver with this capability (now just mv88e6xxx).