Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/129859/?format=api
http://patches.dpdk.org/api/patches/129859/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20230802232550.125487-10-stephen@networkplumber.org/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230802232550.125487-10-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20230802232550.125487-10-stephen@networkplumber.org", "date": "2023-08-02T23:25:40", "name": "[v4,09/19] net/mlx[45]: remove word sanity", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "1c39accd91fa83093ee986d4d42f757c6158f456", "submitter": { "id": 27, "url": "http://patches.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 1, "url": "http://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20230802232550.125487-10-stephen@networkplumber.org/mbox/", "series": [ { "id": 29100, "url": "http://patches.dpdk.org/api/series/29100/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=29100", "date": "2023-08-02T23:25:31", "name": "replace use of term \"sanity check\"", "version": 4, "mbox": "http://patches.dpdk.org/series/29100/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/129859/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/129859/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id E16B742FBA;\n\tThu, 3 Aug 2023 01:27:04 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id A4E6143285;\n\tThu, 3 Aug 2023 01:26:14 +0200 (CEST)", "from mail-pl1-f171.google.com (mail-pl1-f171.google.com\n [209.85.214.171])\n by mails.dpdk.org (Postfix) with ESMTP id A0C6A4326D\n for <dev@dpdk.org>; Thu, 3 Aug 2023 01:26:10 +0200 (CEST)", "by mail-pl1-f171.google.com with SMTP id\n d9443c01a7336-1b8b2b60731so3138635ad.2\n for <dev@dpdk.org>; Wed, 02 Aug 2023 16:26:10 -0700 (PDT)", "from hermes.local (204-195-127-207.wavecable.com. [204.195.127.207])\n by smtp.gmail.com with ESMTPSA id\n jd20-20020a170903261400b001b8b0ac2258sm12960870plb.174.2023.08.02.16.26.08\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 02 Aug 2023 16:26:09 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1691018769;\n x=1691623569;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=rII1qsx1QO6fzlWmnbTLtfQ6mQocvAz9iA5y1p8DMOA=;\n b=Odb0+zbEjG1k5+Mv1g34uQcDTbgRIjOiJuceo2jvX87fdlfUjd4UI68tHCooWrBup6\n OKgv+BwDeTvW8qnojWMTHbfcfTZxuEqsD6vhDHoJK8JuoIfzGFwKr2QQIDRIG/D6pUNl\n d0kp6w3JQHFz8I5zT8MXctVpqi5ZF1o5lDako1lTJLTH6g+atQba1t3t0hR06VQNFEwu\n EG9YKzFnnAuB0psUs70hMC3FYgDrz1WtwMeMJo2qUvXrtuv+zJkBBGtU1TzGh1rUj3X5\n OcZSGBH6xUqOLkgKelcpR5aODnxhQIfrBERNqyny7sXeDNwji0BZVHIAkzfrR7pCTM11\n bVlw==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20221208; t=1691018769; x=1691623569;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=rII1qsx1QO6fzlWmnbTLtfQ6mQocvAz9iA5y1p8DMOA=;\n b=iUw4qhMGxaNgw2lBhVZS45y7YaR5iUfia92bSLXblXdbequcLM1TfK4AqoWu2llXUb\n PYmwakgcUsKnMcyaivDa88kLMSfgtiMJmFoYRKUo7xCfEbw/5hdLmKJtxJgck+JEztQO\n F294tXb0zC7H0K1wsIeIjHSOSOT6zie9cH5Tq1f0G/nbsdksSQx8m63dnOZyUlWKeSdB\n IoD67lluej4d7Gq9fS51CRcPGM5MV1iO0+8/vBUqU8SWwjr4mkm49V9uCWPYj58BFiFF\n Xi1v4pvFG8UEqlc3ywnNMbCImkbIC+AnWcEYhdVxzRVmV3W0XfySqlg7UZ56+pMYeuiF\n Ak4g==", "X-Gm-Message-State": "ABy/qLZfThSbAu8uFDUiubajFLt5+m0V+Gl3ASSI+C8u77FQCXjobF5o\n re+1GMfv9I+IPpDLS6P5u+0iX4NvqxTATKe/OrRS8w==", "X-Google-Smtp-Source": "\n APBJJlHJ0tcvV/ePigV9SGPgvEszOXTcNpv8n/cUlMNdXM30k18t7Y8wJ4w+Kq2Y/g0mgWZqvqApxg==", "X-Received": "by 2002:a17:902:e751:b0:1b0:f727:bc41 with SMTP id\n p17-20020a170902e75100b001b0f727bc41mr18927040plf.42.1691018769633;\n Wed, 02 Aug 2023 16:26:09 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Matan Azrad <matan@nvidia.com>,\n Viacheslav Ovsiienko <viacheslavo@nvidia.com>, Ori Kam <orika@nvidia.com>,\n Suanming Mou <suanmingm@nvidia.com>", "Subject": "[PATCH v4 09/19] net/mlx[45]: remove word sanity", "Date": "Wed, 2 Aug 2023 16:25:40 -0700", "Message-Id": "<20230802232550.125487-10-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230802232550.125487-1-stephen@networkplumber.org>", "References": "<20230517161603.117728-1-stephen@networkplumber.org>\n <20230802232550.125487-1-stephen@networkplumber.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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>,\n <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>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "The term \"sanity\" is on non-inclusive naming list.\nRemove it from the Nvidia drivers.\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n drivers/common/mlx5/linux/mlx5_common_os.c | 2 +-\n drivers/net/mlx4/mlx4.c | 2 +-\n drivers/net/mlx4/mlx4_flow.c | 6 +++---\n drivers/net/mlx5/mlx5_flow_dv.c | 2 +-\n drivers/net/mlx5/mlx5_flow_hw.c | 6 ++----\n drivers/net/mlx5/mlx5_rxq.c | 2 +-\n drivers/net/mlx5/mlx5_rxtx_vec.h | 2 +-\n 7 files changed, 10 insertions(+), 12 deletions(-)", "diff": "diff --git a/drivers/common/mlx5/linux/mlx5_common_os.c b/drivers/common/mlx5/linux/mlx5_common_os.c\nindex 2ebb8ac8b6e5..7a7aadffa602 100644\n--- a/drivers/common/mlx5/linux/mlx5_common_os.c\n+++ b/drivers/common/mlx5/linux/mlx5_common_os.c\n@@ -234,7 +234,7 @@ mlx5_get_ifname_sysfs(const char *ibdev_path, char *ifname)\n /**\n * Suffix RTE_EAL_PMD_PATH with \"-glue\".\n *\n- * This function performs a sanity check on RTE_EAL_PMD_PATH before\n+ * This function performs a check on RTE_EAL_PMD_PATH before\n * suffixing its last component.\n *\n * @param buf[out]\ndiff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c\nindex a54016f4a235..e3048d5b6ec0 100644\n--- a/drivers/net/mlx4/mlx4.c\n+++ b/drivers/net/mlx4/mlx4.c\n@@ -1206,7 +1206,7 @@ static struct rte_pci_driver mlx4_driver = {\n /**\n * Suffix RTE_EAL_PMD_PATH with \"-glue\".\n *\n- * This function performs a sanity check on RTE_EAL_PMD_PATH before\n+ * This function performs a check on RTE_EAL_PMD_PATH before\n * suffixing its last component.\n *\n * @param buf[out]\ndiff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c\nindex 8ef9fd2db44e..25b185d8710c 100644\n--- a/drivers/net/mlx4/mlx4_flow.c\n+++ b/drivers/net/mlx4/mlx4_flow.c\n@@ -514,7 +514,7 @@ mlx4_flow_merge_tcp(struct rte_flow *flow,\n }\n \n /**\n- * Perform basic sanity checks on a pattern item.\n+ * Perform basic checks on a pattern item.\n *\n * @param[in] item\n * Item specification.\n@@ -730,7 +730,7 @@ mlx4_flow_prepare(struct mlx4_priv *priv,\n \t\t\tgoto exit_item_not_supported;\n \t\tproc = next;\n \t\t/*\n-\t\t * Perform basic sanity checks only once, while handle is\n+\t\t * Perform basic checks only once, while handle is\n \t\t * not allocated.\n \t\t */\n \t\tif (flow == &temp) {\n@@ -799,7 +799,7 @@ mlx4_flow_prepare(struct mlx4_priv *priv,\n \t\t\t\trss_key = mlx4_rss_hash_key_default;\n \t\t\t\trss_key_len = MLX4_RSS_HASH_KEY_SIZE;\n \t\t\t}\n-\t\t\t/* Sanity checks. */\n+\t\t\t/* Input checks. */\n \t\t\tfor (i = 0; i < rss->queue_num; ++i)\n \t\t\t\tif (rss->queue[i] >=\n \t\t\t\t ETH_DEV(priv)->data->nb_rx_queues)\ndiff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c\nindex a8dd9920e6a4..7c21a9d6e91f 100644\n--- a/drivers/net/mlx5/mlx5_flow_dv.c\n+++ b/drivers/net/mlx5/mlx5_flow_dv.c\n@@ -8753,7 +8753,7 @@ flow_dv_prepare(struct rte_eth_dev *dev,\n \n #ifdef RTE_LIBRTE_MLX5_DEBUG\n /**\n- * Sanity check for match mask and value. Similar to check_valid_spec() in\n+ * Check for match mask and value. Similar to check_valid_spec() in\n * kernel driver. If unmasked bit is present in value, it returns failure.\n *\n * @param match_mask\ndiff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c\nindex 5395969eb07e..0e7baa7c1a6e 100644\n--- a/drivers/net/mlx5/mlx5_flow_hw.c\n+++ b/drivers/net/mlx5/mlx5_flow_hw.c\n@@ -6008,11 +6008,10 @@ flow_hw_tx_tag_regc_mask(struct rte_eth_dev *dev)\n \tstruct mlx5_priv *priv = dev->data->dev_private;\n \tuint32_t mask = priv->sh->dv_regc0_mask;\n \n-\t/* Mask is verified during device initialization. Sanity checking here. */\n+\t/* Mask is verified during device initialization. */\n \tMLX5_ASSERT(mask != 0);\n \t/*\n \t * Availability of sufficient number of bits in REG_C_0 is verified on initialization.\n-\t * Sanity checking here.\n \t */\n \tMLX5_ASSERT(__builtin_popcount(mask) >= __builtin_popcount(priv->vport_meta_mask));\n \treturn mask;\n@@ -6024,12 +6023,11 @@ flow_hw_tx_tag_regc_value(struct rte_eth_dev *dev)\n \tstruct mlx5_priv *priv = dev->data->dev_private;\n \tuint32_t tag;\n \n-\t/* Mask is verified during device initialization. Sanity checking here. */\n+\t/* Mask is verified during device initialization. */\n \tMLX5_ASSERT(priv->vport_meta_mask != 0);\n \ttag = priv->vport_meta_tag >> (rte_bsf32(priv->vport_meta_mask));\n \t/*\n \t * Availability of sufficient number of bits in REG_C_0 is verified on initialization.\n-\t * Sanity checking here.\n \t */\n \tMLX5_ASSERT((tag & priv->sh->dv_regc0_mask) == tag);\n \treturn tag;\ndiff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c\nindex 8ef7860e161e..d18a46dff212 100644\n--- a/drivers/net/mlx5/mlx5_rxq.c\n+++ b/drivers/net/mlx5/mlx5_rxq.c\n@@ -1442,7 +1442,7 @@ mlx5_mprq_alloc_mp(struct rte_eth_dev *dev)\n \tdesc *= 4;\n \tobj_num = desc + MLX5_MPRQ_MP_CACHE_SZ * n_ibv;\n \t/*\n-\t * rte_mempool_create_empty() has sanity check to refuse large cache\n+\t * rte_mempool_create_empty() has basic check to refuse large cache\n \t * size compared to the number of elements.\n \t * CACHE_FLUSHTHRESH_MULTIPLIER is defined in a C file, so using a\n \t * constant number 2 instead.\ndiff --git a/drivers/net/mlx5/mlx5_rxtx_vec.h b/drivers/net/mlx5/mlx5_rxtx_vec.h\nindex 77c3f4efa019..f8376f2b55b4 100644\n--- a/drivers/net/mlx5/mlx5_rxtx_vec.h\n+++ b/drivers/net/mlx5/mlx5_rxtx_vec.h\n@@ -21,7 +21,7 @@\n \t RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM)\n \n /*\n- * Compile time sanity check for vectorized functions.\n+ * Compile time checks for vectorized functions.\n */\n \n #define S_ASSERT_RTE_MBUF(s) \\\n", "prefixes": [ "v4", "09/19" ] }{ "id": 129859, "url": "