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 32A21A0509; Wed, 6 Apr 2022 11:24:00 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 620F34287A; Wed, 6 Apr 2022 11:23:25 +0200 (CEST) Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by mails.dpdk.org (Postfix) with ESMTP id 751A640689 for ; Wed, 6 Apr 2022 11:23:19 +0200 (CEST) Received: from kwepemi500012.china.huawei.com (unknown [172.30.72.55]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4KYJt56ChZzgYQV; Wed, 6 Apr 2022 17:21:33 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by kwepemi500012.china.huawei.com (7.221.188.12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.24; Wed, 6 Apr 2022 17:23:17 +0800 From: "Min Hu (Connor)" To: CC: , Subject: [PATCH 5/7] net/hns3: fix RSS disable Date: Wed, 6 Apr 2022 17:22:38 +0800 Message-ID: <20220406092240.52900-6-humin29@huawei.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20220406092240.52900-1-humin29@huawei.com> References: <20220406092240.52900-1-humin29@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [10.69.192.56] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemi500012.china.huawei.com (7.221.188.12) X-CFilter-Loop: Reflected 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 From: Huisong Li Currently, hns3 PMD disable RSS by resetting redirection table when user set rss_hf to 0 so as to all packets go to queue 0. The implementation may cause following problems: 1) the same type packet may go to different queue on the case of disableing all tuples and partial tuples. The problem is determined by hardware design. 2) affect the configuration of redirection table and user experience. For hns3 hardware, the packets with RSS disabled are always go to the queue corresponding to first entry of the redirection table. Genernally, disable RSS should be implemented by disabling all tuples, This patch fix the implementation. Fixes: c37ca66f2b27 ("net/hns3: support RSS") Cc: stable@dpdk.org Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_ethdev.c | 1 - drivers/net/hns3/hns3_ethdev_vf.c | 1 - drivers/net/hns3/hns3_flow.c | 6 +- drivers/net/hns3/hns3_rss.c | 93 ++++++------------------------- 4 files changed, 18 insertions(+), 83 deletions(-) diff --git a/drivers/net/hns3/hns3_ethdev.c b/drivers/net/hns3/hns3_ethdev.c index 2d47088487..0b565a5614 100644 --- a/drivers/net/hns3/hns3_ethdev.c +++ b/drivers/net/hns3/hns3_ethdev.c @@ -2015,7 +2015,6 @@ hns3_dev_configure(struct rte_eth_dev *dev) goto cfg_err; } - /* When RSS is not configured, redirect the packet queue 0 */ if ((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG) { conf->rxmode.offloads |= RTE_ETH_RX_OFFLOAD_RSS_HASH; rss_conf = conf->rx_adv_conf.rss_conf; diff --git a/drivers/net/hns3/hns3_ethdev_vf.c b/drivers/net/hns3/hns3_ethdev_vf.c index 589de0ab3a..9ccce98d48 100644 --- a/drivers/net/hns3/hns3_ethdev_vf.c +++ b/drivers/net/hns3/hns3_ethdev_vf.c @@ -492,7 +492,6 @@ hns3vf_dev_configure(struct rte_eth_dev *dev) goto cfg_err; } - /* When RSS is not configured, redirect the packet queue 0 */ if ((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG) { conf->rxmode.offloads |= RTE_ETH_RX_OFFLOAD_RSS_HASH; rss_conf = conf->rx_adv_conf.rss_conf; diff --git a/drivers/net/hns3/hns3_flow.c b/drivers/net/hns3/hns3_flow.c index c5a37ee5eb..ed08e19656 100644 --- a/drivers/net/hns3/hns3_flow.c +++ b/drivers/net/hns3/hns3_flow.c @@ -1446,14 +1446,10 @@ hns3_disable_rss(struct hns3_hw *hw) { int ret; - /* Redirected the redirection table to queue 0 */ - ret = hns3_rss_reset_indir_table(hw); + ret = hns3_set_rss_tuple_by_rss_hf(hw, &hw->rss_info.rss_tuple_sets, 0); if (ret) return ret; - /* Disable RSS */ - hw->rss_info.conf.types = 0; - return 0; } diff --git a/drivers/net/hns3/hns3_rss.c b/drivers/net/hns3/hns3_rss.c index eb3c7cceec..087dc28f2d 100644 --- a/drivers/net/hns3/hns3_rss.c +++ b/drivers/net/hns3/hns3_rss.c @@ -237,31 +237,6 @@ hns3_rss_set_algo_key(struct hns3_hw *hw, const uint8_t *key) return 0; } -/* - * Used to configure the tuple selection for RSS hash input. - */ -static int -hns3_rss_set_input_tuple(struct hns3_hw *hw) -{ - struct hns3_rss_conf *rss_config = &hw->rss_info; - struct hns3_rss_input_tuple_cmd *req; - struct hns3_cmd_desc desc_tuple; - int ret; - - hns3_cmd_setup_basic_desc(&desc_tuple, HNS3_OPC_RSS_INPUT_TUPLE, false); - - req = (struct hns3_rss_input_tuple_cmd *)desc_tuple.data; - - req->tuple_field = - rte_cpu_to_le_64(rss_config->rss_tuple_sets.rss_tuple_fields); - - ret = hns3_cmd_send(hw, &desc_tuple, 1); - if (ret) - hns3_err(hw, "Configure RSS input tuple mode failed %d", ret); - - return ret; -} - /* * rss_indirection_table command function, opcode:0x0D07. * Used to configure the indirection table of rss. @@ -382,6 +357,8 @@ hns3_set_rss_tuple_by_rss_hf(struct hns3_hw *hw, } tuple->rss_tuple_fields = rte_le_to_cpu_64(req->tuple_field); + /* Update supported flow types when set tuple success */ + hw->rss_info.conf.types = rss_hf; return 0; } @@ -402,7 +379,6 @@ hns3_dev_rss_hash_update(struct rte_eth_dev *dev, struct hns3_adapter *hns = dev->data->dev_private; struct hns3_hw *hw = &hns->hw; struct hns3_rss_tuple_cfg *tuple = &hw->rss_info.rss_tuple_sets; - struct hns3_rss_conf *rss_cfg = &hw->rss_info; uint8_t key_len = rss_conf->rss_key_len; uint64_t rss_hf = rss_conf->rss_hf; uint8_t *key = rss_conf->rss_key; @@ -413,22 +389,6 @@ hns3_dev_rss_hash_update(struct rte_eth_dev *dev, if (ret) goto conf_err; - if (rss_cfg->conf.types && rss_hf == 0) { - /* Disable RSS, reset indirection table by local variable */ - ret = hns3_rss_reset_indir_table(hw); - if (ret) - goto conf_err; - } else if (rss_hf && rss_cfg->conf.types == 0) { - /* Enable RSS, restore indirection table by hw's config */ - ret = hns3_set_rss_indir_table(hw, rss_cfg->rss_indirection_tbl, - hw->rss_ind_tbl_size); - if (ret) - goto conf_err; - } - - /* Update supported flow types when set tuple success */ - rss_cfg->conf.types = rss_hf; - if (key) { if (key_len != HNS3_RSS_KEY_SIZE) { hns3_err(hw, "The hash key len(%u) is invalid", @@ -694,7 +654,8 @@ hns3_config_rss(struct hns3_adapter *hns) struct hns3_hw *hw = &hns->hw; struct hns3_rss_conf *rss_cfg = &hw->rss_info; uint8_t *hash_key = rss_cfg->key; - int ret, ret1; + uint64_t rss_hf; + int ret; enum rte_eth_rx_mq_mode mq_mode = hw->data->dev_conf.rxmode.mq_mode; @@ -710,51 +671,31 @@ hns3_config_rss(struct hns3_adapter *hns) break; } - /* When RSS is off, redirect the packet queue 0 */ - if (((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG) == 0) - hns3_rss_uninit(hns); - /* Configure RSS hash algorithm and hash key offset */ ret = hns3_rss_set_algo_key(hw, hash_key); if (ret) return ret; - /* Configure the tuple selection for RSS hash input */ - ret = hns3_rss_set_input_tuple(hw); + ret = hns3_set_rss_indir_table(hw, rss_cfg->rss_indirection_tbl, + hw->rss_ind_tbl_size); if (ret) return ret; - /* - * When RSS is off, it doesn't need to configure rss redirection table - * to hardware. - */ - if (((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG)) { - ret = hns3_set_rss_indir_table(hw, rss_cfg->rss_indirection_tbl, - hw->rss_ind_tbl_size); - if (ret) - goto rss_tuple_uninit; - } - ret = hns3_set_rss_tc_mode(hw); if (ret) - goto rss_indir_table_uninit; - - return ret; - -rss_indir_table_uninit: - if (((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG)) { - ret1 = hns3_rss_reset_indir_table(hw); - if (ret1 != 0) - return ret; - } - -rss_tuple_uninit: - hns3_rss_tuple_uninit(hw); + return ret; - /* Disable RSS */ - hw->rss_info.conf.types = 0; + /* + * When muli-queue RSS mode flag is not set or unsupported tuples are + * set, disable all tuples. + */ + rss_hf = hw->rss_info.conf.types; + if (!((uint32_t)mq_mode & RTE_ETH_MQ_RX_RSS_FLAG) || + !(rss_hf & HNS3_ETH_RSS_SUPPORT)) + rss_hf = 0; - return ret; + return hns3_set_rss_tuple_by_rss_hf(hw, &hw->rss_info.rss_tuple_sets, + rss_hf); } /* -- 2.33.0