lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210106172621.GA1082418@shredder.lan>
Date:   Wed, 6 Jan 2021 19:26:21 +0200
From:   Ido Schimmel <idosch@...sch.org>
To:     Vladimir Oltean <olteanv@...il.com>
Cc:     "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>, netdev@...r.kernel.org,
        Florian Fainelli <f.fainelli@...il.com>,
        Andrew Lunn <andrew@...n.ch>,
        Vivien Didelot <vivien.didelot@...il.com>,
        Kurt Kanzenbach <kurt@...utronix.de>,
        Hauke Mehrtens <hauke@...ke-m.de>,
        Woojung Huh <woojung.huh@...rochip.com>,
        Microchip Linux Driver Support <UNGLinuxDriver@...rochip.com>,
        Sean Wang <sean.wang@...iatek.com>,
        Landen Chao <Landen.Chao@...iatek.com>,
        Claudiu Manoil <claudiu.manoil@....com>,
        Alexandre Belloni <alexandre.belloni@...tlin.com>,
        Linus Walleij <linus.walleij@...aro.org>,
        Vadym Kochan <vkochan@...vell.com>,
        Taras Chornyi <tchornyi@...vell.com>,
        Jiri Pirko <jiri@...dia.com>, Ido Schimmel <idosch@...dia.com>,
        Grygorii Strashko <grygorii.strashko@...com>,
        Ioana Ciornei <ioana.ciornei@....com>,
        Ivan Vecera <ivecera@...hat.com>
Subject: Re: [PATCH v2 net-next 01/10] net: switchdev: remove vid_begin ->
 vid_end range from VLAN objects

On Wed, Jan 06, 2021 at 07:15:59PM +0200, Vladimir Oltean wrote:
> On Wed, Jan 06, 2021 at 07:08:18PM +0200, Ido Schimmel wrote:
> > On Wed, Jan 06, 2021 at 03:09:57PM +0200, Vladimir Oltean wrote:
> > > Of the existing switchdev pieces of hardware, it appears that only
> > > Mellanox Spectrum supports offloading more than one VLAN at a time.
> > > I have kept that code internal to the driver, because there is some more
> > > bookkeeping that makes use of it, but I deleted it from the switchdev
> > > API. But since the switchdev support for ranges has already been de
> > > facto deleted by a Mellanox employee and nobody noticed for 4 years, I'm
> > > going to assume it's not a biggie.
> > 
> > Which code are you referring to?
> 
> mlxsw_sp_port_vlan_set

OK, we actually need this one. We are using it during driver
initialization to clear VLAN membership for all 4k VLANs in each port in
batch. For some reason, the hardware/firmware default is that ports are
member in all 4k VLANs, which is not in accordance with the kernel.

> 
> > For the switchdev and mlxsw parts:
> > 
> > Reviewed-by: Ido Schimmel <idosch@...dia.com>
> > 
> > I applied the series to our queue, so I should have regression results
> > tomorrow
> 
> Thanks. Could you wait for me to send a v3 though, with that small fixup
> in mv88e6xxx? I'm sure it will raise some red flags for your testing too.

I don't have mv88e6xxx enabled in my config, so I was able to compile
the kernel successfully. Anyway, I can always test more versions.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ