From patchwork Wed Sep 22 03:41:51 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 99382 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id 01FD9A0C45; Wed, 22 Sep 2021 05:43:31 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 08EA941184; Wed, 22 Sep 2021 05:43:27 +0200 (CEST) Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by mails.dpdk.org (Postfix) with ESMTP id 617DF4003F for ; Wed, 22 Sep 2021 05:43:24 +0200 (CEST) Received: from dggeme756-chm.china.huawei.com (unknown [172.30.72.55]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4HDkd10gQGzWMQn; Wed, 22 Sep 2021 11:42:13 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by dggeme756-chm.china.huawei.com (10.3.19.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2308.8; Wed, 22 Sep 2021 11:43:22 +0800 From: "Min Hu (Connor)" To: CC: , Date: Wed, 22 Sep 2021 11:41:51 +0800 Message-ID: <20210922034152.50196-2-humin29@huawei.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210922034152.50196-1-humin29@huawei.com> References: <20210922034152.50196-1-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-ClientProxiedBy: dggems701-chm.china.huawei.com (10.3.19.178) To dggeme756-chm.china.huawei.com (10.3.19.102) X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH 1/2] net/hns3: fix residual old MAC after setting default MAC 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 Sender: "dev" From: Huisong Li This problem occurs in the following scenarios: 1) reset is encountered when the adapter is running. 2) set a new default MAC address After the above two steps, the old default MAC address should be not take effect. But the current behavior is contrary to that. This is due to the change of the "default_addr_setted" in hw->mac from 'true' to 'false' after the reset. As a result, the old MAC address is not removed when the new default MAC address is set. This variable controls whether to delete the old default MAC address when setting the default MAC address. It is only used when the mac_addr_set API is called for the first time. In fact, when a unicast MAC address is deleted, if the address isn't in the MAC address table, the driver doesn't return failure. So this patch remove the redundant and troublesome variables to resolve this problem. Fixes: 7d7f9f80bbfb ("net/hns3: support MAC address related operations") Cc: stable@dpdk.org Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_ethdev.c | 38 ++++++++++------------------------ drivers/net/hns3/hns3_ethdev.h | 1 - 2 files changed, 11 insertions(+), 28 deletions(-) diff --git a/drivers/net/hns3/hns3_ethdev.c b/drivers/net/hns3/hns3_ethdev.c index 7d37004972..5c8ac5754f 100644 --- a/drivers/net/hns3/hns3_ethdev.c +++ b/drivers/net/hns3/hns3_ethdev.c @@ -1651,7 +1651,7 @@ hns3_remove_mc_addr_common(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) static int hns3_add_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr, - uint32_t idx, __rte_unused uint32_t pool) + __rte_unused uint32_t idx, __rte_unused uint32_t pool) { struct hns3_hw *hw = HNS3_DEV_PRIVATE_TO_HW(dev->data->dev_private); char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; @@ -1682,8 +1682,6 @@ hns3_add_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr, return ret; } - if (idx == 0) - hw->mac.default_addr_setted = true; rte_spinlock_unlock(&hw->lock); return ret; @@ -1748,30 +1746,19 @@ hns3_set_default_mac_addr(struct rte_eth_dev *dev, struct hns3_hw *hw = HNS3_DEV_PRIVATE_TO_HW(dev->data->dev_private); struct rte_ether_addr *oaddr; char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; - bool default_addr_setted; int ret, ret_val; - /* - * It has been guaranteed that input parameter named mac_addr is valid - * address in the rte layer of DPDK framework. - */ - oaddr = (struct rte_ether_addr *)hw->mac.mac_addr; - default_addr_setted = hw->mac.default_addr_setted; - if (default_addr_setted && !!rte_is_same_ether_addr(mac_addr, oaddr)) - return 0; - rte_spinlock_lock(&hw->lock); - if (default_addr_setted) { - ret = hns3_remove_uc_addr_common(hw, oaddr); - if (ret) { - hns3_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, - oaddr); - hns3_warn(hw, "Remove old uc mac address(%s) fail: %d", - mac_str, ret); + oaddr = (struct rte_ether_addr *)hw->mac.mac_addr; + ret = hns3_remove_uc_addr_common(hw, oaddr); + if (ret) { + hns3_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, + oaddr); + hns3_warn(hw, "Remove old uc mac address(%s) fail: %d", + mac_str, ret); - rte_spinlock_unlock(&hw->lock); - return ret; - } + rte_spinlock_unlock(&hw->lock); + return ret; } ret = hns3_add_uc_addr_common(hw, mac_addr); @@ -1790,7 +1777,6 @@ hns3_set_default_mac_addr(struct rte_eth_dev *dev, rte_ether_addr_copy(mac_addr, (struct rte_ether_addr *)hw->mac.mac_addr); - hw->mac.default_addr_setted = true; rte_spinlock_unlock(&hw->lock); return 0; @@ -1811,7 +1797,6 @@ hns3_set_default_mac_addr(struct rte_eth_dev *dev, hns3_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, oaddr); hns3_warn(hw, "Failed to restore old uc mac addr(%s): %d", mac_str, ret_val); - hw->mac.default_addr_setted = false; } rte_spinlock_unlock(&hw->lock); @@ -3473,7 +3458,6 @@ hns3_get_board_configuration(struct hns3_hw *hw) hw->rss_dis_flag = false; memcpy(hw->mac.mac_addr, cfg.mac_addr, RTE_ETHER_ADDR_LEN); hw->mac.phy_addr = cfg.phy_addr; - hw->mac.default_addr_setted = false; hw->num_tx_desc = cfg.tqp_desc_num; hw->num_rx_desc = cfg.tqp_desc_num; hw->dcb_info.num_pg = 1; @@ -5931,7 +5915,7 @@ hns3_do_stop(struct hns3_adapter *hns) return ret; } } - hw->mac.default_addr_setted = false; + return 0; } diff --git a/drivers/net/hns3/hns3_ethdev.h b/drivers/net/hns3/hns3_ethdev.h index 0e4e4269a1..243a4046ae 100644 --- a/drivers/net/hns3/hns3_ethdev.h +++ b/drivers/net/hns3/hns3_ethdev.h @@ -188,7 +188,6 @@ enum hns3_media_type { struct hns3_mac { uint8_t mac_addr[RTE_ETHER_ADDR_LEN]; - bool default_addr_setted; /* whether default addr(mac_addr) is set */ uint8_t media_type; uint8_t phy_addr; uint8_t link_duplex : 1; /* ETH_LINK_[HALF/FULL]_DUPLEX */ From patchwork Wed Sep 22 03:41:52 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 99383 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id 43803A0C45; Wed, 22 Sep 2021 05:43:35 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 20D5F4118C; Wed, 22 Sep 2021 05:43:28 +0200 (CEST) Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by mails.dpdk.org (Postfix) with ESMTP id 8F41C40141 for ; Wed, 22 Sep 2021 05:43:24 +0200 (CEST) Received: from dggeme756-chm.china.huawei.com (unknown [172.30.72.53]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4HDkYW1pM2zRJ9g; Wed, 22 Sep 2021 11:39:11 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by dggeme756-chm.china.huawei.com (10.3.19.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2308.8; Wed, 22 Sep 2021 11:43:22 +0800 From: "Min Hu (Connor)" To: CC: , Date: Wed, 22 Sep 2021 11:41:52 +0800 Message-ID: <20210922034152.50196-3-humin29@huawei.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210922034152.50196-1-humin29@huawei.com> References: <20210922034152.50196-1-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-ClientProxiedBy: dggems701-chm.china.huawei.com (10.3.19.178) To dggeme756-chm.china.huawei.com (10.3.19.102) X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH 2/2] net/hns3: fix input parameters of MAC related functions 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 Sender: "dev" From: Huisong Li When adding multicast and unicast MAC addresses, three descriptors and one descriptor are required for querying or adding MAC VLAN table, respectively. This patch uses the number of descriptors as input parameter to complete this task to make the function more secure. Fixes: 7d7f9f80bbfb ("net/hns3: support MAC address related operations") Cc: stable@dpdk.org Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_cmd.h | 3 +- drivers/net/hns3/hns3_ethdev.c | 88 +++++++++++++++++++--------------- 2 files changed, 51 insertions(+), 40 deletions(-) diff --git a/drivers/net/hns3/hns3_cmd.h b/drivers/net/hns3/hns3_cmd.h index a4683de0aa..81bc9e9d98 100644 --- a/drivers/net/hns3/hns3_cmd.h +++ b/drivers/net/hns3/hns3_cmd.h @@ -923,7 +923,8 @@ enum hns3_mac_vlan_add_resp_code { HNS3_ADD_MC_OVERFLOW, /* ADD failed for MC overflow */ }; -#define HNS3_MC_MAC_VLAN_ADD_DESC_NUM 3 +#define HNS3_MC_MAC_VLAN_OPS_DESC_NUM 3 +#define HNS3_UC_MAC_VLAN_OPS_DESC_NUM 1 #define HNS3_MAC_VLAN_BIT0_EN_B 0 #define HNS3_MAC_VLAN_BIT1_EN_B 1 diff --git a/drivers/net/hns3/hns3_ethdev.c b/drivers/net/hns3/hns3_ethdev.c index 5c8ac5754f..cabf73ffbc 100644 --- a/drivers/net/hns3/hns3_ethdev.c +++ b/drivers/net/hns3/hns3_ethdev.c @@ -1427,28 +1427,31 @@ hns3_get_mac_vlan_cmd_status(struct hns3_hw *hw, uint16_t cmdq_resp, static int hns3_lookup_mac_vlan_tbl(struct hns3_hw *hw, struct hns3_mac_vlan_tbl_entry_cmd *req, - struct hns3_cmd_desc *desc, bool is_mc) + struct hns3_cmd_desc *desc, uint8_t desc_num) { uint8_t resp_code; uint16_t retval; int ret; + int i; - hns3_cmd_setup_basic_desc(&desc[0], HNS3_OPC_MAC_VLAN_ADD, true); - if (is_mc) { - desc[0].flag |= rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); - memcpy(desc[0].data, req, - sizeof(struct hns3_mac_vlan_tbl_entry_cmd)); - hns3_cmd_setup_basic_desc(&desc[1], HNS3_OPC_MAC_VLAN_ADD, - true); - desc[1].flag |= rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); - hns3_cmd_setup_basic_desc(&desc[2], HNS3_OPC_MAC_VLAN_ADD, + if (desc_num == HNS3_MC_MAC_VLAN_OPS_DESC_NUM) { + for (i = 0; i < desc_num - 1; i++) { + hns3_cmd_setup_basic_desc(&desc[i], + HNS3_OPC_MAC_VLAN_ADD, true); + desc[i].flag |= rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); + if (i == 0) + memcpy(desc[i].data, req, + sizeof(struct hns3_mac_vlan_tbl_entry_cmd)); + } + hns3_cmd_setup_basic_desc(&desc[i], HNS3_OPC_MAC_VLAN_ADD, true); - ret = hns3_cmd_send(hw, desc, HNS3_MC_MAC_VLAN_ADD_DESC_NUM); } else { + hns3_cmd_setup_basic_desc(&desc[0], HNS3_OPC_MAC_VLAN_ADD, + true); memcpy(desc[0].data, req, sizeof(struct hns3_mac_vlan_tbl_entry_cmd)); - ret = hns3_cmd_send(hw, desc, 1); } + ret = hns3_cmd_send(hw, desc, desc_num); if (ret) { hns3_err(hw, "lookup mac addr failed for cmd_send, ret =%d.", ret); @@ -1464,38 +1467,40 @@ hns3_lookup_mac_vlan_tbl(struct hns3_hw *hw, static int hns3_add_mac_vlan_tbl(struct hns3_hw *hw, struct hns3_mac_vlan_tbl_entry_cmd *req, - struct hns3_cmd_desc *mc_desc) + struct hns3_cmd_desc *desc, uint8_t desc_num) { uint8_t resp_code; uint16_t retval; int cfg_status; int ret; + int i; - if (mc_desc == NULL) { - struct hns3_cmd_desc desc; - - hns3_cmd_setup_basic_desc(&desc, HNS3_OPC_MAC_VLAN_ADD, false); - memcpy(desc.data, req, + if (desc_num == HNS3_UC_MAC_VLAN_OPS_DESC_NUM) { + hns3_cmd_setup_basic_desc(desc, HNS3_OPC_MAC_VLAN_ADD, false); + memcpy(desc->data, req, sizeof(struct hns3_mac_vlan_tbl_entry_cmd)); - ret = hns3_cmd_send(hw, &desc, 1); - resp_code = (rte_le_to_cpu_32(desc.data[0]) >> 8) & 0xff; - retval = rte_le_to_cpu_16(desc.retval); + ret = hns3_cmd_send(hw, desc, desc_num); + resp_code = (rte_le_to_cpu_32(desc->data[0]) >> 8) & 0xff; + retval = rte_le_to_cpu_16(desc->retval); cfg_status = hns3_get_mac_vlan_cmd_status(hw, retval, resp_code, HNS3_MAC_VLAN_ADD); } else { - hns3_cmd_reuse_desc(&mc_desc[0], false); - mc_desc[0].flag |= rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); - hns3_cmd_reuse_desc(&mc_desc[1], false); - mc_desc[1].flag |= rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); - hns3_cmd_reuse_desc(&mc_desc[2], false); - mc_desc[2].flag &= rte_cpu_to_le_16(~HNS3_CMD_FLAG_NEXT); - memcpy(mc_desc[0].data, req, + for (i = 0; i < desc_num; i++) { + hns3_cmd_reuse_desc(&desc[i], false); + if (i == desc_num - 1) + desc[i].flag &= + rte_cpu_to_le_16(~HNS3_CMD_FLAG_NEXT); + else + desc[i].flag |= + rte_cpu_to_le_16(HNS3_CMD_FLAG_NEXT); + } + memcpy(desc[0].data, req, sizeof(struct hns3_mac_vlan_tbl_entry_cmd)); - mc_desc[0].retval = 0; - ret = hns3_cmd_send(hw, mc_desc, HNS3_MC_MAC_VLAN_ADD_DESC_NUM); - resp_code = (rte_le_to_cpu_32(mc_desc[0].data[0]) >> 8) & 0xff; - retval = rte_le_to_cpu_16(mc_desc[0].retval); + desc[0].retval = 0; + ret = hns3_cmd_send(hw, desc, desc_num); + resp_code = (rte_le_to_cpu_32(desc[0].data[0]) >> 8) & 0xff; + retval = rte_le_to_cpu_16(desc[0].retval); cfg_status = hns3_get_mac_vlan_cmd_status(hw, retval, resp_code, HNS3_MAC_VLAN_ADD); @@ -1540,7 +1545,7 @@ hns3_add_uc_addr_common(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) struct hns3_adapter *hns = HNS3_DEV_HW_TO_ADAPTER(hw); struct hns3_mac_vlan_tbl_entry_cmd req; struct hns3_pf *pf = &hns->pf; - struct hns3_cmd_desc desc[3]; + struct hns3_cmd_desc desc; char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; uint16_t egress_port = 0; uint8_t vf_id; @@ -1574,10 +1579,12 @@ hns3_add_uc_addr_common(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) * it if the entry is inexistent. Repeated unicast entry * is not allowed in the mac vlan table. */ - ret = hns3_lookup_mac_vlan_tbl(hw, &req, desc, false); + ret = hns3_lookup_mac_vlan_tbl(hw, &req, &desc, + HNS3_UC_MAC_VLAN_OPS_DESC_NUM); if (ret == -ENOENT) { if (!hns3_is_umv_space_full(hw)) { - ret = hns3_add_mac_vlan_tbl(hw, &req, NULL); + ret = hns3_add_mac_vlan_tbl(hw, &req, &desc, + HNS3_UC_MAC_VLAN_OPS_DESC_NUM); if (!ret) hns3_update_umv_space(hw, false); return ret; @@ -1867,8 +1874,8 @@ hns3_update_desc_vfid(struct hns3_cmd_desc *desc, uint8_t vfid, bool clr) static int hns3_add_mc_addr(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) { + struct hns3_cmd_desc desc[HNS3_MC_MAC_VLAN_OPS_DESC_NUM]; struct hns3_mac_vlan_tbl_entry_cmd req; - struct hns3_cmd_desc desc[3]; char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; uint8_t vf_id; int ret; @@ -1885,7 +1892,8 @@ hns3_add_mc_addr(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) memset(&req, 0, sizeof(req)); hns3_set_bit(req.entry_type, HNS3_MAC_VLAN_BIT0_EN_B, 0); hns3_prepare_mac_addr(&req, mac_addr->addr_bytes, true); - ret = hns3_lookup_mac_vlan_tbl(hw, &req, desc, true); + ret = hns3_lookup_mac_vlan_tbl(hw, &req, desc, + HNS3_MC_MAC_VLAN_OPS_DESC_NUM); if (ret) { /* This mac addr do not exist, add new entry for it */ memset(desc[0].data, 0, sizeof(desc[0].data)); @@ -1899,7 +1907,8 @@ hns3_add_mc_addr(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) */ vf_id = HNS3_PF_FUNC_ID; hns3_update_desc_vfid(desc, vf_id, false); - ret = hns3_add_mac_vlan_tbl(hw, &req, desc); + ret = hns3_add_mac_vlan_tbl(hw, &req, desc, + HNS3_MC_MAC_VLAN_OPS_DESC_NUM); if (ret) { if (ret == -ENOSPC) hns3_err(hw, "mc mac vlan table is full"); @@ -1932,7 +1941,8 @@ hns3_remove_mc_addr(struct hns3_hw *hw, struct rte_ether_addr *mac_addr) memset(&req, 0, sizeof(req)); hns3_set_bit(req.entry_type, HNS3_MAC_VLAN_BIT0_EN_B, 0); hns3_prepare_mac_addr(&req, mac_addr->addr_bytes, true); - ret = hns3_lookup_mac_vlan_tbl(hw, &req, desc, true); + ret = hns3_lookup_mac_vlan_tbl(hw, &req, desc, + HNS3_MC_MAC_VLAN_OPS_DESC_NUM); if (ret == 0) { /* * This mac addr exist, remove this handle's VFID for it.