From:   Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
To:     Jakub Kicinski <kuba@kernel.org>
CC:     "davem@davemloft.net" <davem@davemloft.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        "alexandre.belloni@bootlin.com" <alexandre.belloni@bootlin.com>,
        "f.fainelli@gmail.com" <f.fainelli@gmail.com>,
        "andrew@lunn.ch" <andrew@lunn.ch>,
        "vivien.didelot@gmail.com" <vivien.didelot@gmail.com>,
        Marouen Ghodhbane <marouen.ghodhbane@nxp.com>
Subject: RE: [EXT] Re: [PATCH net-next] net: dsa: tag_ocelot: use traffic
 class to map priority on injected header
Thread-Topic: [EXT] Re: [PATCH net-next] net: dsa: tag_ocelot: use traffic
 class to map priority on injected header
Thread-Index: AQHX9lieJJXBj9Ptw0OYJnrgFnlWoqw/GG+AgABhZpA=
Date:   Thu, 23 Dec 2021 04:27:29 +0000
Message-ID: <AM6PR04MB5782E1EDD9F0FAF06E7D691AF07E9@AM6PR04MB5782.eurprd04.prod.outlook.com>
References: <20211221110209.31309-1-xiaoliang.yang_1@nxp.com>
 <20211222142337.0325219b@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
In-Reply-To: <20211222142337.0325219b@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?ns2JyTatHyiTGMwPjTm2AzBbVrauqZ8B3ja756Oo3V3+V678XPOAvAKQRpEj?=
 =?us-ascii?Q?Uo/p4iuh8DjgUEi8CjUFooOGuIOPGssGf8Xvww3ZCY4auBA5KN9WzI4JsOVB?=
 =?us-ascii?Q?bIAzY6BNGSJiORzBSj1juISuwW+g4bk3qFvaf2v7PzY2o3vvlKmX6i5rMfn9?=
 =?us-ascii?Q?qWg2jwv5IoxXKYKEA37cmhAsRAm3gViKvi7bw02zl9Z0cTKJJ/ZgT2PbnwwZ?=
 =?us-ascii?Q?r+x7S/Qs6exwCBPUYvXJAYRq2smlDyNtk1HfgFEQWBJA87xtcZLg4fh849SS?=
 =?us-ascii?Q?Rgm0j9KLrhfVOxLCtfC4QbwnrtXnouNctNdX636g3XlB5ggfPpaLe4SSc460?=
 =?us-ascii?Q?595G0fyd/rs44Tn9vdyFjlqPeUeUmmYq5g6p938WT42ehhxe6YW48dpOBGpj?=
 =?us-ascii?Q?uHuMlm+4PUKCfIm7ifDlqnCcxccw18FM8eBQXXKbIinfDgwhFeTtEy/Z90VY?=
 =?us-ascii?Q?gomtzjx3x9hhUf98LPPanLeub/kKxqnnhjs+j0BYYTl+onlSZB6sBSgXJ0We?=
 =?us-ascii?Q?I0RUtilWdU8wad6o1nskGTs4quPQjSL7bpFD87sNk0gTWULy+XWcO1MPMciS?=
 =?us-ascii?Q?p8/2ZmCjaeYXi4mxSmUSnteDffqMNaaL8YseP4D9DesEEYJcZwNZMP8tugEq?=
 =?us-ascii?Q?3g2IyP1OWoTL/5b2823V5Q6/JNirD/d94VOjoFxZzkjpVgj3OJAOQ5eOlaI/?=
 =?us-ascii?Q?h7AIEGdqQepAj7xbdy94JKqDylLkRon1AqK/mkUIWB5/QA4aZcgJCYJleI+b?=
 =?us-ascii?Q?RZGY7VmYf0hxqAvcu0WnJfcoY87hYzXbKoZ196WeIHJ/OH8G3NaFXKfBdbav?=
 =?us-ascii?Q?DedChrm4TzcdipYGblENbtpZxOlcMH4I9p6e3wERyWn7RFbFp1FVm7gzP4Fu?=
 =?us-ascii?Q?0hzq7RSZ39YG3N8y7dVcy9N9Ny7nhKh2BiNKnlSYi7vNgaxZcVEnxFtQeNih?=
 =?us-ascii?Q?kmGF2KQLLryTaMuNFla/onaBc4cAaDFVf1RJbYQNGMxFiy5hY0aUo+7+wQ1l?=
 =?us-ascii?Q?JvnC4w9jvZdl4Lcn0/GlxPh7fknLAocEDCwOoAgEHalLuCeeGYuzO9vHP2kp?=
 =?us-ascii?Q?m1GrfA8ftTo/mlNYIf7yFAHOSMxEXMWNu3nqLBMWQjyeSywba2EEEa2rceFZ?=
 =?us-ascii?Q?FrZOmLQRAnbHB+xcgDJyPk3+u/ZV5Tfn80Ur/dM3R62LYg26ubgOM2Jl4uSX?=
 =?us-ascii?Q?LnvMLKvgKvM2uA3yIy+Iz85QRg7iDj+lHlsE+y0vFlfNzVWhphFVPhBgMKCs?=
 =?us-ascii?Q?Gg8zPK+IwjC8eh5R7ihlryJwlwEs98kGA/tur8ohLBP5ehBWpG68q+JcKJCL?=
 =?us-ascii?Q?iSupRsDsYYzawM9NRlsuEu3dTvDizK5z5dxZ41dI5yh3u2h8W0/ej7FvckSz?=
 =?us-ascii?Q?Sg71M5tDHHVNgV0zC55J77QChWrzMoD3OBoVhxsUcAx7dJkJo1Nmh4rigK9G?=
 =?us-ascii?Q?iLKh2PGqz7uADObdInMmCDq1qz1TeEMu38iziDx7XiqJG07XqsJWx+DpIZtf?=
 =?us-ascii?Q?jTrVn4+R11jD019nicSlrVOel+ye/9yqEPyAO7ygo7Zb8S8Tnso8emQ0zndX?=
 =?us-ascii?Q?TcK8hYDUPJpRLAhAihj2gVi14FuKEpuam0BekOaAv86mAg1gmGEKttbJeddV?=
 =?us-ascii?Q?y1EwwgYbUA+TUen5cknb029mKPJ1d0m7sm7TT8krSZOKMfYH+t0DldnwRa0B?=
 =?us-ascii?Q?GfnnZQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB5782.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0ada0cfd-b6bf-48fd-500d-08d9c5cc87e3
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Dec 2021 04:27:29.2707
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: PzQMfc6Lnva8v16G2HT52KEzVE031jCAXePi3R/qcG9nzwFxV4bHhaU2sxY6NJaMKlh/qLTvIwYqTvbHyKHyCoB70vUN+ODXjzfPcril5Yw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR0402MB3733
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On Wed, 23 Dec 2021 06:24 Jakub Kicinski wrote:
> > For Ocelot switches, the CPU injected frames have an injection header
> > where it can specify the QoS class of the packet and the DSA tag, now
> > it uses the SKB priority to set that. If a traffic class to priority
> > mapping is configured on the netdevice (with mqprio for example ...),
> > it won't be considered for CPU injected headers. This patch make the
> > QoS class aligned to the priority to traffic class mapping if it exists=
.
> >
> > Signed-off-by: Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
> > Signed-off-by: Marouen Ghodhbane <marouen.ghodhbane@nxp.com>
>=20
> Is this a fix? Looks like one.
Yes, It can be seen as a fix, I will add fix tag and resend to net, thanks.