From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 892F4459CC; Wed, 18 Sep 2024 15:58:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D847642E94; Wed, 18 Sep 2024 15:58:14 +0200 (CEST) Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2045.outbound.protection.outlook.com [40.107.94.45]) by mails.dpdk.org (Postfix) with ESMTP id 16C3D432B2; Wed, 18 Sep 2024 15:58:14 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=uqTRphrRBDRAeonS7OiC045Jof5gi8atUw9EjeS2eniMhEbGxlO/KvcsQx5188LKa0GFZSL3dNHbCa15wTKs0rEKZ1evz+11sSgHDeEc7CqilE7PszQ+Uy0NOTF5swYCB+qFu3MAX2X9JgPwTWfqPGqaOeZBCvqxNujz/RQgGi27ca1CFk9eFC2mtjU01Fd9zA9dMBaiV0F8XzRdj69ejA5DJeLBO9DLd5DHbO5ivZcXgaH9C+GPMAq8EEwSEbSdzYSDUHEt+yaOxfFUeKASNejop18njMiwQWf1ae+urGLyfDeQ0G/CfkBO42VXhghnVR1XZEJzXJ974tZPR/0xCA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=gN6MbwIyN6hWjm7hO0/MK632JWgy+43PhwZpu4lX/DQ=; b=m19tN1N0HKy7mCdCBGaOabm8N39QRNMRyurD9jNoUyk+CnNqYZvUIUP3aDA3hDKPZDDAAiXujGIcId1S4jJpl6YHQI0TE+3/haxkULQ4/uMQpaFONbxp8VNK9c9UxjXhxz4YrRbYL1IkCZXL42nxe4lfPYCAs1ZvEIBMoP6OHc7sM7z9HDFoTfLu3n8H8IFLwQzWcnQGGIXOupFRZEqAO6BwhMj38YhqD3OOCPCj25Z4WWQoFumfliNNBtRSCatKrCR7mEGRX/xmLW6HLozg7+PcKUQQtt3WDDxsl68sEk5mu+Bhx3nS/aK52JAl36K9lNsZb0o2OIdqIxK3VxD29A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=gN6MbwIyN6hWjm7hO0/MK632JWgy+43PhwZpu4lX/DQ=; b=uoAjB3NHixEsMRm0js+yVJjdEjbREJdUlGjwS4QIgMM0Kch29E1WNIDOgFgdlJJS38eBaLUaaz/AsZMENXbanmPxKEKnlFsLIDiEpZhOzEUjxWomG8grpnxGqQnucHzJuwktqHXWg3auuLEv1hgRBZbia5h6qUE5DLQRBchwYlXB26JzKRcfetLhDMMCfmIjtc7OHg2VPP7vBa4euOmK3kfAxTl0fOPZR1m+sbn61HG2TiTq973oFgcenVmyRPqv2gKNyqLOyJS3MFCm0/NS+nhi4UBLCR2ArBpUOzgJ6dF6tAHVqe8uPDTWp29bBYu5U4tXVaDWz0W/wOnle9Zzeg== Received: from CH3PR12MB8460.namprd12.prod.outlook.com (2603:10b6:610:156::15) by SA1PR12MB6919.namprd12.prod.outlook.com (2603:10b6:806:24e::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7962.24; Wed, 18 Sep 2024 13:58:11 +0000 Received: from CH3PR12MB8460.namprd12.prod.outlook.com ([fe80::2e9d:7265:3b84:59b5]) by CH3PR12MB8460.namprd12.prod.outlook.com ([fe80::2e9d:7265:3b84:59b5%4]) with mapi id 15.20.7962.022; Wed, 18 Sep 2024 13:58:11 +0000 From: Dariusz Sosnowski To: Slava Ovsiienko , "dev@dpdk.org" CC: Matan Azrad , Raslan Darawsheh , Ori Kam , "stable@dpdk.org" Subject: RE: [PATCH v2 7/9] net/mlx5: fix next protocol validation after flex item Thread-Topic: [PATCH v2 7/9] net/mlx5: fix next protocol validation after flex item Thread-Index: AQHbCdFHZ32k2nqMO0mFymbB20oOlbJdke7Q Date: Wed, 18 Sep 2024 13:58:11 +0000 Message-ID: References: <20240911160458.524732-1-viacheslavo@nvidia.com> <20240918134623.8441-1-viacheslavo@nvidia.com> <20240918134623.8441-8-viacheslavo@nvidia.com> In-Reply-To: <20240918134623.8441-8-viacheslavo@nvidia.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: CH3PR12MB8460:EE_|SA1PR12MB6919:EE_ x-ms-office365-filtering-correlation-id: 82b4b662-51f0-420b-2fd0-08dcd7e9eeba x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; ARA:13230040|376014|366016|1800799024|38070700018; x-microsoft-antispam-message-info: =?us-ascii?Q?jN2Pko53t/nr0xOCJX2QXMe9/cZpu2DYG5LyLs1QEWRSsZ8nLsi17vhr149U?= =?us-ascii?Q?6uNhtp3Wkxe/yHyc9JgYNtlEvQlZ/aFovW54ooO/06a9HfX5oePkwNUsG5K4?= =?us-ascii?Q?6/5JYeVt/HtGnOw4miuj1WWglXgkUcszfX/Nn49lLURcU47CibUjfm45DNFF?= =?us-ascii?Q?VW5AtaFABfwL0rmmaKtUe9Zr3QVC5FP3+Ye9fUsNnS2FGDKDP0ptHRDddaVc?= =?us-ascii?Q?mTEw/LNebRd/BRRCRfOL3OB+wk45oYHQui+mXj2yxgtQbv0gutQ08TS6BKr1?= =?us-ascii?Q?DYhzi8+RN18fx+nJy6YcZFjgyw3HQvGpwxNE2L7RI0I+/ujJR9wsvzZa12mp?= =?us-ascii?Q?Pq73/664clas4izLxJo2uNz/5zQqkRNkc053PrtfQfKOpuk4bigPTm/gI4Tp?= =?us-ascii?Q?EIjry/fKrQT+0zw1E5OoeepK61rpj2rbsA9FYN27Ie0z6u23KNx00RSRv0F1?= =?us-ascii?Q?m1+o9VcUXxAmblStgUFBzpsRMiZ85wh9m/cMHtwd/PieiWWOlamNlpN38nYU?= =?us-ascii?Q?qmidClj3NCnpInIeHKByV7jBqjGx1TBhyve8QOnumiXwoee+a4fyw5RKAnYV?= =?us-ascii?Q?+620Lo7S2PLIqUAVkENBXdBGX/qGS16eeAU7wf7eMIldpBUtcRngueKs5IYB?= =?us-ascii?Q?QWYfRE9Uc8qV4+IAUiH+0isKZtxxnjt7urY0unOXqkBv6Nw8ssYt3dILK07L?= =?us-ascii?Q?m8BHiDInMz6BsWOF1fwJkYlG0wxxyV9S7OObnLkC/yAJT93yqbbtrLuFBNWb?= =?us-ascii?Q?eKwNDqERtbNmzh1rAJtAH18ux00JfT2O/JlzTiR77Trnia5uFd4tiIhvtfWB?= =?us-ascii?Q?WrkZFk63bhqHYmWd/AUN64IJxuvDqNZiA16JJnjN4mW+rI6jX4wbAiwinjCI?= =?us-ascii?Q?Epho6mbOq5YoZn+DO9OARA7VTqJBrfobwoluOQPd2BX793SHQMz9J2yAR93N?= =?us-ascii?Q?qtpIf0BTngfZJ0H4znQJ+b50mHONlEGBPJcG1v4zfYVDkCdrWQ762GlCO+gh?= =?us-ascii?Q?u4JeHVz6u6KZ0Ulpy0Hdb+avPJRanRwJ5t7awwvoEl0lGyjnP1SI5xX9d9Gw?= =?us-ascii?Q?9eLzqs4C8/UPcOlOGP11XJtkch9CtgYm8xLQRy8jrCTw0o2g8YGyHeLq7jIV?= =?us-ascii?Q?Ixb4gzG2Mjsb1ek4roa3SY6E83cls06qldXOy9CQtrdnJbzU1b8orczJF5VU?= =?us-ascii?Q?jc8CVafFjV3uY7RwZy0GyulhqUIqA49Mi9uWNd18MPuciVKNxq68Hy3eTLJX?= =?us-ascii?Q?ThFf4T81ZWhDGxVmC3KxJWYc+28GrFSeWIkkbtwKv5nT0tisWY8pJuDW9cUV?= =?us-ascii?Q?qTAYNzHvqS5heagqPxMS/8AhfB5G9zhDyRzwOJ0Ic7Cf02liJ1YsZpb/EFNX?= =?us-ascii?Q?IBY8ekqdWkyGVnbUGDUDemvslGv+fX3EiLQmKVYqhJ1zOqOaWA=3D=3D?= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH3PR12MB8460.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(376014)(366016)(1800799024)(38070700018); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?0FP6sFr/6sv6MwM7UrMKyKg4pfqTPgu7TNBjzuZYaC0JZ18DyshEACR0az3g?= =?us-ascii?Q?VHDYlM81T4YuqMPOyeNznLIc1kluK/GWntVaHSafrXZp4aZsssM0G6ml/XOp?= =?us-ascii?Q?LlYfSv5ABO3b6YdSQtBtPMKrncRRh3CxdF1x8gbOu4KcZTXUQPQEqUA7FAkD?= =?us-ascii?Q?nzzE9bsABrc2VXDErHXb6qwLCBmyRnxj0onGykNCWSbeqy1RE6dn84orceZC?= =?us-ascii?Q?t7u+feIJB6B0dsT3ikEcf5lll1/8HwVRZ3y/JvQ6h8dTUfC+jIx9dE5HG7Ng?= =?us-ascii?Q?LvjKlApvTTN+bJKCtQfp8rE0zBCkPhZhDpNtmJMaMYcGSoWxR3kF0GnxTDwP?= =?us-ascii?Q?j4MT0OZn5ddft/npeoed7o7JDua/VwGDqCYnYMVdzFC916WoXUVPaF9WfCgi?= =?us-ascii?Q?/7h01K1T6G8VUtOJMgBaUb7xA2DdWmP8RpgR8OWd1gkioCjdyo4AzqOc4hR7?= =?us-ascii?Q?EZvxUy7HPJk7Qiv22cafzItvYoJZaFjBrg2fZftJapofioMzucpbCbp+CwlB?= =?us-ascii?Q?gpQQiMpd8JCdszyBdqTVYE8FrUYNLHtpxfjHHC53AWiijzW7undeZrAdbPnE?= =?us-ascii?Q?H7durMjA5n6jW0Q77C3LutasqAZZhWIyHHZ2RdkZsPrYRbGx7oENc38iaPnf?= =?us-ascii?Q?w8VM6pJAl8Ce0HAuNzCqIWnGejwpzPlb2zLdM8JwTd6XtZ7DyW80v0HFCYE9?= =?us-ascii?Q?fu8PjEUGRAC0SSq0ray/M7xDbnp2Zt3pJAK2HgctUzrEEhIrETGj9iqqsbjy?= =?us-ascii?Q?2Xh3xscfcFxuk+X1z2QuziFQ/Mk+XEz99VpLObDr0H+V2S5IgvLBWs4khvlm?= =?us-ascii?Q?c+OzDRJz23g79vAa/kiS+tJT3QPwTmAAvBKsYV4cROHZskBkdyfwFD24r52m?= =?us-ascii?Q?qXNHkfVKOYplqeTHqDOaMClYbIMlWxpek3weRuxQ2YmpTW8UlmozCFP4RbHb?= =?us-ascii?Q?yqVT0jxrx5AJvhE+yZZw19Fs54+F9w8GR1gbgm2tA8dHKXImXXmYGDLFdJ4v?= =?us-ascii?Q?3+EseQSGWf2XbkmWLD6qNf4RBajheTJdXs2/qSxwPIcl8c4GWATpVHNxl0cP?= =?us-ascii?Q?yn/B/DahzorF47IrjhQg5+HuETbsvc2oVPQkpbrbScaesYAj/fgcUe3Suf1Y?= =?us-ascii?Q?HgxA1mebiDV42TFFLFeX3L3nEY7W/oX8w9SJjG2h9cLHRVKpegod7Jul9OGo?= =?us-ascii?Q?0Dd2xrHEvZ6YzRgZ5sDTd7YtsdwGjJ0OJMH2YBK05iZWK21iTuwiWxn37ude?= =?us-ascii?Q?yv/w/YMMPQ/fWrtoc6/uWAGWEtM7/bzcny2J+3OKfCIEedgbtO+QXe1nl92i?= =?us-ascii?Q?ZYd4Djey2qtKkSoWoFpyKU/cgVA7rn4EFfZ86uPlrwCvnr0yazFfOUcZIeOK?= =?us-ascii?Q?CrD3I9S/xmXIYwvjnqf1kjXBh8dIz/UVMtKomilT1jpgz/dVbEikhRMs+QBd?= =?us-ascii?Q?3yNlnZX9C1NY+dpkZXqjgQEES9K+d+OvvNYVKdxuaMVCLwwZ79ZG4uDTHJIC?= =?us-ascii?Q?9e+5hWMci4gXaQFM6mBALisFHk1zGgCzBHeOQyGF/TLQn7+j7KxR2Nm5WlA3?= =?us-ascii?Q?fHPV1S7uHxJxTAhvaGVc32u3eEKmCLYf6KFVxLGz?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8460.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 82b4b662-51f0-420b-2fd0-08dcd7e9eeba X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Sep 2024 13:58:11.1881 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: tfyGLYrURS0XRMqevw10i6MfM7lgnSucClCWR+yR8EL+9gNi0ZZ3K0F2EWVLFYcJ9mC5RLv05dp/fV4UMRS9bA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB6919 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org > -----Original Message----- > From: Slava Ovsiienko > Sent: Wednesday, September 18, 2024 15:46 > To: dev@dpdk.org > Cc: Matan Azrad ; Raslan Darawsheh > ; Ori Kam ; Dariusz Sosnowski > ; stable@dpdk.org > Subject: [PATCH v2 7/9] net/mlx5: fix next protocol validation after flex= item >=20 > On the flow validation some items may check the preceding protocols. > In case of flex item the next protocol is opaque (or can be multiple > ones) we should set neutral value and allow successful validation, for ex= ample, > for the combination of flex and following ESP items. >=20 > Fixes: a23e9b6e3ee9 ("net/mlx5: handle flex item in flows") > Cc: stable@dpdk.org >=20 > Signed-off-by: Viacheslav Ovsiienko > --- > drivers/net/mlx5/mlx5_flow_dv.c | 2 ++ > 1 file changed, 2 insertions(+) >=20 > diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow= _dv.c > index a51d4dd1a4..b18bb430d7 100644 > --- a/drivers/net/mlx5/mlx5_flow_dv.c > +++ b/drivers/net/mlx5/mlx5_flow_dv.c > @@ -8196,6 +8196,8 @@ flow_dv_validate(struct rte_eth_dev *dev, const > struct rte_flow_attr *attr, > tunnel !=3D 0, > error); > if (ret < 0) > return ret; > + /* Reset for next proto, it is unknown. */ > + next_protocol =3D 0xff; > break; > case RTE_FLOW_ITEM_TYPE_METER_COLOR: > ret =3D flow_dv_validate_item_meter_color(dev, items, > -- > 2.34.1 Acked-by: Dariusz Sosnowski Resending the Ack for each patch separately, because patchwork assigned my = Ack for the series to v1, not v2. Best regards, Dariusz Sosnowski