From patchwork Thu Aug 31 09:56:29 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Renyong Wan X-Patchwork-Id: 130944 X-Patchwork-Delegate: thomas@monjalon.net 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 8139841FDA; Thu, 31 Aug 2023 11:59:18 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C0D45402DB; Thu, 31 Aug 2023 11:58:13 +0200 (CEST) Received: from VLXDG1SPAM1.ramaxel.com (email.unionmem.com [221.4.138.186]) by mails.dpdk.org (Postfix) with ESMTP id CCD19402B4 for ; Thu, 31 Aug 2023 11:58:08 +0200 (CEST) Received: from V12DG1MBS03.ramaxel.local ([172.26.18.33]) by VLXDG1SPAM1.ramaxel.com with ESMTP id 37V9vgLt064337; Thu, 31 Aug 2023 17:57:42 +0800 (GMT-8) (envelope-from wanry@3snic.com) Received: from localhost.localdomain (10.64.136.151) by V12DG1MBS03.ramaxel.local (172.26.18.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2375.17; Thu, 31 Aug 2023 17:57:41 +0800 From: To: CC: , Renyong Wan , Steven Song Subject: [PATCH v2 11/32] net/sssnic: add dev MAC ops Date: Thu, 31 Aug 2023 17:56:29 +0800 Message-ID: <20230831095650.219964-12-wanry@3snic.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230831095650.219964-1-wanry@3snic.com> References: <20230831095650.219964-1-wanry@3snic.com> MIME-Version: 1.0 X-Originating-IP: [10.64.136.151] X-ClientProxiedBy: V12DG1MBS03.ramaxel.local (172.26.18.33) To V12DG1MBS03.ramaxel.local (172.26.18.33) X-DNSRBL: X-SPAM-SOURCE-CHECK: pass X-MAIL: VLXDG1SPAM1.ramaxel.com 37V9vgLt064337 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: Renyong Wan Signed-off-by: Steven Song Signed-off-by: Renyong Wan --- doc/guides/nics/features/sssnic.ini | 2 + drivers/net/sssnic/base/sssnic_api.c | 174 +++++++++++++++++ drivers/net/sssnic/base/sssnic_api.h | 4 + drivers/net/sssnic/base/sssnic_cmd.h | 25 +++ drivers/net/sssnic/base/sssnic_hw.h | 1 + drivers/net/sssnic/sssnic_ethdev.c | 270 ++++++++++++++++++++++++++- drivers/net/sssnic/sssnic_ethdev.h | 2 + 7 files changed, 477 insertions(+), 1 deletion(-) diff --git a/doc/guides/nics/features/sssnic.ini b/doc/guides/nics/features/sssnic.ini index 6d9786db7e..602839d301 100644 --- a/doc/guides/nics/features/sssnic.ini +++ b/doc/guides/nics/features/sssnic.ini @@ -4,6 +4,8 @@ ; Refer to default.ini for the full list of available PMD features. ; [Features] +Unicast MAC filter = Y +Multicast MAC filter = Y Linux = Y ARMv8 = Y x86-64 = Y diff --git a/drivers/net/sssnic/base/sssnic_api.c b/drivers/net/sssnic/base/sssnic_api.c index bf0859cd63..76266dfbae 100644 --- a/drivers/net/sssnic/base/sssnic_api.c +++ b/drivers/net/sssnic/base/sssnic_api.c @@ -133,3 +133,177 @@ sssnic_capability_get(struct sssnic_hw *hw, struct sssnic_capability *capa) return 0; } + +int +sssnic_mac_addr_get(struct sssnic_hw *hw, uint8_t *addr) +{ + int ret; + struct sssnic_mac_addr_cmd cmd; + struct sssnic_msg msg; + uint32_t cmd_len; + uint16_t func; + + if (hw == NULL || addr == NULL) + return -EINVAL; + + if (SSSNIC_FUNC_TYPE(hw) == SSSNIC_FUNC_TYPE_VF) + func = SSSNIC_PF_FUNC_IDX(hw); + else + func = SSSNIC_MPU_FUNC_IDX; + + memset(&cmd, 0, sizeof(cmd)); + cmd_len = sizeof(cmd); + cmd.function = SSSNIC_FUNC_IDX(hw); + sssnic_msg_init(&msg, (uint8_t *)&cmd, cmd_len, SSSNIC_GET_MAC_ADDR_CMD, + func, SSSNIC_LAN_MODULE, SSSNIC_MSG_TYPE_REQ); + ret = sssnic_mbox_send(hw, &msg, (uint8_t *)&cmd, &cmd_len, 0); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to send mbox message, ret=%d", ret); + return ret; + } + + if (cmd_len == 0 || cmd.common.status != 0) { + PMD_DRV_LOG(ERR, + "Bad response to SSSNIC_GET_DEF_MAC_ADDR_CMD, len=%u, status=%u", + cmd_len, cmd.common.status); + return -EIO; + } + + rte_memcpy(addr, cmd.addr, 6); + + return 0; +} + +int +sssnic_mac_addr_update(struct sssnic_hw *hw, uint8_t *new, uint8_t *old) +{ + int ret; + struct sssnic_mac_addr_update_cmd cmd; + struct sssnic_msg msg; + uint32_t cmd_len; + uint16_t func; + + if (hw == NULL || new == NULL || old == NULL) + return -EINVAL; + + if (SSSNIC_FUNC_TYPE(hw) == SSSNIC_FUNC_TYPE_VF) + func = SSSNIC_PF_FUNC_IDX(hw); + else + func = SSSNIC_MPU_FUNC_IDX; + + memset(&cmd, 0, sizeof(cmd)); + cmd_len = sizeof(cmd); + cmd.function = SSSNIC_FUNC_IDX(hw); + rte_memcpy(cmd.new_addr, new, 6); + rte_memcpy(cmd.old_addr, old, 6); + sssnic_msg_init(&msg, (uint8_t *)&cmd, cmd_len, + SSSNIC_UPDATE_MAC_ADDR_CMD, func, SSSNIC_LAN_MODULE, + SSSNIC_MSG_TYPE_REQ); + ret = sssnic_mbox_send(hw, &msg, (uint8_t *)&cmd, &cmd_len, 0); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to send mbox message, ret=%d", ret); + return ret; + } + + if (cmd_len == 0 || cmd.common.status != 0) { + if (cmd.common.status == SSSNIC_MAC_ADDR_CMD_STATUS_IGNORED) { + PMD_DRV_LOG(WARNING, + "MAC address operation is ignored"); + return 0; + } + PMD_DRV_LOG(ERR, + "Bad response to SSSNIC_UPDATE_MAC_ADDR_CMD, len=%u, status=%u", + cmd_len, cmd.common.status); + return -EIO; + } + + return 0; +} + +int +sssnic_mac_addr_add(struct sssnic_hw *hw, uint8_t *addr) +{ + int ret; + struct sssnic_mac_addr_cmd cmd; + struct sssnic_msg msg; + uint32_t cmd_len; + uint16_t func; + + if (hw == NULL || addr == NULL) + return -EINVAL; + + if (SSSNIC_FUNC_TYPE(hw) == SSSNIC_FUNC_TYPE_VF) + func = SSSNIC_PF_FUNC_IDX(hw); + else + func = SSSNIC_MPU_FUNC_IDX; + + memset(&cmd, 0, sizeof(cmd)); + cmd_len = sizeof(cmd); + cmd.function = SSSNIC_FUNC_IDX(hw); + rte_memcpy(cmd.addr, addr, 6); + sssnic_msg_init(&msg, (uint8_t *)&cmd, cmd_len, SSSNIC_ADD_MAC_ADDR_CMD, + func, SSSNIC_LAN_MODULE, SSSNIC_MSG_TYPE_REQ); + ret = sssnic_mbox_send(hw, &msg, (uint8_t *)&cmd, &cmd_len, 0); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to send mbox message, ret=%d", ret); + return ret; + } + + if (cmd_len == 0 || cmd.common.status != 0) { + if (cmd.common.status == SSSNIC_MAC_ADDR_CMD_STATUS_IGNORED) { + PMD_DRV_LOG(WARNING, + "MAC address operation is ignored"); + return 0; + } + PMD_DRV_LOG(ERR, + "Bad response to SSSNIC_ADD_MAC_ADDR_CMD, len=%u, status=%u", + cmd_len, cmd.common.status); + return -EIO; + } + + return 0; +} + +int +sssnic_mac_addr_del(struct sssnic_hw *hw, uint8_t *addr) +{ + int ret; + struct sssnic_mac_addr_cmd cmd; + struct sssnic_msg msg; + uint32_t cmd_len; + uint16_t func; + + if (hw == NULL || addr == NULL) + return -EINVAL; + + if (SSSNIC_FUNC_TYPE(hw) == SSSNIC_FUNC_TYPE_VF) + func = SSSNIC_PF_FUNC_IDX(hw); + else + func = SSSNIC_MPU_FUNC_IDX; + + memset(&cmd, 0, sizeof(cmd)); + cmd_len = sizeof(cmd); + cmd.function = SSSNIC_FUNC_IDX(hw); + rte_memcpy(cmd.addr, addr, 6); + sssnic_msg_init(&msg, (uint8_t *)&cmd, cmd_len, SSSNIC_DEL_MAC_ADDR_CMD, + func, SSSNIC_LAN_MODULE, SSSNIC_MSG_TYPE_REQ); + ret = sssnic_mbox_send(hw, &msg, (uint8_t *)&cmd, &cmd_len, 0); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to send mbox message, ret=%d", ret); + return ret; + } + + if (cmd_len == 0 || cmd.common.status != 0) { + if (cmd.common.status == SSSNIC_MAC_ADDR_CMD_STATUS_IGNORED) { + PMD_DRV_LOG(WARNING, + "MAC address operation is ignored"); + return 0; + } + PMD_DRV_LOG(ERR, + "Bad response to SSSNIC_DEL_MAC_ADDR_CMD, len=%u, status=%u", + cmd_len, cmd.common.status); + return -EIO; + } + + return 0; +} diff --git a/drivers/net/sssnic/base/sssnic_api.h b/drivers/net/sssnic/base/sssnic_api.h index 8011cc8b0f..b985230595 100644 --- a/drivers/net/sssnic/base/sssnic_api.h +++ b/drivers/net/sssnic/base/sssnic_api.h @@ -27,5 +27,9 @@ int sssnic_msix_attr_get(struct sssnic_hw *hw, uint16_t msix_idx, int sssnic_msix_attr_set(struct sssnic_hw *hw, uint16_t msix_idx, struct sssnic_msix_attr *attr); int sssnic_capability_get(struct sssnic_hw *hw, struct sssnic_capability *capa); +int sssnic_mac_addr_get(struct sssnic_hw *hw, uint8_t *addr); +int sssnic_mac_addr_update(struct sssnic_hw *hw, uint8_t *new, uint8_t *old); +int sssnic_mac_addr_add(struct sssnic_hw *hw, uint8_t *addr); +int sssnic_mac_addr_del(struct sssnic_hw *hw, uint8_t *addr); #endif /* _SSSNIC_API_H_ */ diff --git a/drivers/net/sssnic/base/sssnic_cmd.h b/drivers/net/sssnic/base/sssnic_cmd.h index 79192affbc..e70184866e 100644 --- a/drivers/net/sssnic/base/sssnic_cmd.h +++ b/drivers/net/sssnic/base/sssnic_cmd.h @@ -20,6 +20,14 @@ enum sssnic_mgmt_cmd_id { #define SSSNIC_GET_CAPABILITY_CMD 0 +#define SSSNIC_MAC_ADDR_CMD_STATUS_IGNORED 0x4 +enum sssnic_mac_addr_cmd_id { + SSSNIC_GET_MAC_ADDR_CMD = 20, + SSSNIC_ADD_MAC_ADDR_CMD, + SSSNIC_DEL_MAC_ADDR_CMD, + SSSNIC_UPDATE_MAC_ADDR_CMD, +}; + struct sssnic_cmd_common { uint8_t status; uint8_t version; @@ -125,4 +133,21 @@ struct sssnic_capability_get_cmd { uint32_t resvd3[63]; }; +struct sssnic_mac_addr_cmd { + struct sssnic_cmd_common common; + uint16_t function; + uint16_t vlan; + uint16_t resvd; + uint8_t addr[6]; +}; + +struct sssnic_mac_addr_update_cmd { + struct sssnic_cmd_common common; + uint16_t function; + uint16_t vlan; + uint16_t resvd0; + uint8_t old_addr[6]; + uint16_t resvd1; + uint8_t new_addr[6]; +}; #endif /* _SSSNIC_CMD_H_ */ diff --git a/drivers/net/sssnic/base/sssnic_hw.h b/drivers/net/sssnic/base/sssnic_hw.h index 5f20a9465d..f8fe2ac7e1 100644 --- a/drivers/net/sssnic/base/sssnic_hw.h +++ b/drivers/net/sssnic/base/sssnic_hw.h @@ -70,6 +70,7 @@ struct sssnic_hw { #define SSSNIC_PHY_PORT(hw) ((hw)->phy_port) #define SSSNIC_FUNC_TYPE(hw) ((hw)->attr.func_type) #define SSSNIC_AF_FUNC_IDX(hw) ((hw)->attr.af_idx) +#define SSSNIC_PF_FUNC_IDX(hw) ((hw)->attr.pf_idx) enum sssnic_module { SSSNIC_COMM_MODULE = 0, diff --git a/drivers/net/sssnic/sssnic_ethdev.c b/drivers/net/sssnic/sssnic_ethdev.c index 2d7ed7d60b..92a20649fc 100644 --- a/drivers/net/sssnic/sssnic_ethdev.c +++ b/drivers/net/sssnic/sssnic_ethdev.c @@ -7,6 +7,7 @@ #include "sssnic_log.h" #include "base/sssnic_hw.h" +#include "base/sssnic_api.h" #include "sssnic_ethdev.h" static int @@ -63,6 +64,258 @@ sssnic_ethdev_infos_get(struct rte_eth_dev *ethdev, return 0; } +static int +sssnic_ethdev_mac_addr_set(struct rte_eth_dev *ethdev, + struct rte_ether_addr *mac_addr) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, mac_addr); + + ret = sssnic_mac_addr_update(hw, mac_addr->addr_bytes, + netdev->default_addr.addr_bytes); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to update default MAC address: %s", + mac_str); + return ret; + } + rte_ether_addr_copy(mac_addr, &netdev->default_addr); + + PMD_DRV_LOG(INFO, "Updated default MAC address %s of port %u", mac_str, + ethdev->data->port_id); + + return 0; +} + +static void +sssnic_ethdev_mac_addr_remove(struct rte_eth_dev *ethdev, uint32_t index) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + struct rte_ether_addr *mac; + char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + mac = ðdev->data->mac_addrs[index]; + ret = sssnic_mac_addr_del(hw, mac->addr_bytes); + if (ret != 0) { + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, mac); + PMD_DRV_LOG(ERR, "Failed to delete MAC address %s", mac_str); + } +} + +static int +sssnic_ethdev_mac_addr_add(struct rte_eth_dev *ethdev, + struct rte_ether_addr *mac_addr, __rte_unused uint32_t index, + __rte_unused uint32_t vmdq) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + if (rte_is_multicast_ether_addr(mac_addr)) { + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, + mac_addr); + PMD_DRV_LOG(ERR, + "Invalid MAC address:%s, cannot be multicast address", + mac_str); + } + + ret = sssnic_mac_addr_add(hw, mac_addr->addr_bytes); + if (ret != 0) { + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, + mac_addr); + PMD_DRV_LOG(ERR, "Failed to add MAC address %s", mac_str); + return ret; + } + + return 0; +} + +static void +sssnic_ethdev_mcast_addrs_clean(struct rte_eth_dev *ethdev) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + int i; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + for (i = 0; i < SSSNIC_ETHDEV_MAX_NUM_MC_MAC; i++) { + if (rte_is_zero_ether_addr(&netdev->mcast_addrs[i])) + break; + + ret = sssnic_mac_addr_del(hw, + netdev->mcast_addrs[i].addr_bytes); + if (ret != 0) + PMD_DRV_LOG(WARNING, "Failed to delete MAC address"); + + memset(&netdev->mcast_addrs[i], 0, + sizeof(struct rte_ether_addr)); + } +} + +static int +sssnic_ethdev_set_mc_addr_list(struct rte_eth_dev *ethdev, + struct rte_ether_addr *mc_addr_set, uint32_t nb_mc_addr) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; + uint32_t i; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + if (nb_mc_addr > SSSNIC_ETHDEV_MAX_NUM_MC_MAC) { + PMD_DRV_LOG(ERR, + "Failed to set mcast address list to port %u, excceds max number:%u", + ethdev->data->port_id, SSSNIC_ETHDEV_MAX_NUM_MC_MAC); + return -EINVAL; + } + + for (i = 0; i < nb_mc_addr; i++) { + if (!rte_is_multicast_ether_addr(&mc_addr_set[i])) { + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, + &mc_addr_set[i]); + PMD_DRV_LOG(ERR, "Invalid Multicast MAC address: %s", + mac_str); + return -EINVAL; + } + } + + sssnic_ethdev_mcast_addrs_clean(ethdev); + + for (i = 0; i < nb_mc_addr; i++) { + ret = sssnic_mac_addr_add(hw, mc_addr_set[i].addr_bytes); + if (ret != 0) { + sssnic_ethdev_mcast_addrs_clean(ethdev); + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, + &mc_addr_set[i]); + PMD_DRV_LOG(ERR, + "Failed to add Multicast MAC address: %s", + mac_str); + return ret; + } + rte_ether_addr_copy(&mc_addr_set[i], &netdev->mcast_addrs[i]); + } + + return 0; +} + +static int +sssnic_ethdev_mac_addrs_init(struct rte_eth_dev *ethdev) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + struct rte_ether_addr default_addr; + char mac_str[RTE_ETHER_ADDR_FMT_SIZE]; + + PMD_INIT_FUNC_TRACE(); + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + ethdev->data->mac_addrs = rte_zmalloc(NULL, + SSSNIC_ETHDEV_MAX_NUM_UC_MAC * sizeof(struct rte_ether_addr), + 0); + if (ethdev->data->mac_addrs == NULL) { + PMD_DRV_LOG(ERR, + "Failed to allocate memory to store %u mac addresses", + SSSNIC_ETHDEV_MAX_NUM_UC_MAC); + return -ENOMEM; + } + + netdev->mcast_addrs = rte_zmalloc(NULL, + SSSNIC_ETHDEV_MAX_NUM_MC_MAC * sizeof(struct rte_ether_addr), + 0); + if (netdev->mcast_addrs == NULL) { + PMD_DRV_LOG(ERR, + "Failed to allocate memory to store %u mcast addresses", + SSSNIC_ETHDEV_MAX_NUM_MC_MAC); + ret = -ENOMEM; + goto alloc_mcast_addr_fail; + } + + /* initialize default MAC address */ + memset(&default_addr, 0, sizeof(default_addr)); + ret = sssnic_mac_addr_get(hw, default_addr.addr_bytes); + if (ret != 0) + PMD_DRV_LOG(NOTICE, + "Could not get default MAC address, will use random address"); + + if (rte_is_zero_ether_addr(&default_addr)) + rte_eth_random_addr(default_addr.addr_bytes); + + rte_ether_format_addr(mac_str, RTE_ETHER_ADDR_FMT_SIZE, &default_addr); + + ret = sssnic_mac_addr_add(hw, default_addr.addr_bytes); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to add default MAC address: %s", + mac_str); + goto add_ether_addr_fail; + } + + rte_ether_addr_copy(&default_addr, ðdev->data->mac_addrs[0]); + rte_ether_addr_copy(&default_addr, &netdev->default_addr); + + PMD_DRV_LOG(INFO, "Port %u default MAC address: %s", + ethdev->data->port_id, mac_str); + + return 0; + +add_ether_addr_fail: + rte_free(netdev->mcast_addrs); + netdev->mcast_addrs = NULL; +alloc_mcast_addr_fail: + rte_free(ethdev->data->mac_addrs); + ethdev->data->mac_addrs = NULL; + return ret; +} + +static void +sssnic_ethdev_mac_addrs_clean(struct rte_eth_dev *ethdev) +{ + int ret; + struct sssnic_netdev *netdev; + struct sssnic_hw *hw; + int i; + + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); + hw = SSSNIC_NETDEV_TO_HW(netdev); + + for (i = 0; i < SSSNIC_ETHDEV_MAX_NUM_UC_MAC; i++) { + if (rte_is_zero_ether_addr(ðdev->data->mac_addrs[i])) + continue; + + ret = sssnic_mac_addr_del(hw, + ethdev->data->mac_addrs[i].addr_bytes); + if (ret != 0) + PMD_DRV_LOG(ERR, + "Failed to delete MAC address from port %u", + ethdev->data->port_id); + } + + sssnic_ethdev_mcast_addrs_clean(ethdev); +} + static int sssnic_ethdev_configure(struct rte_eth_dev *ethdev) { @@ -80,6 +333,7 @@ sssnic_ethdev_release(struct rte_eth_dev *ethdev) { struct sssnic_hw *hw = SSSNIC_ETHDEV_TO_HW(ethdev); + sssnic_ethdev_mac_addrs_clean(ethdev); sssnic_hw_shutdown(hw); rte_free(hw); } @@ -87,6 +341,10 @@ sssnic_ethdev_release(struct rte_eth_dev *ethdev) static const struct eth_dev_ops sssnic_ethdev_ops = { .dev_configure = sssnic_ethdev_configure, .dev_infos_get = sssnic_ethdev_infos_get, + .mac_addr_set = sssnic_ethdev_mac_addr_set, + .mac_addr_remove = sssnic_ethdev_mac_addr_remove, + .mac_addr_add = sssnic_ethdev_mac_addr_add, + .set_mc_addr_list = sssnic_ethdev_set_mc_addr_list, }; static int @@ -118,12 +376,22 @@ sssnic_ethdev_init(struct rte_eth_dev *ethdev) return ret; } + ret = sssnic_ethdev_mac_addrs_init(ethdev); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Failed to initialize MAC addresses"); + goto mac_addrs_init_fail; + } + netdev->max_num_rxq = SSSNIC_MAX_NUM_RXQ(hw); netdev->max_num_txq = SSSNIC_MAX_NUM_TXQ(hw); ethdev->dev_ops = &sssnic_ethdev_ops; return 0; + +mac_addrs_init_fail: + sssnic_hw_shutdown(0); + return ret; } static int @@ -140,7 +408,7 @@ sssnic_ethdev_uninit(struct rte_eth_dev *ethdev) sssnic_ethdev_release(ethdev); - return -EINVAL; + return 0; } static int diff --git a/drivers/net/sssnic/sssnic_ethdev.h b/drivers/net/sssnic/sssnic_ethdev.h index 70dd43d5c0..636b3fd04c 100644 --- a/drivers/net/sssnic/sssnic_ethdev.h +++ b/drivers/net/sssnic/sssnic_ethdev.h @@ -58,6 +58,8 @@ struct sssnic_netdev { void *hw; + struct rte_ether_addr *mcast_addrs; + struct rte_ether_addr default_addr; uint16_t max_num_txq; uint16_t max_num_rxq; };