linux-kernel - Re: [PATCH net-next v3] net: marvell: prestera: add firmware v4.0 support

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: <SJ0PR18MB4009D5BA67AF1FFED9C9F79BB2BF9@SJ0PR18MB4009.namprd18.prod.outlook.com>
Date:   Thu, 21 Oct 2021 07:40:59 +0000
From:   "Volodymyr Mytnyk [C]" <vmytnyk@...vell.com>
To:     Andrew Lunn <andrew@...n.ch>
CC:     "kuba@...nel.org" <kuba@...nel.org>,
        Mickey Rachamim <mickeyr@...vell.com>,
        Serhiy Pshyk <serhiy.pshyk@...ision.eu>,
        Taras Chornyi <taras.chornyi@...ision.eu>,
        "Vadym Kochan [C]" <vkochan@...vell.com>,
        Yevhen Orlov <yevhen.orlov@...ision.eu>,
        "Taras Chornyi [C]" <tchornyi@...vell.com>,
        "David S. Miller" <davem@...emloft.net>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net-next v3] net: marvell: prestera: add firmware v4.0
 support

Hi Andrew

> On Wed, Oct 20, 2021 at 12:32:28PM +0300, Volodymyr Mytnyk wrote:
> > From: Volodymyr Mytnyk <vmytnyk@...vell.com>
> > 
> > Add firmware (FW) version 4.0 support for Marvell Prestera driver.
> > 
> > Major changes have been made to new v4.0 FW ABI to add support of new 
> > features, introduce the stability of the FW ABI and ensure better 
> > forward compatibility for the future driver vesrions.
> > 
> > Current v4.0 FW feature set support does not expect any changes to 
> > ABI, as it was defined and tested through long period of time.
> > The ABI may be extended in case of new features, but it will not break 
> > the backward compatibility.
> 
> So if we decide to merge this, we will hold you to this statement. You can never again break the ABI. The driver will always support v4.0 firmware.
> 
> Are you really sure you have it correct this time?
> 
>     Andrew
> 

So, the v4.0 FW supports stable ABI for the following set of features:

  - L1 port management
  - lag, vlans
  - routing, ecmp
  - tc templates, chains, conntrack, policer
  - bridge port insolation
  - HW counters
  - storm control

Some of those features are not fully supported yet by the driver. So, while we support these in the driver, the ABI is not expected to be changed.
Of course, with new features to come, the FW will be extended, while the backward compatibility will remain stable with the previous FW version. 
Bottom line, for this set of features, the ABI is expected to remain as is.

  Volodymyr