From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
by inbox.dpdk.org (Postfix) with ESMTP id EFB7043409;
Thu, 30 Nov 2023 02:21:18 +0100 (CET)
Received: from mails.dpdk.org (localhost [127.0.0.1])
by mails.dpdk.org (Postfix) with ESMTP id 6752E402C8;
Thu, 30 Nov 2023 02:21:18 +0100 (CET)
Received: from NAM12-MW2-obe.outbound.protection.outlook.com
(mail-mw2nam12on2125.outbound.protection.outlook.com [40.107.244.125])
by mails.dpdk.org (Postfix) with ESMTP id 3CBFE40297
for <dev@dpdk.org>; Thu, 30 Nov 2023 02:21:16 +0100 (CET)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;
b=Wf0mvIiTIXVFl68bFHcaycu1/pJSIQ8VgFXvmU9DlH+8Ty67oplEKlUdfeX8wjGUVzEDOOdpJ6kC+QWnAFoTA3T0AXCMgm3Hjwwcd14/2y5vaBcrOh0ADpHEAZW1lMJIG0Gb7wUFflQ1jDNs+dz/dftDfLx4qQhAJnDpnhLmLlBEsnE3iUnWZiJdSZUkNYifKBDnaNV+shPEWYs45Kll6BlQFPDksuUqj7sAvxd7oMYJp+CREEoobelkC38fjmuO0z5mr4cAt/BhCtIggHhwFszEIm5L1rAGp9hDv2l9gYnqPKLs8rLixLo2a2ZMRrkzf+jH4qS2yzBX9lUWFLDvKg==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;
s=arcselector9901;
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=BEbrW7qybrBU3ciPQa7XSJPJK20gV1YV+xzugIZoriI=;
b=Mm6WXQvXT/9tDAfwhHFF0PLKrox0YlZwJCTxC/8VnGxKK4k5D2UkThbXh+0Pf1XX7g+pZugpVbvAoSU6piH/InSn4yrwYrdYrQLCtknHLjllthZ+u+tixJb9xKu1qpms/QiAEmWvGNTCu2EUUsNHpljv7gm7bT/tpMsAudlEtM+T4ujTKzzeRtdh9y0panr4pbqTnlhlRSEUxCV5+kMiqtOtfrarwfJmoL6iD41Hfq9nUbbhroS8pcnoRsWNTqIdu5/GjqjnvIajIkFiuoICUSEkcBxTalh3JB1dxJGcdJdOCEELNQeF1rIYcN45AAA09cb4izipYY7IRtPesnwbUA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass
smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com;
dkim=pass header.d=corigine.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com;
h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
bh=BEbrW7qybrBU3ciPQa7XSJPJK20gV1YV+xzugIZoriI=;
b=iAqdjUZ2GFwZP2ZJfFPIIyTPU7kftQIqIsappa+hlZ7vcW41f+n8yoZtYPPcsc/AFEtewTEXECOT1Ki82gIWYmiDtOnZlf14zyd7FZ4j3GRyPWEf6tJwYu3ub6lWVylEqiB53Qly5ABef/3N/AnkzlhXdJ6mW76AkLUGCU8++iU=
Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5)
by BL3PR13MB5212.namprd13.prod.outlook.com (2603:10b6:208:343::8)
with Microsoft SMTP Server (version=TLS1_2,
cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7046.22; Thu, 30 Nov
2023 01:21:12 +0000
Received: from SJ0PR13MB5545.namprd13.prod.outlook.com
([fe80::45b3:d83c:cff2:4a1b]) by SJ0PR13MB5545.namprd13.prod.outlook.com
([fe80::45b3:d83c:cff2:4a1b%6]) with mapi id 15.20.7046.023; Thu, 30 Nov 2023
01:21:12 +0000
From: Chaoyong He <chaoyong.he@corigine.com>
To: Stephen Hemminger <stephen@networkplumber.org>, "dev@dpdk.org"
<dev@dpdk.org>
CC: Hemant Agrawal <hemant.agrawal@nxp.com>, Sachin Saxena
<sachin.saxena@nxp.com>, Nithin Dabilpuram <ndabilpuram@marvell.com>, Kiran
Kumar K <kirankumark@marvell.com>, Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>, Andrew Rybchenko
<andrew.rybchenko@oktetlabs.ru>, Bruce Richardson
<bruce.richardson@intel.com>, Kevin Laatz <kevin.laatz@intel.com>, Chas
Williams <chas3@att.com>, "Min Hu (Connor)" <humin29@huawei.com>, Liron Himi
<lironh@marvell.com>, Devendra Singh Rawat <dsinghrawat@marvell.com>, Alok
Prasad <palok@marvell.com>
Subject: RE: [PATCH v5 19/19] drivers: remove use of term sanity check
Thread-Topic: [PATCH v5 19/19] drivers: remove use of term sanity check
Thread-Index: AQHaIun1T0TjfRqAS0aa/zl+ONSDarCSEPcg
Date: Thu, 30 Nov 2023 01:21:12 +0000
Message-ID: <SJ0PR13MB5545CCF76726DA72699256929E82A@SJ0PR13MB5545.namprd13.prod.outlook.com>
References: <20230802232550.125487-1-stephen@networkplumber.org>
<20231129173122.126895-1-stephen@networkplumber.org>
<20231129173122.126895-20-stephen@networkplumber.org>
In-Reply-To: <20231129173122.126895-20-stephen@networkplumber.org>
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=corigine.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: SJ0PR13MB5545:EE_|BL3PR13MB5212:EE_
x-ms-office365-filtering-correlation-id: 17c64fd1-1412-494e-0173-08dbf142a3f5
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: GYf7dZIcqaMRRodqjAkShC6c9qTng6/qGRhE8vSRgKGDGa1UY6eVNk9JXaWY9iWeiqD5bh8iqByPgNi3YnTmKpyp7UfcawXQLoZE8cf7nb03u3STBPJXl1tMjhd+H+BcQzCGB/ig52an9P0roDhxvL9Qe0RtJlD/HCT4C32n2wrpzmYkVyBlWrNV8KuUr8NJvuTNBNCJQmg579J7zzl8zV29pi1AFJec57720xAwH2W/0ilV+fn6dHF49kL6w11DFTQyUFYsiyEsaZftL96FdrjqTFOAy4X9eeOJeH0YILe/ZVm5ntRLoS2MhIdHTvClhwFGgQ9XP9/JRVBMa6YyUKMuoYfg6IVxcYXOkkiVUM8MlebCWig6RANb00QpSjFQHLLq2SXT3XmmoZLlTUdN6C5F6wqwLJeWFtSy2Yiok9yxCAoAZ5RPwFyMWHu6DjzYlVvE+LSIj1nVO2x5ihVZB6elWzhRwmN5C9mBxCwap3n04tKN9Cibsta3wD533MtQUDV0N/FMfa6iFzeFYK36HQAIAXhzbWNnNRjga7UZQHxone6D3YlW+JjOBPgFKz93VyGi8gUFfOMeVF6JZ6xwIbwJx6NZyHCYbtada9HoXnOuNVTkWGvy5J3Pel1DVOFbd3RBlUOMLv1PdBevvPDnb1JAg/VYFvLb0ORHwBWQZnc4RrmfNpLbbAbwhN7whL8ugKsGJCaFLL/EIaL+PLei6g==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE;
SFS:(13230031)(346002)(376002)(396003)(366004)(136003)(39840400004)(230922051799003)(186009)(1800799012)(451199024)(64100799003)(64756008)(66476007)(66556008)(66446008)(66946007)(316002)(52536014)(8676002)(4326008)(76116006)(478600001)(54906003)(8936002)(44832011)(110136005)(41300700001)(5660300002)(7416002)(86362001)(2906002)(33656002)(122000001)(38100700002)(71200400001)(7696005)(6506007)(83380400001)(53546011)(202311291699003)(26005)(9686003)(38070700009)(55016003);
DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?TxYaQaAh+nPMd28QlTKx6lPcy2pDO0lNgEeIPkQDFaNu9o/rvgs9cG+ds2W5?=
=?us-ascii?Q?6e62HxA1DylO1iWfsItiC3sWbaKYMPS3MhPWVzcYXIAyp9f69tWbYDuO49TK?=
=?us-ascii?Q?mmZYgIIxkanxriGJ+U1remGFdZ/oy0gN3k/9aPQh9X1djDbytifbQSbK3xN5?=
=?us-ascii?Q?nRISW687+Af6mjca5jhptx2wt35dS8Vr0YJhtyGkmr1fLJ7+zb6HUybdvf6p?=
=?us-ascii?Q?pAmBa+cmLyKLYt2F2ish6ea2zSLFcDptorfKWKODqp3dcyfOr5uIZktC+5aB?=
=?us-ascii?Q?/6/2Z6+1kJEZ/F2tPnhR0K3kYbkInu0CtOf9MWwXzvO4At34nylp1yN1M1Z1?=
=?us-ascii?Q?P4/4j67AIw6RiaU5hHT6bY0gJwrGJrgLrFi2S1TmbWSbUfg6zgpXLqA50wPl?=
=?us-ascii?Q?d83ngHj/LX0ujgCLrmO2U1J7ZV2ywks2L6OGv7Hzl7k18XcggKwQi1aslg3K?=
=?us-ascii?Q?X3KAomi4u2eLZ8K3WGDK14n9VSWIdzoqnUGWGUkq06hA+irqO+ZLxnlGB+MC?=
=?us-ascii?Q?JPwpop2MBVkEZD9a/eYPYqi4l36sD3GLcm9o+8xW/SrVxVoV9C7Z8TOyA59h?=
=?us-ascii?Q?lAf7udaKei47oAHxtYkQXkm31hrdtG9/6jj7kuidLmTo5GHTB78FaIH4c1d6?=
=?us-ascii?Q?KP7+NjchPTiPO96W2xDuHD0uSkvqrL1X287ml0wYjIDOPKikTz0rtpQkMPjq?=
=?us-ascii?Q?AhpBddzBPX96wLQhX1xU6NmsSaLy7AS1yZ0gBdWLC9+DRl8A4D56E4ESNdZn?=
=?us-ascii?Q?nHDC++TU6GQz3NvQZ2rWm83SWQyofxULlSJkkcQpAGdfFi3vJRpaDdRvSBPC?=
=?us-ascii?Q?hNnPmhHK87VeM3iFGwfEgRhjZxdCmj2gBg+B3rBLHE3iNFNBfaFyglz8DUg0?=
=?us-ascii?Q?MchLdNGZMhEvhroszRepq2aCa4vxKA/RUL2c/+Uga0chWwkwGvqiOCmKV3qo?=
=?us-ascii?Q?rXJZfjejFGklgXk4DT/DMLg1JS5rs+GJDVUc9GcRD6UExxwJnCgdjs4u3Hkc?=
=?us-ascii?Q?wgawvqSZl4Nl131XJljJQUz2PhF28trA1HeXDoNy7A0VpwaieE6vl+ftJBji?=
=?us-ascii?Q?Yd/PdGpra5Xl/XMK+0Shw9rhXV1Yu9vCAAkC3J+jJ2PnzM7dih8BUicytGR9?=
=?us-ascii?Q?s6Bmmy3bghebE0DK0gqpn7JMISSzYhDnGMIVls/y8AJ6fkKCU8dwxGPVtWIz?=
=?us-ascii?Q?1rHLntVdyEidePwr3yFy0hHFtZtbh82BfkpIeGlCQvDGUliuOTWYQfRxZ2Fq?=
=?us-ascii?Q?3sfBqHu87TY3ZUT9rXmeM0rOQhiirHqXa3wRXCNofeI2fxcEI6fq65fqMv0J?=
=?us-ascii?Q?4rWovIfy83yVqpc3mb/3jex4IryIbTBNWB6Gu9HYiyCplTcNxCVzFC+psxsB?=
=?us-ascii?Q?pwvH6hJaF59oo3Qw2Lo2aUbROfhuRng7YrPZLvJIwBYiP6baNclukN6CmQqX?=
=?us-ascii?Q?zmN4YhKgFXo+Jc7l80td5uspx5Dnz7DV5ZNI/fdRii05Hgj2NOAyVNVW+XTl?=
=?us-ascii?Q?4P7E5KUZqgCm8trXuZpGHGZfVdSIEB26zcUfVi45q4QQfmf9yNf8YTXttTsr?=
=?us-ascii?Q?WwFizCOjJ0ouXCztJDjEaqRxx/dV+g3T5JncosZL?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 17c64fd1-1412-494e-0173-08dbf142a3f5
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Nov 2023 01:21:12.3801 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: fe128f2c-073b-4c20-818e-7246a585940c
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: HF3SozbiEeIjSkkBhO0DmmRC4SMxlNJckz/8f7S0Elbwxir5NcjYTeztRhV0vd6c0ECbmfJIXtqc6tfQGX6A78IWFne4NuNB8FFs4jZIiRk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL3PR13MB5212
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Looks good to me, thanks.
Acked-by: Chaoyong He <chaoyong.he@corigine.com>
> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: Thursday, November 30, 2023 1:26 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger <stephen@networkplumber.org>; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Sachin Saxena <sachin.saxena@nxp.com>;
> Nithin Dabilpuram <ndabilpuram@marvell.com>; Kiran Kumar K
> <kirankumark@marvell.com>; Sunil Kumar Kori <skori@marvell.com>; Satha
> Rao <skoteshwar@marvell.com>; Andrew Rybchenko
> <andrew.rybchenko@oktetlabs.ru>; Bruce Richardson
> <bruce.richardson@intel.com>; Kevin Laatz <kevin.laatz@intel.com>; Chas
> Williams <chas3@att.com>; Min Hu (Connor) <humin29@huawei.com>; Liron
> Himi <lironh@marvell.com>; Chaoyong He <chaoyong.he@corigine.com>;
> Devendra Singh Rawat <dsinghrawat@marvell.com>; Alok Prasad
> <palok@marvell.com>
> Subject: [PATCH v5 19/19] drivers: remove use of term sanity check
>=20
> Replace non-inclusive language with better comments.
> Covers multiple drivers
>=20
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
> drivers/bus/fslmc/qbman/qbman_sys_decl.h | 2 +-
> drivers/common/cnxk/roc_npa.c | 2 +-
> drivers/common/dpaax/caamflib/desc.h | 2 +-
> drivers/common/sfc_efx/base/ef10_nvram.c | 4 ++--
> drivers/common/sfc_efx/base/efx_rx.c | 2 +-
> drivers/dma/idxd/idxd_pci.c | 2 +-
> drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +-
> drivers/net/mvpp2/mrvl_qos.c | 2 +-
> drivers/net/nfp/nfp_flow.c | 2 +-
> drivers/net/qede/qede_rxtx.c | 6 +++---
> 10 files changed, 13 insertions(+), 13 deletions(-)
>=20
> diff --git a/drivers/bus/fslmc/qbman/qbman_sys_decl.h
> b/drivers/bus/fslmc/qbman/qbman_sys_decl.h
> index caaae417773c..2b6162d0ceaf 100644
> --- a/drivers/bus/fslmc/qbman/qbman_sys_decl.h
> +++ b/drivers/bus/fslmc/qbman/qbman_sys_decl.h
> @@ -9,7 +9,7 @@
> #include <compat.h>
> #include <fsl_qbman_base.h>
>=20
> -/* Sanity check */
> +/* Byte order check */
> #if (__BYTE_ORDER__ !=3D __ORDER_BIG_ENDIAN__) && \
> (__BYTE_ORDER__ !=3D __ORDER_LITTLE_ENDIAN__) #error "Unknown
> endianness!"
> diff --git a/drivers/common/cnxk/roc_npa.c
> b/drivers/common/cnxk/roc_npa.c index 5e38ecf0414e..bd8ab1b39584
> 100644
> --- a/drivers/common/cnxk/roc_npa.c
> +++ b/drivers/common/cnxk/roc_npa.c
> @@ -622,7 +622,7 @@ npa_aura_alloc(struct npa_lf *lf, const uint32_t
> block_count, int pool_id, {
> int rc, aura_id;
>=20
> - /* Sanity check */
> + /* parameter check */
> if (!lf || !aura || !aura_handle)
> return NPA_ERR_PARAM;
>=20
> diff --git a/drivers/common/dpaax/caamflib/desc.h
> b/drivers/common/dpaax/caamflib/desc.h
> index 4a1285c4d488..37c87eebd5b4 100644
> --- a/drivers/common/dpaax/caamflib/desc.h
> +++ b/drivers/common/dpaax/caamflib/desc.h
> @@ -107,7 +107,7 @@ extern enum rta_sec_era rta_sec_era;
>=20
> /*
> * ONE - should always be set. Combination of ONE (always
> - * set) and ZRO (always clear) forms an endianness sanity check
> + * set) and ZRO (always clear) forms an endianness check
> */
> #define HDR_ONE BIT(23)
> #define HDR_ZRO BIT(15)
> diff --git a/drivers/common/sfc_efx/base/ef10_nvram.c
> b/drivers/common/sfc_efx/base/ef10_nvram.c
> index ce8357fa94ce..fad9f0f507f3 100644
> --- a/drivers/common/sfc_efx/base/ef10_nvram.c
> +++ b/drivers/common/sfc_efx/base/ef10_nvram.c
> @@ -595,13 +595,13 @@ tlv_update_partition_len_and_cks(
>=20
> /*
> * We just modified the partition, so the total length may not be
> - * valid. Don't use tlv_find(), which performs some sanity checks
> + * valid. Don't use tlv_find(), which performs some checks
> * that may fail here.
> */
> partition.data =3D cursor->block;
> memcpy(&partition.tlv_cursor, cursor, sizeof (*cursor));
> header =3D (struct tlv_partition_header *)partition.data;
> - /* Sanity check. */
> + /* Header check */
> if (__LE_TO_CPU_32(header->tag) !=3D TLV_TAG_PARTITION_HEADER) {
> rc =3D EFAULT;
> goto fail1;
> diff --git a/drivers/common/sfc_efx/base/efx_rx.c
> b/drivers/common/sfc_efx/base/efx_rx.c
> index dce9ada55d28..28676c5a5b99 100644
> --- a/drivers/common/sfc_efx/base/efx_rx.c
> +++ b/drivers/common/sfc_efx/base/efx_rx.c
> @@ -930,7 +930,7 @@ efx_rx_qcreate_internal(
> ndescs, id, flags, eep, erp)) !=3D 0)
> goto fail4;
>=20
> - /* Sanity check queue creation result */
> + /* Check queue creation result */
> if (flags & EFX_RXQ_FLAG_RSS_HASH) {
> const efx_rx_prefix_layout_t *erplp =3D &erp->er_prefix_layout;
> const efx_rx_prefix_field_info_t *rss_hash_field; diff --git
> a/drivers/dma/idxd/idxd_pci.c b/drivers/dma/idxd/idxd_pci.c index
> a78889a7efa0..ba36a96f085d 100644
> --- a/drivers/dma/idxd/idxd_pci.c
> +++ b/drivers/dma/idxd/idxd_pci.c
> @@ -204,7 +204,7 @@ init_pci_device(struct rte_pci_device *dev, struct
> idxd_dmadev *idxd,
> goto err;
> }
>=20
> - /* sanity check device status */
> + /* check device status */
> if (pci->regs->gensts & GENSTS_DEV_STATE_MASK) {
> /* need function-level-reset (FLR) or is enabled */
> IDXD_PMD_ERR("Device status is not disabled, cannot init");
> diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c
> b/drivers/net/bonding/rte_eth_bond_8023ad.c
> index 79f1b3f1a0f2..57a67bd029ed 100644
> --- a/drivers/net/bonding/rte_eth_bond_8023ad.c
> +++ b/drivers/net/bonding/rte_eth_bond_8023ad.c
> @@ -1476,7 +1476,7 @@ bond_8023ad_setup_validate(uint16_t port_id,
> return -EINVAL;
>=20
> if (conf !=3D NULL) {
> - /* Basic sanity check */
> + /* Basic input check */
> if (conf->slow_periodic_ms =3D=3D 0 ||
> conf->fast_periodic_ms >=3D conf-
> >slow_periodic_ms ||
> conf->long_timeout_ms =3D=3D 0 ||
> diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mvpp2/mrvl_qos.c
> index f43217bc589c..db5052a9ab6f 100644
> --- a/drivers/net/mvpp2/mrvl_qos.c
> +++ b/drivers/net/mvpp2/mrvl_qos.c
> @@ -260,7 +260,7 @@ get_entry_values(const char *entry, uint8_t *tab,
> if (nb_rng_tokens !=3D 2)
> return -3;
>=20
> - /* Range and sanity checks. */
> + /* Range and input checks. */
> if (get_val_securely(rng_tokens[0], &token_val) < 0)
> return -4;
> beg =3D (char)token_val;
> diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c inde=
x
> f832b52d8996..c867697e05c9 100644
> --- a/drivers/net/nfp/nfp_flow.c
> +++ b/drivers/net/nfp/nfp_flow.c
> @@ -2071,7 +2071,7 @@ nfp_flow_compile_item_proc(struct
> nfp_flower_representor *repr,
> break;
> }
>=20
> - /* Perform basic sanity checks */
> + /* Perform basic checks */
> ret =3D nfp_flow_item_check(item, proc);
> if (ret !=3D 0) {
> PMD_DRV_LOG(ERR, "nfp flow item %d check failed",
> item->type); diff --git a/drivers/net/qede/qede_rxtx.c
> b/drivers/net/qede/qede_rxtx.c index c35585f5fdd6..8b89841e03b1
> 100644
> --- a/drivers/net/qede/qede_rxtx.c
> +++ b/drivers/net/qede/qede_rxtx.c
> @@ -1383,7 +1383,7 @@ qede_rx_process_tpa_end_cqe(struct qede_dev
> *qdev,
> cqe->len_list[0]);
> /* Update total length and frags based on end TPA */
> rx_mb =3D rxq->tpa_info[cqe->tpa_agg_index].tpa_head;
> - /* TODO: Add Sanity Checks */
> + /* TODO: Add more checks */
> rx_mb->nb_segs =3D cqe->num_of_bds;
> rx_mb->pkt_len =3D cqe->total_packet_len;
>=20
> @@ -2211,7 +2211,7 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq,
> struct rte_mbuf **tx_pkts,
>=20
> #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
> static inline void
> -qede_mpls_tunn_tx_sanity_check(struct rte_mbuf *mbuf,
> +qede_mpls_tunn_tx_basic_check(struct rte_mbuf *mbuf,
> struct qede_tx_queue *txq)
> {
> if (((mbuf->outer_l2_len + mbuf->outer_l3_len) / 2) > 0xff) @@ -
> 2473,7 +2473,7 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts,
> uint16_t nb_pkts)
>=20
> RTE_MBUF_F_TX_TUNNEL_MPLSINUDP) {
> mplsoudp_flg =3D true;
> #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
> - qede_mpls_tunn_tx_sanity_check(mbuf, txq);
> + qede_mpls_tunn_tx_basic_check(mbuf, txq);
> #endif
> /* Outer L4 offset in two byte words */
> tunn_l4_hdr_start_offset =3D
> --
> 2.42.0