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 22CD7A0524 for ; Fri, 5 Feb 2021 12:39:20 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 141DC188A61; Fri, 5 Feb 2021 12:39:20 +0100 (CET) Received: from mail-wr1-f41.google.com (mail-wr1-f41.google.com [209.85.221.41]) by mails.dpdk.org (Postfix) with ESMTP id 3D6B2188A66 for ; Fri, 5 Feb 2021 12:39:18 +0100 (CET) Received: by mail-wr1-f41.google.com with SMTP id c4so7287712wru.9 for ; Fri, 05 Feb 2021 03:39:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=q6x8nfFOjugck6LEqOTtdw1SiWmOfG9NnC3WHLd6mDk=; b=tTjvYnkxTNf7UhNV7wCrSJb+9GI0UFfbyZnPhzn8O0UuOUJ8wnIleeCty9xe7Q3/Y5 h/WvcUeNDUgkKIPPj68dRcAZwrVrlT4JeE5YxrPxYBD/rLKzyddGA8iupX0y8UKpjXUe ZTl1f7HKGdQYLD4Ve3Fze/eEpHslEBqPZnFsL3c8IedFbDejrRyAOZBthkQozGwcUeZA W3TGITjjVZgSSU3egqlpGLGwnN8WqMBVM1SD4mbXfzQV/jLR2Q+5iJZMR6xpY6BYKp/i 3oEIHDTFle9DKf1OnnQewpQZLBoAbPdYgZKKGa2D72YlauSrmJSlXzzKmyQjLC3NRVeM smOg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=q6x8nfFOjugck6LEqOTtdw1SiWmOfG9NnC3WHLd6mDk=; b=ewjCe7CCTaeIQAonGStHrBKGNxhR7/Wflt7Fy6UUqt7JARCw+BVntnTrSIjB6U2TbV /BAAw6flOQNqR98d7bD5bEr1/77IUQxQURaFh8eGC8C2JCFx7seRmGng9ckL/ecrsM1W FzLJHwIxBJ491JAS/qbexFqx3dD7rNHojLkyIaIKHiR80iLrhLYy8G01BwTyO0kM0uBU 9tFmcYnaCXBd9Go3nOE7QjaeG+X3huLr+1rV83+STUevn3U4qpEKi3T6VqXux3t2h7zS jHmKu1D/0K9o8VE/xWcLnJMITa80KkzI7SSp6VGD61HjLtUcb1XdHj77UxTRDCP67Ptk XzmA== X-Gm-Message-State: AOAM533pqvmml4VMoQWWgQKuvkiqTK20w6/jxJIf0bwBCx7EVqlSaUHn AqytYW8K//54zrgXW1mHgPA= X-Google-Smtp-Source: ABdhPJwOQ3tVg5uLaU49T3yoafmb4eKMfAruRCezF8zsIALwsHLYpYGV5RSsOg4/iZYPZJwy/gDylQ== X-Received: by 2002:a05:6000:22a:: with SMTP id l10mr4707412wrz.131.1612525158043; Fri, 05 Feb 2021 03:39:18 -0800 (PST) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id z14sm11600116wrm.5.2021.02.05.03.39.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Feb 2021 03:39:17 -0800 (PST) From: luca.boccassi@gmail.com To: Lijun Ou Cc: dpdk stable Date: Fri, 5 Feb 2021 11:19:09 +0000 Message-Id: <20210205111920.1272063-263-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210205111920.1272063-1-luca.boccassi@gmail.com> References: <20210205111920.1272063-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'net/hns3: adjust some comments' has been queued to stable release 20.11.1 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to stable release 20.11.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 02/07/21. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/b6d4d8a21fc5456a5eaa71cd5de5805f43d62c97 Thanks. Luca Boccassi --- >From b6d4d8a21fc5456a5eaa71cd5de5805f43d62c97 Mon Sep 17 00:00:00 2001 From: Lijun Ou Date: Fri, 22 Jan 2021 18:18:49 +0800 Subject: [PATCH] net/hns3: adjust some comments [ upstream commit f77b3c3a7b4a968dd42935420df0c00eefb506da ] Fix some error comments and remove some meaningless comments. Fixes: f8e7fcbfd0b8 ("net/hns3: support flow action of queue region") Fixes: fcba820d9b9e ("net/hns3: support flow director") Fixes: c37ca66f2b27 ("net/hns3: support RSS") Fixes: ec674cb742e5 ("net/hns3: fix flushing RSS rule") Signed-off-by: Lijun Ou --- drivers/net/hns3/hns3_flow.c | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/net/hns3/hns3_flow.c b/drivers/net/hns3/hns3_flow.c index f303df4ad8..e936098611 100644 --- a/drivers/net/hns3/hns3_flow.c +++ b/drivers/net/hns3/hns3_flow.c @@ -91,9 +91,9 @@ net_addr_to_host(uint32_t *dst, const rte_be32_t *src, size_t len) /* * This function is used to find rss general action. * 1. As we know RSS is used to spread packets among several queues, the flow - * API provide the struct rte_flow_action_rss, user could config it's field + * API provide the struct rte_flow_action_rss, user could config its field * sush as: func/level/types/key/queue to control RSS function. - * 2. The flow API also support queue region configuration for hns3. It was + * 2. The flow API also supports queue region configuration for hns3. It was * implemented by FDIR + RSS in hns3 hardware, user can create one FDIR rule * which action is RSS queues region. * 3. When action is RSS, we use the following rule to distinguish: @@ -128,11 +128,11 @@ hns3_find_rss_general_action(const struct rte_flow_item pattern[], rss = act->conf; if (have_eth && rss->conf.queue_num) { /* - * Patter have ETH and action's queue_num > 0, indicate this is + * Pattern have ETH and action's queue_num > 0, indicate this is * queue region configuration. * Because queue region is implemented by FDIR + RSS in hns3 - * hardware, it need enter FDIR process, so here return NULL to - * avoid enter RSS process. + * hardware, it needs to enter FDIR process, so here return NULL + * to avoid enter RSS process. */ return NULL; } @@ -405,7 +405,6 @@ hns3_handle_actions(struct rte_eth_dev *dev, return 0; } -/* Parse to get the attr and action info of flow director rule. */ static int hns3_check_attr(const struct rte_flow_attr *attr, struct rte_flow_error *error) { @@ -800,7 +799,7 @@ hns3_parse_sctp(const struct rte_flow_item *item, struct hns3_fdir_rule *rule, } /* - * Check items before tunnel, save inner configs to outer configs,and clear + * Check items before tunnel, save inner configs to outer configs, and clear * inner configs. * The key consists of two parts: meta_data and tuple keys. * Meta data uses 15 bits, including vlan_num(2bit), des_port(12bit) and tunnel @@ -1490,10 +1489,8 @@ hns3_hw_rss_hash_set(struct hns3_hw *hw, struct rte_flow_action_rss *rss_config) if (ret) return ret; - /* Update algorithm of hw */ hw->rss_info.conf.func = rss_config->func; - /* Set flow type supported */ tuple = &hw->rss_info.rss_tuple_sets; ret = hns3_set_rss_tuple_by_rss_hf(hw, tuple, rss_config->types); if (ret) @@ -1578,7 +1575,7 @@ hns3_config_rss_filter(struct rte_eth_dev *dev, if (rss_flow_conf.queue_num) { /* * Due the content of queue pointer have been reset to - * 0, the rss_info->conf.queue should be set NULL + * 0, the rss_info->conf.queue should be set to NULL */ rss_info->conf.queue = NULL; rss_info->conf.queue_num = 0; @@ -1744,7 +1741,7 @@ hns3_flow_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr, /* * Create or destroy a flow rule. * Theorically one rule can match more than one filters. - * We will let it use the filter which it hitt first. + * We will let it use the filter which it hit first. * So, the sequence matters. */ static struct rte_flow * -- 2.29.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2021-02-05 11:18:40.772865947 +0000 +++ 0263-net-hns3-adjust-some-comments.patch 2021-02-05 11:18:29.314700784 +0000 @@ -1 +1 @@ -From f77b3c3a7b4a968dd42935420df0c00eefb506da Mon Sep 17 00:00:00 2001 +From b6d4d8a21fc5456a5eaa71cd5de5805f43d62c97 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit f77b3c3a7b4a968dd42935420df0c00eefb506da ] + @@ -12 +13,0 @@ -Cc: stable@dpdk.org @@ -20 +21 @@ -index 8a5179d6e0..f2bff1eb1a 100644 +index f303df4ad8..e936098611 100644 @@ -58 +59 @@ -@@ -782,7 +781,7 @@ hns3_parse_sctp(const struct rte_flow_item *item, struct hns3_fdir_rule *rule, +@@ -800,7 +799,7 @@ hns3_parse_sctp(const struct rte_flow_item *item, struct hns3_fdir_rule *rule, @@ -67 +68 @@ -@@ -1473,10 +1472,8 @@ hns3_hw_rss_hash_set(struct hns3_hw *hw, struct rte_flow_action_rss *rss_config) +@@ -1490,10 +1489,8 @@ hns3_hw_rss_hash_set(struct hns3_hw *hw, struct rte_flow_action_rss *rss_config) @@ -78 +79 @@ -@@ -1561,7 +1558,7 @@ hns3_config_rss_filter(struct rte_eth_dev *dev, +@@ -1578,7 +1575,7 @@ hns3_config_rss_filter(struct rte_eth_dev *dev, @@ -87 +88 @@ -@@ -1727,7 +1724,7 @@ hns3_flow_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr, +@@ -1744,7 +1741,7 @@ hns3_flow_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr,