> >-----Original Message-----
> >From: Stephen Hemminger <step...@networkplumber.org>
> >Sent: Wednesday, November 29, 2023 10:56 PM
> >To: dev@dpdk.org
> >Cc: Stephen Hemminger <step...@networkplumber.org>; Hemant
> Agrawal
> ><hemant.agra...@nxp.com>; Sachin Saxena <sachin.sax...@nxp.com>;
> Nithin
> >Kumar Dabilpuram <ndabilpu...@marvell.com>; Kiran Kumar Kokkilagadda
> ><kirankum...@marvell.com>; Sunil Kumar Kori <sk...@marvell.com>;
> Satha
> >Koteswara Rao Kottidi <skotesh...@marvell.com>; Andrew Rybchenko
> ><andrew.rybche...@oktetlabs.ru>; Bruce Richardson
> ><bruce.richard...@intel.com>; Kevin Laatz <kevin.la...@intel.com>; Chas
> >Williams <ch...@att.com>; Min Hu (Connor) <humi...@huawei.com>;
> Liron
> >Himi <lir...@marvell.com>; Chaoyong He <chaoyong...@corigine.com>;
> >Devendra Singh Rawat <dsinghra...@marvell.com>; Alok Prasad
> ><pa...@marvell.com>
> >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 <step...@networkplumber.org>
> >Acked-by: Hemant Agrawal <hemant.agra...@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(-)
> >
> >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>
> >
> >-/* Sanity check */
> >+/* Byte order check */
> > #if (__BYTE_ORDER__ != __ORDER_BIG_ENDIAN__) && \
> >     (__BYTE_ORDER__ != __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 = cursor->block;
> >     memcpy(&partition.tlv_cursor, cursor, sizeof (*cursor));
> >     header = (struct tlv_partition_header *)partition.data;
> >-    /* Sanity check. */
> >+    /* Header check */
> >     if (__LE_TO_CPU_32(header->tag) != TLV_TAG_PARTITION_HEADER)
> {
> >             rc = 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)) != 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 = &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 != NULL) {
> >-            /* Basic sanity check */
> >+            /* Basic input check */
> >             if (conf->slow_periodic_ms == 0 ||
> >                             conf->fast_periodic_ms >= conf-
> >>slow_periodic_ms ||
> >                             conf->long_timeout_ms == 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 != 2)
> >                             return -3;
> >
> >-                    /* Range and sanity checks. */
> >+                    /* Range and input checks. */
> >                     if (get_val_securely(rng_tokens[0], &token_val) < 0)
> >                             return -4;
> >                     beg = (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 = nfp_flow_item_check(item, proc);
> >             if (ret != 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 = rxq->tpa_info[cqe->tpa_agg_index].tpa_head;
> >-    /* TODO:  Add Sanity Checks */
> >+    /* TODO:  Add more checks */
> >     rx_mb->nb_segs = cqe->num_of_bds;
> >     rx_mb->pkt_len = 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 = 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 =
> >--
> >2.42.0
> 
> Acked for qede
> Acked-by: Devendra Singh Rawat <dsinghra...@marvell.com>

Acked for fslmc and dpaa
Acked-by: Hemant Agrawal <hemant.agra...@nxp.com>

Reply via email to