Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   <Daniel.Machon@microchip.com>
To:     <kuba@kernel.org>
CC:     <petrm@nvidia.com>, <netdev@vger.kernel.org>,
        <davem@davemloft.net>, <maxime.chevallier@bootlin.com>,
        <thomas.petazzoni@bootlin.com>, <edumazet@google.com>,
        <pabeni@redhat.com>, <Lars.Povlsen@microchip.com>,
        <Steen.Hegelund@microchip.com>, <UNGLinuxDriver@microchip.com>,
        <joe@perches.com>, <linux@armlinux.org.uk>,
        <Horatiu.Vultur@microchip.com>, <Julia.Lawall@inria.fr>,
        <vladimir.oltean@nxp.com>, <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH net-next v2 1/6] net: dcb: add new pcp selector to app
 object
Thread-Topic: [PATCH net-next v2 1/6] net: dcb: add new pcp selector to app
 object
Thread-Index: AQHY1DNRRCKA3W+lZk2MiTdViI2XRa335cAAgADSrQCAA5l7gIAAjykAgABgQQA=
Date:   Mon, 3 Oct 2022 21:59:49 +0000
Message-ID: <YztdsF6b6SM9E5rw@DEN-LT-70577>
References: <20220929185207.2183473-1-daniel.machon@microchip.com>
 <20220929185207.2183473-2-daniel.machon@microchip.com>
 <87leq1uiyc.fsf@nvidia.com> <20220930175452.1937dadd@kernel.org>
 <87pmf9xrrd.fsf@nvidia.com> <20221003092522.6aaa6d55@kernel.org>
In-Reply-To: <20221003092522.6aaa6d55@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?vFwFvVK8ZbDZzqid5W9a1dc/4zGa3KBO904Oj2BmPmlSPvqoGbfcVHOUcPv4?=
 =?us-ascii?Q?WX2I8YirMfD7eELOQ5wT1b1zsXRXPkxktQzny0ocrQ+OuUF/GHDPjhX/XbvX?=
 =?us-ascii?Q?osvz7+fEO1Rsf/iV6L7dgW+pDVavSHzcu7oSSHe6vl49MOgaQvpOWjtAz3Ts?=
 =?us-ascii?Q?cF5cU1c+eqz8Hwe8SttMgKNOJ9w9ktHe/1Iz/O8B6PtDcqE1QxO74hLYo6Tv?=
 =?us-ascii?Q?p/Kik/zduGWiRHJfpWPtwgZpNQ3QDQJwUh4PrRxsXZDPRSrZimQ5lK2JYBy7?=
 =?us-ascii?Q?lxIR0z3z0MEnoQWShTkOtUICNpcO9KTJ120KOgt8MSaSI+I5V7YCSBU6uPSS?=
 =?us-ascii?Q?2yFdZiFCP/dsOPTI5hG0k+qBkrMaQwB75j3nf4cY4nRNMfFizvpZKSXInP+K?=
 =?us-ascii?Q?d2gHnyBtHceuHuW+xCt0+ARcEZBWAJp6gitM+dDoic5hGqnlyzaQr09yJdW7?=
 =?us-ascii?Q?ef07Mjk87apd2778OBoP7Z3xyRUVOIFLbQwUa2Ij6OqaTkIvSc8jdm+oz1VK?=
 =?us-ascii?Q?32hhOTEd5MvVxtNZOBPcLq6inkOFCFyMo97ekl5yX2wJNA630ME/0WJW5dTB?=
 =?us-ascii?Q?SODdHT3DIVhODnAoLBgoYBJhUFnNomLDESEu30HUX3FEkFDYdk54VCm9UJKZ?=
 =?us-ascii?Q?AYZJ/yeSMMezEH3FOFJ05sjAk+c8RJ9nPpvnNvlaXD10woKCk6VU3IIpKEow?=
 =?us-ascii?Q?gA1Ha3t0eFKSoWvolwVJQXplNN+FgLmX3Clag8orN8MRf0xqQ2vYNkFryFex?=
 =?us-ascii?Q?clpYxdTy6zQMVa+i3RgBqg+u00iTkKjPfmnnoCsCKvBoGnf5smom1MUorOya?=
 =?us-ascii?Q?8DGxYVFc/swVU0LFFOrZ2xbKBJjKzqdMaJFm5cQoR2cKym+dvbSTXJBtWlbN?=
 =?us-ascii?Q?kckgnZXOyQ6khjk5xgrfJrt3s5WZUAbFdt0kHd7rbNlUgO4dCkT72z43Hyd2?=
 =?us-ascii?Q?fBn+KcwKtN+hk2/cLqLgRmvgM8Zl3H/bfK04552I4YeWuZ+9F+6nBlJ6fZTl?=
 =?us-ascii?Q?nyu6TwRwneo9nbq5WDTD3ipUgtL6+iSg+8cBi0dvRjWwiwksgxeTnWFx4PLc?=
 =?us-ascii?Q?PjDZHNDxf7x7MDpP9TDMdjsQdjzun9DEcF1VtZ7ny1R93wTgYc8PgbCh3kyE?=
 =?us-ascii?Q?HK26dltm3K+DgILpIWyCLIvW52CUkbJCOyyvLWdKaHyt0HUF4RtKImG5MTfJ?=
 =?us-ascii?Q?AFr5Yo4BYHj9Wko9Wj0A1coo7eIqrJVpeeyLGRMLLjNjgKJorfIZl4BYBtE7?=
 =?us-ascii?Q?/cjSRH6mDI2KZF/X6yWqYczOwxxcUy4CGVHUQ1fD3CHj06tnV96ax4/38kqO?=
 =?us-ascii?Q?MQnU+6S+OxlOOkUrB8hHVVB83Z3oSwICOPaoksofgP3or4a86lVoZ24kQHY4?=
 =?us-ascii?Q?M8/RsaMSTy3xKgIh1ZL2/MfrcSXAaC6LtkFsLibqbGUQZcF1cHLCo/GZJI2s?=
 =?us-ascii?Q?Y0eKT+Y/brzf4/GCxSN51kfwwwlZQODXMJYgnEe723y17MoNzuLtpq320TbH?=
 =?us-ascii?Q?apgI2ADW+Zl7GO9HaYSOaoQkobpM7J8RpHaHYIpcnbn6LaCwVL4V+x+QzUzV?=
 =?us-ascii?Q?TcwgCGXgP56yP3li9MyahRV28jW8WLNyrvusHGjdRhSffIw41xFdXdaEaLaU?=
 =?us-ascii?Q?4DYiHUo9QefmcgaEUIh7xPE=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <5B6EE76AFA541948AA9FFCBD6FF76C87@namprd11.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5580.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6dd61d64-06d1-4cc9-248c-08daa58a97d0
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Oct 2022 21:59:49.6791
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: BWik15/aQiJb4IvAb1X0bQ5H1KdVkc5Ot64czVqYQ6o/aPE/rxTJNyZCN3vh+NjCe/NXOpqWdfJ3euMt7ZiPCAmIQ5Prm9mrm2iSgDN/Bwo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6356
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,
        SPF_PASS autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> On Mon, 3 Oct 2022 09:52:59 +0200 Petr Machata wrote:
> > I assumed the policy is much more strict with changes like this. If you
> > think it's OK, I'm fine with it as well.
> >
> > The userspace (lldpad in particular) is doing the opposite thing BTW:
> > assuming everything in the nest is a DCB_ATTR_IEEE_APP. When we start
> > emitting the new attribute, it will get confused.
>=20
> Can you add an attribute or a flag to the request which would turn
> emitting the new attrs on?

As for this sparx5 impl. the new attrs or any other app attr, will not be
emitted by lldpad, since APP tx cannot be enabled when set/get_dcbx is not=
=20
supported. IIUIC.

If lldpad was idd able to emit the new pcp app entries, they would be
emitted as invalid TLV's (assuming 255 or 24 selector value), because the
selector would be either zero or seven, which is currently not used for
any selector by the std. We then have time to patch lldpad to do whatever
with the new attr. Wouldn't this be acceptable?

As for iproute2/dcb, the new attr will just get ignored with a warning.