Add device management operations via netlink: - Link up/down control - Link status query (checks IFF_UP and IFF_RUNNING) - Promiscuous mode enable/disable - All-multicast mode enable/disable - MTU configuration - MAC address configuration
All operations use the persistent netlink socket (pmd->nlsk_fd) and ifindex for stable interface identification across renames and namespace moves. Signed-off-by: Stephen Hemminger <[email protected]> --- doc/guides/nics/features/rtap.ini | 4 ++ drivers/net/rtap/rtap.h | 1 + drivers/net/rtap/rtap_ethdev.c | 102 ++++++++++++++++++++++++++++++ 3 files changed, 107 insertions(+) diff --git a/doc/guides/nics/features/rtap.ini b/doc/guides/nics/features/rtap.ini index 9bef9e341d..ce0804d795 100644 --- a/doc/guides/nics/features/rtap.ini +++ b/doc/guides/nics/features/rtap.ini @@ -4,6 +4,10 @@ ; Refer to default.ini for the full list of available PMD features. ; [Features] +Link status = Y +MTU update = Y +Promiscuous mode = Y +Allmulticast mode = Y Scattered Rx = P Basic stats = Y Stats per queue = Y diff --git a/drivers/net/rtap/rtap.h b/drivers/net/rtap/rtap.h index abfde20e60..aeda7f3f25 100644 --- a/drivers/net/rtap/rtap.h +++ b/drivers/net/rtap/rtap.h @@ -69,6 +69,7 @@ struct rtap_pmd { /* rtap_ethdev.c */ int rtap_queue_open(struct rte_eth_dev *dev, uint16_t queue_id); void rtap_queue_close(struct rte_eth_dev *dev, uint16_t queue_id); +int rtap_link_update(struct rte_eth_dev *dev, int wait_to_complete); /* rtap_netlink.c */ int rtap_nl_open(unsigned int groups); diff --git a/drivers/net/rtap/rtap_ethdev.c b/drivers/net/rtap/rtap_ethdev.c index 888f8e7f39..b6c6421e3d 100644 --- a/drivers/net/rtap/rtap_ethdev.c +++ b/drivers/net/rtap/rtap_ethdev.c @@ -108,9 +108,100 @@ rtap_tap_open(const char *name, struct ifreq *ifr, uint8_t persist) return -1; } +static int +rtap_set_link_up(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, IFF_UP, IFF_UP); +} + +static int +rtap_set_link_down(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, 0, IFF_UP); +} + +static int +rtap_promiscuous_enable(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, IFF_PROMISC, IFF_PROMISC); +} + +static int +rtap_promiscuous_disable(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, 0, IFF_PROMISC); +} + +static int +rtap_allmulticast_enable(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, IFF_ALLMULTI, IFF_ALLMULTI); +} + +static int +rtap_allmulticast_disable(struct rte_eth_dev *dev) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_change_flags(pmd->nlsk_fd, pmd->if_index, 0, IFF_ALLMULTI); +} + +int +rtap_link_update(struct rte_eth_dev *dev, int wait_to_complete __rte_unused) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + struct rte_eth_link link = { + .link_speed = RTE_ETH_SPEED_NUM_UNKNOWN, + .link_duplex = RTE_ETH_LINK_FULL_DUPLEX, + .link_autoneg = RTE_ETH_LINK_FIXED, + .link_status = RTE_ETH_LINK_DOWN, + }; + unsigned int flags = 0; + + if (rtap_nl_get_flags(pmd->nlsk_fd, pmd->if_index, &flags) < 0) + return -1; + + if ((flags & IFF_UP) && (flags & IFF_RUNNING)) + link.link_status = RTE_ETH_LINK_UP; + + rte_eth_linkstatus_set(dev, &link); + return 0; +} + +static int +rtap_mtu_set(struct rte_eth_dev *dev, uint16_t mtu) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_set_mtu(pmd->nlsk_fd, pmd->if_index, mtu); +} + +static int +rtap_macaddr_set(struct rte_eth_dev *dev, struct rte_ether_addr *addr) +{ + struct rtap_pmd *pmd = dev->data->dev_private; + + return rtap_nl_set_mac(pmd->nlsk_fd, pmd->if_index, addr); +} + static int rtap_dev_start(struct rte_eth_dev *dev) { + int ret = rtap_set_link_up(dev); + + if (ret != 0) + return ret; + dev->data->dev_link.link_status = RTE_ETH_LINK_UP; for (uint16_t i = 0; i < dev->data->nb_rx_queues; i++) { dev->data->rx_queue_state[i] = RTE_ETH_QUEUE_STATE_STARTED; @@ -127,6 +218,8 @@ rtap_dev_stop(struct rte_eth_dev *dev) dev->data->dev_link.link_status = RTE_ETH_LINK_DOWN; + rtap_set_link_down(dev); + for (uint16_t i = 0; i < dev->data->nb_rx_queues; i++) { dev->data->rx_queue_state[i] = RTE_ETH_QUEUE_STATE_STOPPED; dev->data->tx_queue_state[i] = RTE_ETH_QUEUE_STATE_STOPPED; @@ -365,6 +458,15 @@ static const struct eth_dev_ops rtap_ops = { .dev_configure = rtap_dev_configure, .dev_infos_get = rtap_dev_info, .dev_close = rtap_dev_close, + .link_update = rtap_link_update, + .dev_set_link_up = rtap_set_link_up, + .dev_set_link_down = rtap_set_link_down, + .mac_addr_set = rtap_macaddr_set, + .mtu_set = rtap_mtu_set, + .promiscuous_enable = rtap_promiscuous_enable, + .promiscuous_disable = rtap_promiscuous_disable, + .allmulticast_enable = rtap_allmulticast_enable, + .allmulticast_disable = rtap_allmulticast_disable, .stats_get = rtap_stats_get, .stats_reset = rtap_stats_reset, .rx_queue_setup = rtap_rx_queue_setup, -- 2.51.0

