From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 6185A1B2F5 for ; Wed, 1 Nov 2017 16:47:44 +0100 (CET) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP; 01 Nov 2017 08:47:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,329,1505804400"; d="scan'208";a="144707248" Received: from awal-z170x.ir.intel.com ([163.33.210.59]) by orsmga004.jf.intel.com with ESMTP; 01 Nov 2017 08:47:42 -0700 From: Mohammad Abdul Awal To: dev@dpdk.org Cc: Mohammad Abdul Awal , Remy Horton , Declan Doherty Date: Wed, 1 Nov 2017 15:46:26 +0000 Message-Id: <1509551187-25726-5-git-send-email-mohammad.abdul.awal@intel.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1509551187-25726-1-git-send-email-mohammad.abdul.awal@intel.com> References: <1509551187-25726-1-git-send-email-mohammad.abdul.awal@intel.com> Subject: [dpdk-dev] [PATCH 5/6] net/i40e: Enable port representor PMD and broker for fortville PMD driver X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 01 Nov 2017 15:47:45 -0000 Signed-off-by: Mohammad Abdul Awal Signed-off-by: Remy Horton Signed-off-by: Declan Doherty --- drivers/net/i40e/Makefile | 1 + drivers/net/i40e/i40e_ethdev.c | 16 ++ drivers/net/i40e/i40e_ethdev.h | 1 + drivers/net/i40e/i40e_prep_ops.c | 472 +++++++++++++++++++++++++++++++++++++++ drivers/net/i40e/i40e_prep_ops.h | 46 ++++ drivers/net/i40e/rte_pmd_i40e.c | 47 ++++ drivers/net/i40e/rte_pmd_i40e.h | 18 ++ 7 files changed, 601 insertions(+) create mode 100644 drivers/net/i40e/i40e_prep_ops.c create mode 100644 drivers/net/i40e/i40e_prep_ops.h diff --git a/drivers/net/i40e/Makefile b/drivers/net/i40e/Makefile index 9ab8c84..641bf26 100644 --- a/drivers/net/i40e/Makefile +++ b/drivers/net/i40e/Makefile @@ -113,6 +113,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += i40e_fdir.c SRCS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += i40e_flow.c SRCS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += rte_pmd_i40e.c SRCS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += i40e_tm.c +SRCS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += i40e_prep_ops.c # install this header file SYMLINK-$(CONFIG_RTE_LIBRTE_I40E_PMD)-include := rte_pmd_i40e.h diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c index bcd9ef1..00ffe6b 100644 --- a/drivers/net/i40e/i40e_ethdev.c +++ b/drivers/net/i40e/i40e_ethdev.c @@ -67,6 +67,7 @@ #include "i40e_pf.h" #include "i40e_regs.h" #include "rte_pmd_i40e.h" +#include "i40e_prep_ops.h" #define ETH_I40E_FLOATING_VEB_ARG "enable_floating_veb" #define ETH_I40E_FLOATING_VEB_LIST_ARG "floating_veb_list" @@ -1122,6 +1123,17 @@ eth_i40e_dev_init(struct rte_eth_dev *dev) hw->bus.func = pci_dev->addr.function; hw->adapter_stopped = 0; + /* init representor broker */ + if (rte_representor_enabled()) { + ret = i40e_port_representor_broker_init(dev, &pf->broker, + pci_dev); + if (ret) { + PMD_INIT_LOG(ERR, "Representor broker register failed " + "with ret=%d\n", ret); + return ret; + } + } + /* Make sure all is clean before doing PF reset */ i40e_clear_hw(hw); @@ -1457,6 +1469,10 @@ eth_i40e_dev_uninit(struct rte_eth_dev *dev) pci_dev = RTE_ETH_DEV_TO_PCI(dev); intr_handle = &pci_dev->intr_handle; + /* free port representor pmds */ + if (rte_representor_enabled()) + rte_representor_broker_uninit(pf->broker); + if (hw->adapter_stopped == 0) i40e_dev_close(dev); diff --git a/drivers/net/i40e/i40e_ethdev.h b/drivers/net/i40e/i40e_ethdev.h index cd67453..9e962eb 100644 --- a/drivers/net/i40e/i40e_ethdev.h +++ b/drivers/net/i40e/i40e_ethdev.h @@ -957,6 +957,7 @@ struct i40e_pf { bool gtp_replace_flag; /* 1 - GTP-C/U filter replace is done */ bool qinq_replace_flag; /* QINQ filter replace is done */ struct i40e_tm_conf tm_conf; + struct rte_representor_broker *broker; /* Dynamic Device Personalization */ bool gtp_support; /* 1 - support GTP-C and GTP-U */ diff --git a/drivers/net/i40e/i40e_prep_ops.c b/drivers/net/i40e/i40e_prep_ops.c new file mode 100644 index 0000000..3261b97 --- /dev/null +++ b/drivers/net/i40e/i40e_prep_ops.c @@ -0,0 +1,472 @@ +/*- + * BSD LICENSE + * + * Copyright(c) 2017 Intel Corporation. All rights reserved. + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in + * the documentation and/or other materials provided with the + * distribution. + * * Neither the name of Intel Corporation nor the names of its + * contributors may be used to endorse or promote products derived + * from this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +#include +#include +#include + +#include "base/i40e_type.h" +#include "base/virtchnl.h" +#include "i40e_ethdev.h" +#include "i40e_rxtx.h" +#include "rte_pmd_i40e.h" + +#include "i40e_prep_ops.h" + +struct i40e_representor_private_data { + struct rte_eth_dev *pf_ethdev; +}; + +static int +i40e_representor_link_update(struct rte_eth_dev *ethdev, int wait_to_complete) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + return i40e_dev_link_update(i40e_priv_data->pf_ethdev, + wait_to_complete); +} + +static void +i40e_representor_dev_infos_get(struct rte_eth_dev *ethdev, + struct rte_eth_dev_info *dev_info) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + struct i40e_pf_vf *vf; + struct i40e_vsi *vsi; + struct i40e_pf *pf; + + if (!is_i40e_supported(i40e_priv_data->pf_ethdev)) { + PMD_DRV_LOG(ERR, "Invalid PF dev."); + return; + } + + pf = I40E_DEV_PRIVATE_TO_PF( + i40e_priv_data->pf_ethdev->data->dev_private); + if (representor->vport_id >= pf->vf_num || !pf->vfs) { + PMD_DRV_LOG(ERR, "Invalid VF ID."); + return; + } + + vf = &pf->vfs[representor->vport_id]; + vsi = vf->vsi; + if (!vsi) { + PMD_DRV_LOG(ERR, "Invalid VSI."); + return; + } + + /* get dev info for the vdev */ + dev_info->pci_dev = RTE_ETH_DEV_TO_PCI(ethdev); + dev_info->max_rx_queues = vsi->nb_qps; + dev_info->max_tx_queues = vsi->nb_qps; + dev_info->min_rx_bufsize = I40E_BUF_SIZE_MIN; + dev_info->max_rx_pktlen = I40E_FRAME_SIZE_MAX; + dev_info->hash_key_size = (I40E_VFQF_HKEY_MAX_INDEX + 1) * + sizeof(uint32_t); + dev_info->reta_size = ETH_RSS_RETA_SIZE_64; + dev_info->flow_type_rss_offloads = I40E_RSS_OFFLOAD_ALL; + dev_info->max_mac_addrs = I40E_NUM_MACADDR_MAX; + dev_info->rx_offload_capa = + DEV_RX_OFFLOAD_VLAN_STRIP | + DEV_RX_OFFLOAD_QINQ_STRIP | + DEV_RX_OFFLOAD_IPV4_CKSUM | + DEV_RX_OFFLOAD_UDP_CKSUM | + DEV_RX_OFFLOAD_TCP_CKSUM; + dev_info->tx_offload_capa = + DEV_TX_OFFLOAD_VLAN_INSERT | + DEV_TX_OFFLOAD_QINQ_INSERT | + DEV_TX_OFFLOAD_IPV4_CKSUM | + DEV_TX_OFFLOAD_UDP_CKSUM | + DEV_TX_OFFLOAD_TCP_CKSUM | + DEV_TX_OFFLOAD_SCTP_CKSUM; + + dev_info->default_rxconf = (struct rte_eth_rxconf) { + .rx_thresh = { + .pthresh = I40E_DEFAULT_RX_PTHRESH, + .hthresh = I40E_DEFAULT_RX_HTHRESH, + .wthresh = I40E_DEFAULT_RX_WTHRESH, + }, + .rx_free_thresh = I40E_DEFAULT_RX_FREE_THRESH, + .rx_drop_en = 0, + }; + + dev_info->default_txconf = (struct rte_eth_txconf) { + .tx_thresh = { + .pthresh = I40E_DEFAULT_TX_PTHRESH, + .hthresh = I40E_DEFAULT_TX_HTHRESH, + .wthresh = I40E_DEFAULT_TX_WTHRESH, + }, + .tx_free_thresh = I40E_DEFAULT_TX_FREE_THRESH, + .tx_rs_thresh = I40E_DEFAULT_TX_RSBIT_THRESH, + .txq_flags = ETH_TXQ_FLAGS_NOMULTSEGS | + ETH_TXQ_FLAGS_NOOFFLOADS, + }; + + dev_info->rx_desc_lim = (struct rte_eth_desc_lim) { + .nb_max = I40E_MAX_RING_DESC, + .nb_min = I40E_MIN_RING_DESC, + .nb_align = I40E_ALIGN_RING_DESC, + }; + + dev_info->tx_desc_lim = (struct rte_eth_desc_lim) { + .nb_max = I40E_MAX_RING_DESC, + .nb_min = I40E_MIN_RING_DESC, + .nb_align = I40E_ALIGN_RING_DESC, + }; +} + +static int +i40e_representor_stats_get(struct rte_eth_dev *ethdev, + struct rte_eth_stats *stats) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + return rte_pmd_i40e_get_vf_stats( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, stats); +} + +static void +i40e_representor_stats_reset(struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_reset_vf_stats(i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id); +} + +static void +i40e_representor_promiscuous_enable(struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_unicast_promisc( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, 1); +} + +static void +i40e_representor_promiscuous_disable(struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_unicast_promisc( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, 0); +} + +static void +i40e_representor_allmulticast_enable(struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_multicast_promisc( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, 1); +} + +static void +i40e_representor_allmulticast_disable(struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_multicast_promisc( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, 0); +} + +static void +i40e_representor_mac_addr_remove(struct rte_eth_dev *ethdev, uint32_t index) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_remove_vf_mac_addr( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, ðdev->data->mac_addrs[index]); +} + +static void +i40e_representor_mac_addr_set(struct rte_eth_dev *ethdev, + struct ether_addr *mac_addr) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_mac_addr( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, mac_addr); +} + +static int +i40e_representor_vlan_filter_set(struct rte_eth_dev *ethdev, + uint16_t vlan_id, int on) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + uint32_t vfid; + uint64_t vf_mask; + + vfid = representor->vport_id; + vf_mask = 1ULL << vfid; + + return rte_pmd_i40e_set_vf_vlan_filter( + i40e_priv_data->pf_ethdev->data->port_id, + vlan_id, vf_mask, on); +} + +static int +i40e_representor_vlan_offload_set(struct rte_eth_dev *ethdev, int mask) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + struct rte_eth_dev *pdev; + struct i40e_pf_vf *vf; + struct i40e_vsi *vsi; + struct i40e_pf *pf; + uint32_t vfid; + + pdev = i40e_priv_data->pf_ethdev; + vfid = representor->vport_id; + + if (!is_i40e_supported(pdev)) { + PMD_DRV_LOG(ERR, "Invalid PF dev."); + return -EINVAL; + } + + pf = I40E_DEV_PRIVATE_TO_PF(pdev->data->dev_private); + + if (vfid >= pf->vf_num || !pf->vfs) { + PMD_DRV_LOG(ERR, "Invalid VF ID."); + return -EINVAL; + } + + vf = &pf->vfs[vfid]; + vsi = vf->vsi; + if (!vsi) { + PMD_DRV_LOG(ERR, "Invalid VSI."); + return -EINVAL; + } + + /* Fixme: even though there 3 kinds of hw offloads, hw_vlan_filter, + * hw_vlan_strip, and hw_vlan_extend. + * currently the hw_vlan_extend offload by vsi is not implemented. + */ + + if (mask & ETH_VLAN_FILTER_MASK) { + /* Enable or disable VLAN filtering offload */ + if (ethdev->data->dev_conf.rxmode.hw_vlan_filter) + return i40e_vsi_config_vlan_filter(vsi, TRUE); + else + return i40e_vsi_config_vlan_filter(vsi, FALSE); + } + + if (mask & ETH_VLAN_STRIP_MASK) { + /* Enable or disable VLAN stripping offload */ + if (ethdev->data->dev_conf.rxmode.hw_vlan_strip) + return i40e_vsi_config_vlan_stripping(vsi, TRUE); + else + return i40e_vsi_config_vlan_stripping(vsi, FALSE); + } + + return -EINVAL; +} + +static void +i40e_representor_vlan_strip_queue_set(struct rte_eth_dev *ethdev, + __rte_unused uint16_t rx_queue_id, int on) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + rte_pmd_i40e_set_vf_vlan_stripq( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, on); +} + +static int +i40e_representor_vlan_pvid_set(struct rte_eth_dev *ethdev, uint16_t vlan_id, + __rte_unused int on) +{ + struct rte_representor_port *representor = ethdev->data->dev_private; + struct i40e_representor_private_data *i40e_priv_data = + representor->priv_data; + + return rte_pmd_i40e_set_vf_vlan_insert( + i40e_priv_data->pf_ethdev->data->port_id, + representor->vport_id, vlan_id); +} + +struct eth_dev_ops i40e_representor_dev_ops = { + .link_update = i40e_representor_link_update, + .dev_infos_get = i40e_representor_dev_infos_get, + + .stats_get = i40e_representor_stats_get, + .stats_reset = i40e_representor_stats_reset, + + .promiscuous_enable = i40e_representor_promiscuous_enable, + .promiscuous_disable = i40e_representor_promiscuous_disable, + + .allmulticast_enable = i40e_representor_allmulticast_enable, + .allmulticast_disable = i40e_representor_allmulticast_disable, + + .mac_addr_remove = i40e_representor_mac_addr_remove, + .mac_addr_set = i40e_representor_mac_addr_set, + + .vlan_filter_set = i40e_representor_vlan_filter_set, + .vlan_offload_set = i40e_representor_vlan_offload_set, + .vlan_strip_queue_set = i40e_representor_vlan_strip_queue_set, + .vlan_pvid_set = i40e_representor_vlan_pvid_set +}; + +static int +i40e_port_representor_init(struct rte_representor_broker *broker, + struct rte_eth_dev *ethdev) +{ + struct rte_eth_dev *pf_ethdev; + struct rte_eth_link *link; + struct rte_representor_port *port; + struct i40e_pf *i40e_pf; + struct i40e_pf_vf *i40e_vf; + + port = ethdev->data->dev_private; + + /** + * Allocate private data for i40e port representor and save the physical + * functions ethdev handle + */ + port->priv_data = rte_zmalloc_socket("i40e_port_representor_priv_data", + sizeof(struct i40e_representor_private_data), + RTE_CACHE_LINE_SIZE, ethdev->device->numa_node); + if (!port->priv_data) + return -ENOMEM; + + pf_ethdev = (struct rte_eth_dev *)broker->private_data; + + ((struct i40e_representor_private_data *)port->priv_data)->pf_ethdev = + pf_ethdev; + + + i40e_pf = I40E_DEV_PRIVATE_TO_PF(pf_ethdev->data->dev_private); + i40e_vf = &i40e_pf->vfs[port->vport_id]; + + if (!i40e_vf->vsi) { + PMD_DRV_LOG(ERR, "Invalid VSI."); + return -EINVAL; + } + + /* Set representor device ops */ + ethdev->dev_ops = &i40e_representor_dev_ops; + + /* Setting the number queues allocated to the VF */ + ethdev->data->nb_rx_queues = i40e_vf->vsi->nb_qps; + ethdev->data->nb_tx_queues = i40e_vf->vsi->nb_qps; + + /* Link state. Inherited from PF */ + link = &pf_ethdev->data->dev_link; + + ethdev->data->dev_link.link_speed = link->link_speed; + ethdev->data->dev_link.link_duplex = link->link_duplex; + ethdev->data->dev_link.link_status = link->link_status; + ethdev->data->dev_link.link_autoneg = link->link_autoneg; + + /* No data-path so no RX/TX functions */ + ethdev->rx_pkt_burst = NULL; + ethdev->tx_pkt_burst = NULL; + + return 0; +} + +static int +i40e_port_representor_uninit(struct rte_representor_broker *broker __rte_unused, + struct rte_eth_dev *ethdev) +{ + struct rte_representor_port *port = ethdev->data->dev_private; + + rte_free(port->priv_data); + + return 0; +} + +struct rte_representor_broker_port_ops i40e_broker_port_ops = { + .port_init = i40e_port_representor_init, + .port_uninit = i40e_port_representor_uninit +}; + +int +i40e_port_representor_broker_init(struct rte_eth_dev *dev, + struct rte_representor_broker **broker, + const struct rte_pci_device *pci_dev) +{ + struct rte_bus *bus; + + *broker = rte_zmalloc_socket("rte_port_representor_broker", + sizeof(**broker), RTE_CACHE_LINE_SIZE, + rte_socket_id()); + if (!*broker) { + RTE_LOG(ERR, EAL, "Not enough memory for representor " + "broker\n"); + return -ENOMEM; + } + + bus = rte_bus_find_by_device(&pci_dev->device); + + /* Set i40e broker parameters */ + (*broker)->bus = bus->name; + (*broker)->device = pci_dev->name; + (*broker)->nb_virtual_ports = pci_dev->max_vfs; + (*broker)->ops = &i40e_broker_port_ops; + (*broker)->private_data = dev; + + return rte_representor_broker_init(*broker); +} diff --git a/drivers/net/i40e/i40e_prep_ops.h b/drivers/net/i40e/i40e_prep_ops.h new file mode 100644 index 0000000..c155da8 --- /dev/null +++ b/drivers/net/i40e/i40e_prep_ops.h @@ -0,0 +1,46 @@ +/*- + * BSD LICENSE + * + * Copyright(c) 2017 Intel Corporation. All rights reserved. + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in + * the documentation and/or other materials provided with the + * distribution. + * * Neither the name of Intel Corporation nor the names of its + * contributors may be used to endorse or promote products derived + * from this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +#ifndef _I40E_PREP_OPS_H_ +#define _I40E_PREP_OPS_H_ + +#include + +struct rte_representor_broker_port_ops *rte_i40e_broker_port_ops; + +int +i40e_port_representor_broker_init(struct rte_eth_dev *dev, + struct rte_representor_broker **broker, + const struct rte_pci_device *pci_dev); + +#endif /* _I40E_PREP_OPS_H_ */ diff --git a/drivers/net/i40e/rte_pmd_i40e.c b/drivers/net/i40e/rte_pmd_i40e.c index aeb92af..641a35c 100644 --- a/drivers/net/i40e/rte_pmd_i40e.c +++ b/drivers/net/i40e/rte_pmd_i40e.c @@ -599,6 +599,53 @@ rte_pmd_i40e_set_vf_mac_addr(uint16_t port, uint16_t vf_id, return 0; } +static const struct ether_addr null_mac_addr; + +int +rte_pmd_i40e_remove_vf_mac_addr(uint8_t port, uint16_t vf_id, + struct ether_addr *mac_addr) +{ + struct rte_eth_dev *dev; + struct i40e_pf_vf *vf; + struct i40e_vsi *vsi; + struct i40e_pf *pf; + + if (i40e_validate_mac_addr((u8 *)mac_addr) != I40E_SUCCESS) + return -EINVAL; + + RTE_ETH_VALID_PORTID_OR_ERR_RET(port, -ENODEV); + + dev = &rte_eth_devices[port]; + + if (!is_i40e_supported(dev)) + return -ENOTSUP; + + pf = I40E_DEV_PRIVATE_TO_PF(dev->data->dev_private); + + if (vf_id >= pf->vf_num || !pf->vfs) + return -EINVAL; + + vf = &pf->vfs[vf_id]; + vsi = vf->vsi; + if (!vsi) { + PMD_DRV_LOG(ERR, "Invalid VSI."); + return -EINVAL; + } + + if (!is_same_ether_addr(mac_addr, &vf->mac_addr)) { + PMD_DRV_LOG(ERR, "Mac address does not match."); + return -EINVAL; + } + + /* reset the mac with null mac */ + ether_addr_copy(&null_mac_addr, &vf->mac_addr); + + /* Remove the mac */ + i40e_vsi_delete_mac(vsi, mac_addr); + + return 0; +} + /* Set vlan strip on/off for specific VF from host */ int rte_pmd_i40e_set_vf_vlan_stripq(uint16_t port, uint16_t vf_id, uint8_t on) diff --git a/drivers/net/i40e/rte_pmd_i40e.h b/drivers/net/i40e/rte_pmd_i40e.h index 580ca4a..1b1f725 100644 --- a/drivers/net/i40e/rte_pmd_i40e.h +++ b/drivers/net/i40e/rte_pmd_i40e.h @@ -467,6 +467,24 @@ int rte_pmd_i40e_set_vf_mac_addr(uint16_t port, uint16_t vf_id, struct ether_addr *mac_addr); /** + * Remove the VF MAC address. + * + * @param port + * The port identifier of the Ethernet device. + * @param vf_id + * VF id. + * @param mac_addr + * VF MAC address. + * @return + * - (0) if successful. + * - (-ENODEV) if *port* invalid. + * - (-EINVAL) if *vf* or *mac_addr* is invalid. + */ +int +rte_pmd_i40e_remove_vf_mac_addr(uint8_t port, uint16_t vf_id, + struct ether_addr *mac_addr); + +/** * Enable/Disable vf vlan strip for all queues in a pool * * @param port -- 2.7.4