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 2A75E4340D; Thu, 30 Nov 2023 06:37:56 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C000240297; Thu, 30 Nov 2023 06:37:55 +0100 (CET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2062.outbound.protection.outlook.com [40.107.22.62]) by mails.dpdk.org (Postfix) with ESMTP id 094C0400D6 for ; Thu, 30 Nov 2023 06:37:55 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dXTqKsnojp/eP1ZuFDAOpeZcYTkEtaI9w+vJVtr9fhMIXC4stuFrI4UJujkSNzYjrwaKG/IQ9ygCQc0VjDPjzOlZuZIAN7oPr9IZvfM1s8c28KEk/PEtbSRAYGaH+bgYg+acZVDBy+baCbuKLF+AC23050Ug8oCRQ2f9v45kB7gqkV6rtCT9NQrkZWKqmqCXBPTACnW93UPDVI1XdI+tXHLH/kclSxTVxS/9VCgr8MJOkiUtk9h1qMTyo1l1SC3NpzI/eKcetHW04rYzLj69NxbPyQc8+KWKjbGD0nn8u3mMYlL7AkTJbb+UJg+V/AQsvTmprJo2oVxR1W+8hpwQqw== 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=+emDfkuEkLyxeMoKZ8nuPFP3JT7yE5FkkYGSRJd/x8U=; b=k6tVaLNg3rEjsS7cy1XY3lbN17GSp/BTdkM6stYb/KrNKjw7B++3pw8bePkDdWrdpbT93pareDD9UcxGjm1x3bskdAvFlUwtw3wF2ZT0bqPVJvfXuWVsYuhCMdAfmxmQZn+zJwgGhBZ5KV0E3jcGRGiI9XiPNwkU/Hf0DLC13/CYXLGxSGK7LExNUf405fF8MDSVlHNzXFCrEuSQmasJIthZ6IkPw2FC0l2u2FUMxY5Y7mPmo2aOd1dSe8sLFx0Lyuc660wT3zFUVjTX1U57cO17MSvSSbukwNZDtmxWSxz8zZWba5YAUD//Qcg73rSDImRZk4d11Ao0SJj/IBboqg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+emDfkuEkLyxeMoKZ8nuPFP3JT7yE5FkkYGSRJd/x8U=; b=Z0CqdS/dqijH9MW/gBJExV2cwGrnWX0SmDx+hOCvC2qCd4kp95CH/dgnkIwkfq9iq8q3HKUwJvVskW+TG68oITaFcwnT/KBoY7oYT4BostDv77qFelBtkm5Mvh8wSEY9JRV0r6PTGwywSaWpqkiguXVhwUbKKiTynLyg2d+wh8U= Received: from PAXPR04MB9328.eurprd04.prod.outlook.com (2603:10a6:102:2b6::15) by VI1PR04MB9929.eurprd04.prod.outlook.com (2603:10a6:800:1dc::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7046.19; Thu, 30 Nov 2023 05:37:51 +0000 Received: from PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::99b2:221d:bdda:3d6f]) by PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::99b2:221d:bdda:3d6f%5]) with mapi id 15.20.7046.015; Thu, 30 Nov 2023 05:37:51 +0000 From: Hemant Agrawal To: Devendra Singh Rawat , Stephen Hemminger , "dev@dpdk.org" CC: Sachin Saxena , Nithin Kumar Dabilpuram , Kiran Kumar Kokkilagadda , Sunil Kumar Kori , Satha Koteswara Rao Kottidi , Andrew Rybchenko , Bruce Richardson , Kevin Laatz , Chas Williams , "Min Hu (Connor)" , Liron Himi , Chaoyong He , Alok Prasad Subject: RE: [EXT] [PATCH v5 19/19] drivers: remove use of term sanity check Thread-Topic: [EXT] [PATCH v5 19/19] drivers: remove use of term sanity check Thread-Index: AQHaIun1VuoXKVLOh0uKNDJpxndPD7CSH/KAgAA4uOA= Date: Thu, 30 Nov 2023 05:37:51 +0000 Message-ID: References: <20230802232550.125487-1-stephen@networkplumber.org> <20231129173122.126895-1-stephen@networkplumber.org> <20231129173122.126895-20-stephen@networkplumber.org> In-Reply-To: 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=nxp.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PAXPR04MB9328:EE_|VI1PR04MB9929:EE_ x-ms-office365-filtering-correlation-id: b599b852-615d-4bfe-4dc1-08dbf1667e8b x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 4GYA4Fj8jbkEg/1BWnaqzuh8hgUGaf/8vkdz6mbPwY8frWYMHhNpwCoUX3gwQ6ojjYP5VjzESM29YFBsQf9ZKVi27B1gCaKU7SbJ1FrDuNVCvRXIZw7eo21Dj086C1sy/SPmZT6jqB9zePdLyRla86vNnmEK7Y2QnFI7uen/5E6dEaRTUHPl+uGttsI5qRKKZHL0FFbGB6YgotmeIvxyJ+tOv7B6HAyLBFo28iZnbPCqFChXiX3sUYJQ1ycdYmhdFzRMONu2KilKEKfxsxiOnZja1cw0lCt84nTDuqSFFNvt4psqohodJ/6zzUQ2ykJqhSehhxpCTMc/M/MRJthq5tCOgMqUfkV2jaD4eTCl/27asLr/4LbS+PXZ6n2y3rlFNfLP2uqwr1nCttdE6sOjIXUqwOMFcP9fy5bgPxp8A0fZEW4azhhlb37L1lGlox+lTSSP0ikws/gVrTWvOItjdqde1x70dFiw8ZVcVCKZJP2/4XZg5ffSkYLCtsiXlpxbGuWeNhfJDV7cYH4V+CJ9gtj++XZZcpvuExvKA1IEJSo0XvicgkBezXHNNeYXYIk0wzV7YbbkaAryiAgD2Hi0sySyZ3s1I5cJqVzjtdfLcVEf45WvMFjuoJ0eCpmyYo+PitBP/5rhU5q1xzg59H9BqWBQLdxjBiFRyXiLlzuH2i2Me51YmCJ4pe3TLWdDfC1faPh4gc08H6JHzIfhb1bEIg== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PAXPR04MB9328.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(346002)(136003)(39860400002)(376002)(396003)(366004)(230922051799003)(230273577357003)(230173577357003)(1800799012)(64100799003)(186009)(451199024)(38100700002)(33656002)(41300700001)(38070700009)(55016003)(122000001)(83380400001)(7416002)(5660300002)(44832011)(86362001)(26005)(202311291699003)(9686003)(6506007)(55236004)(7696005)(52536014)(2906002)(8676002)(8936002)(71200400001)(4326008)(66946007)(110136005)(66556008)(54906003)(316002)(66476007)(66446008)(64756008)(478600001)(76116006); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?MuoON9S6dJubvYgvAAKKHEYVN9MfqA+EwVt75vzU+7RV1LHPO3zm15OC8h3J?= =?us-ascii?Q?+mSr2dyZrRPSVG6zZ/4YeNkiq4OCK6u1qs3RMG/bOMlgJJ/weWFum8xmQG+g?= =?us-ascii?Q?qYMR1IDFIe6b9yqy5Ur1dnT3x3P4wOK/IUlmCv5/0ohj7sHRP1KJY+RJI1hP?= =?us-ascii?Q?FVbHQb/vuckVcpmsO9v5VLONsYmRowRMDP4KleNdE3MKko5P8ndB1nNSIitw?= =?us-ascii?Q?dbw2NskP0o2dHzYvg78zqUsUpn8jScoEIaLQ35+2XeNsKJ1ulI5jmVXXXZ9r?= =?us-ascii?Q?c9zT6ndM/Rgx5BHgTigHZvYcYu6QkpiGhaC3LDk/M+7sXFAWMvHZbOwAKaGd?= =?us-ascii?Q?TnSbuIWs5oxKSqPppIEjg7AdJ20jSgSHdI28KanIQEqn+mwwESngyr9EWhYe?= =?us-ascii?Q?wHtv9/dzBvHnV7TC20TkyNt8xqdbQafx7Q2+PcDdKephtA5r4nri7sR1m1LS?= =?us-ascii?Q?mRGwB4QGQy0mkTILd/wYHZ+rXNH18LsuCe0RzXmhjhAoy4sE7ESKZKn6BQdJ?= =?us-ascii?Q?SBCtKrykzoDHqSiMbxdLyAvHNFg91lJ7uj/R6F3ejO/3fA6ZvNQGDP0kjvRh?= =?us-ascii?Q?ozYaBsxSYKHpl7t1b8ID82GMv9WoN03rh6iByYvv1mUw2dYKIVxUqbmfBeLg?= =?us-ascii?Q?11BXPocyVwpirwXOExdiwCRgfkIXi3Vh42SSaut6vrzf1OfY+ktnBiX3f/G2?= =?us-ascii?Q?+Sii4dQz+sBANvfJvxm1uFkKZhABsUSsmZV19xcldgqH8AOY1SBDBPqBcIes?= =?us-ascii?Q?Hru//kci4ijTiuUdrch+uILrh1y2bOJT+fJjqC30Spd0G1meCaXtuHvZHM0E?= =?us-ascii?Q?SMUsmaaUFtPPDR8YSSxbbXK/iOZ73mS5vGTVetnlVxe8WONlbPe0kpvhIFR2?= =?us-ascii?Q?kUnkGMQFXZCli/egoidTy6tKQYIDWIPZ/M33nd++PjAsvmoSlz0KVY+wosb/?= =?us-ascii?Q?l61YqPxshwqK81AoSYlVrnmZyUMp6vsbfNlOssrG7EGuYeDO2A2WhsHksiw4?= =?us-ascii?Q?HLXD6FlyktyzF2VMkTRubBqk6wddDepekzSeDC2wW8UGxMijd8mFcn5hhE/Z?= =?us-ascii?Q?7SmluH+dcA/mFC2WDpZosrxLL9wRqpp0P0l/6a+jy0myh84eflmXhBdbemHG?= =?us-ascii?Q?MajSFHQCQ1aTYKsyzKlyMAIvK83XtBuYaH6KBsVOk+KsximyeTwxcjq8tPgJ?= =?us-ascii?Q?2Lz67SJwfY5Mr5NwnYms9LxFr0fxDnxzAP0FclYJE/5uOknhGWcqsFYyVTKK?= =?us-ascii?Q?tVilSi+TqbLz00ZB+DivzmRhOhMD9isQ0crwmoHQajLfE0D32NbEtqjvJp3B?= =?us-ascii?Q?6SjJoo2isizcHtzS8bXHvaFcZui/PO8Slg7tQsvcWmoRsAcK7biWEqr0JC7T?= =?us-ascii?Q?/ZUqvM0KNRTJP/dLBJw1gP6+tQd4lSFmjV5hismayrzypPsmKd+taOw16H2U?= =?us-ascii?Q?nF838BhvF2EMC2h3yKAg389hoFBJmrI4WxNBCrQFU25FW/fCnuyiYqn8gLz3?= =?us-ascii?Q?qeEi5k0cd/PZ+8FsbHq+IZ7Bfs7zl3FwTchyY11zZ+IImP9zt2H3SrOS5+so?= =?us-ascii?Q?t/mdh4pvYb8MCfjw6MKxyb/Ik083fMooJOgy6J0U?= 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: PAXPR04MB9328.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: b599b852-615d-4bfe-4dc1-08dbf1667e8b X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Nov 2023 05:37:51.4732 (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: 6fUecHKI3UEXfB8WY0uakCj7mpV6n+V2X9rL6RP86zj3JIzWr2E+Ee+QIpgCkrtvaaLj4zfwDn0NWZwkJOmqOg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9929 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: Stephen Hemminger > >Sent: Wednesday, November 29, 2023 10:56 PM > >To: dev@dpdk.org > >Cc: Stephen Hemminger ; Hemant > Agrawal > >; Sachin Saxena ; > Nithin > >Kumar Dabilpuram ; Kiran Kumar Kokkilagadda > >; Sunil Kumar Kori ; > Satha > >Koteswara Rao Kottidi ; Andrew Rybchenko > >; Bruce Richardson > >; Kevin Laatz ; Chas > >Williams ; Min Hu (Connor) ; > Liron > >Himi ; Chaoyong He ; > >Devendra Singh Rawat ; Alok Prasad > > > >Subject: [EXT] [PATCH v5 19/19] drivers: remove use of term sanity > >check > > > >External Email > > > >---------------------------------------------------------------------- > >Replace non-inclusive language with better comments. > >Covers multiple drivers > > > >Signed-off-by: Stephen Hemminger > >Acked-by: Hemant Agrawal > >--- > > 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(-) > > > >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 > > #include > > > >-/* 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; > > > >- /* Sanity check */ > >+ /* parameter check */ > > if (!lf || !aura || !aura_handle) > > return NPA_ERR_PARAM; > > > >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; > > > > /* > > * 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( > > > > /* > > * 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; > > > >- /* 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; > > } > > > >- /* 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; > > > > 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; > > > >- /* 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 > >index > >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; > > } > > > >- /* 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; > > > >@@ -2211,7 +2211,7 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq, > >struct rte_mbuf **tx_pkts, > > > > #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) > > > > 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 >=20 > Acked for qede > Acked-by: Devendra Singh Rawat Acked for fslmc and dpaa Acked-by: Hemant Agrawal