net/nfp: move VF functions into its own file
[dpdk.git] / drivers / net / nfp / nfp_net.c
index fa8ff3c..a46c4dd 100644 (file)
@@ -1,34 +1,8 @@
-/*
- * Copyright (c) 2014, 2015 Netronome Systems, Inc.
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright (c) 2014-2018 Netronome Systems, Inc.
  * All rights reserved.
  *
  * Small portions derived from code Copyright(c) 2010-2015 Intel Corporation.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- *
- * 1. Redistributions of source code must retain the above copyright notice,
- *  this list of conditions and the following disclaimer.
- *
- * 2. 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
- *
- * 3. Neither the name of the copyright holder 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 HOLDER 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.
  */
 
 /*
@@ -43,8 +17,8 @@
 #include <rte_common.h>
 #include <rte_log.h>
 #include <rte_debug.h>
-#include <rte_ethdev.h>
-#include <rte_ethdev_pci.h>
+#include <ethdev_driver.h>
+#include <ethdev_pci.h>
 #include <rte_dev.h>
 #include <rte_ether.h>
 #include <rte_malloc.h>
 #include <rte_string_fns.h>
 #include <rte_alarm.h>
 #include <rte_spinlock.h>
+#include <rte_service_component.h>
+
+#include "eal_firmware.h"
+
+#include "nfpcore/nfp_cpp.h"
+#include "nfpcore/nfp_nffw.h"
+#include "nfpcore/nfp_hwinfo.h"
+#include "nfpcore/nfp_mip.h"
+#include "nfpcore/nfp_rtsym.h"
+#include "nfpcore/nfp_nsp.h"
 
-#include "nfp_nfpu.h"
 #include "nfp_net_pmd.h"
+#include "nfp_rxtx.h"
 #include "nfp_net_logs.h"
 #include "nfp_net_ctrl.h"
+#include "nfp_cpp_bridge.h"
+
+#include <sys/types.h>
+#include <sys/socket.h>
+#include <sys/un.h>
+#include <unistd.h>
+#include <stdio.h>
+#include <sys/ioctl.h>
+#include <errno.h>
 
 /* Prototypes */
-static void nfp_net_close(struct rte_eth_dev *dev);
-static int nfp_net_configure(struct rte_eth_dev *dev);
-static void nfp_net_dev_interrupt_handler(void *param);
-static void nfp_net_dev_interrupt_delayed_handler(void *param);
-static int nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu);
-static void nfp_net_infos_get(struct rte_eth_dev *dev,
-                             struct rte_eth_dev_info *dev_info);
+static int nfp_net_close(struct rte_eth_dev *dev);
 static int nfp_net_init(struct rte_eth_dev *eth_dev);
-static int nfp_net_link_update(struct rte_eth_dev *dev, int wait_to_complete);
-static void nfp_net_promisc_enable(struct rte_eth_dev *dev);
-static void nfp_net_promisc_disable(struct rte_eth_dev *dev);
-static int nfp_net_rx_fill_freelist(struct nfp_net_rxq *rxq);
-static uint32_t nfp_net_rx_queue_count(struct rte_eth_dev *dev,
-                                      uint16_t queue_idx);
-static uint16_t nfp_net_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
-                                 uint16_t nb_pkts);
-static void nfp_net_rx_queue_release(void *rxq);
-static int nfp_net_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
-                                 uint16_t nb_desc, unsigned int socket_id,
-                                 const struct rte_eth_rxconf *rx_conf,
-                                 struct rte_mempool *mp);
-static int nfp_net_tx_free_bufs(struct nfp_net_txq *txq);
-static void nfp_net_tx_queue_release(void *txq);
-static int nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
-                                 uint16_t nb_desc, unsigned int socket_id,
-                                 const struct rte_eth_txconf *tx_conf);
-static int nfp_net_start(struct rte_eth_dev *dev);
-static int nfp_net_stats_get(struct rte_eth_dev *dev,
-                             struct rte_eth_stats *stats);
-static void nfp_net_stats_reset(struct rte_eth_dev *dev);
-static void nfp_net_stop(struct rte_eth_dev *dev);
-static uint16_t nfp_net_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts,
-                                 uint16_t nb_pkts);
-
-/*
- * The offset of the queue controller queues in the PCIe Target. These
- * happen to be at the same offset on the NFP6000 and the NFP3200 so
- * we use a single macro here.
- */
-#define NFP_PCIE_QUEUE(_q)     (0x800 * ((_q) & 0xff))
-
-/* Maximum value which can be added to a queue with one transaction */
-#define NFP_QCP_MAX_ADD        0x7f
-
-#define RTE_MBUF_DMA_ADDR_DEFAULT(mb) \
-       (uint64_t)((mb)->buf_iova + RTE_PKTMBUF_HEADROOM)
-
-/* nfp_qcp_ptr - Read or Write Pointer of a queue */
-enum nfp_qcp_ptr {
-       NFP_QCP_READ_PTR = 0,
-       NFP_QCP_WRITE_PTR
-};
-
-/*
- * nfp_qcp_ptr_add - Add the value to the selected pointer of a queue
- * @q: Base address for queue structure
- * @ptr: Add to the Read or Write pointer
- * @val: Value to add to the queue pointer
- *
- * If @val is greater than @NFP_QCP_MAX_ADD multiple writes are performed.
- */
-static inline void
-nfp_qcp_ptr_add(uint8_t *q, enum nfp_qcp_ptr ptr, uint32_t val)
-{
-       uint32_t off;
-
-       if (ptr == NFP_QCP_READ_PTR)
-               off = NFP_QCP_QUEUE_ADD_RPTR;
-       else
-               off = NFP_QCP_QUEUE_ADD_WPTR;
-
-       while (val > NFP_QCP_MAX_ADD) {
-               nn_writel(rte_cpu_to_le_32(NFP_QCP_MAX_ADD), q + off);
-               val -= NFP_QCP_MAX_ADD;
-       }
-
-       nn_writel(rte_cpu_to_le_32(val), q + off);
-}
-
-/*
- * nfp_qcp_read - Read the current Read/Write pointer value for a queue
- * @q:  Base address for queue structure
- * @ptr: Read or Write pointer
- */
-static inline uint32_t
-nfp_qcp_read(uint8_t *q, enum nfp_qcp_ptr ptr)
-{
-       uint32_t off;
-       uint32_t val;
-
-       if (ptr == NFP_QCP_READ_PTR)
-               off = NFP_QCP_QUEUE_STS_LO;
-       else
-               off = NFP_QCP_QUEUE_STS_HI;
-
-       val = rte_cpu_to_le_32(nn_readl(q + off));
-
-       if (ptr == NFP_QCP_READ_PTR)
-               return val & NFP_QCP_QUEUE_STS_LO_READPTR_mask;
-       else
-               return val & NFP_QCP_QUEUE_STS_HI_WRITEPTR_mask;
-}
-
-/*
- * Functions to read/write from/to Config BAR
- * Performs any endian conversion necessary.
- */
-static inline uint8_t
-nn_cfg_readb(struct nfp_net_hw *hw, int off)
-{
-       return nn_readb(hw->ctrl_bar + off);
-}
-
-static inline void
-nn_cfg_writeb(struct nfp_net_hw *hw, int off, uint8_t val)
-{
-       nn_writeb(val, hw->ctrl_bar + off);
-}
-
-static inline uint32_t
-nn_cfg_readl(struct nfp_net_hw *hw, int off)
-{
-       return rte_le_to_cpu_32(nn_readl(hw->ctrl_bar + off));
-}
-
-static inline void
-nn_cfg_writel(struct nfp_net_hw *hw, int off, uint32_t val)
-{
-       nn_writel(rte_cpu_to_le_32(val), hw->ctrl_bar + off);
-}
-
-static inline uint64_t
-nn_cfg_readq(struct nfp_net_hw *hw, int off)
-{
-       return rte_le_to_cpu_64(nn_readq(hw->ctrl_bar + off));
-}
-
-static inline void
-nn_cfg_writeq(struct nfp_net_hw *hw, int off, uint64_t val)
-{
-       nn_writeq(rte_cpu_to_le_64(val), hw->ctrl_bar + off);
-}
-
-/*
- * Atomically reads link status information from global structure rte_eth_dev.
- *
- * @param dev
- *   - Pointer to the structure rte_eth_dev to read from.
- *   - Pointer to the buffer to be saved with the link status.
- *
- * @return
- *   - On success, zero.
- *   - On failure, negative value.
- */
-static inline int
-nfp_net_dev_atomic_read_link_status(struct rte_eth_dev *dev,
-                                   struct rte_eth_link *link)
-{
-       struct rte_eth_link *dst = link;
-       struct rte_eth_link *src = &dev->data->dev_link;
-
-       if (rte_atomic64_cmpset((uint64_t *)dst, *(uint64_t *)dst,
-                               *(uint64_t *)src) == 0)
-               return -1;
-
-       return 0;
-}
-
-/*
- * Atomically writes the link status information into global
- * structure rte_eth_dev.
- *
- * @param dev
- *   - Pointer to the structure rte_eth_dev to read from.
- *   - Pointer to the buffer to be saved with the link status.
- *
- * @return
- *   - On success, zero.
- *   - On failure, negative value.
- */
-static inline int
-nfp_net_dev_atomic_write_link_status(struct rte_eth_dev *dev,
-                                    struct rte_eth_link *link)
-{
-       struct rte_eth_link *dst = &dev->data->dev_link;
-       struct rte_eth_link *src = link;
-
-       if (rte_atomic64_cmpset((uint64_t *)dst, *(uint64_t *)dst,
-                               *(uint64_t *)src) == 0)
-               return -1;
-
-       return 0;
-}
-
-static void
-nfp_net_rx_queue_release_mbufs(struct nfp_net_rxq *rxq)
-{
-       unsigned i;
-
-       if (rxq->rxbufs == NULL)
-               return;
-
-       for (i = 0; i < rxq->rx_count; i++) {
-               if (rxq->rxbufs[i].mbuf) {
-                       rte_pktmbuf_free_seg(rxq->rxbufs[i].mbuf);
-                       rxq->rxbufs[i].mbuf = NULL;
-               }
-       }
-}
-
-static void
-nfp_net_rx_queue_release(void *rx_queue)
-{
-       struct nfp_net_rxq *rxq = rx_queue;
-
-       if (rxq) {
-               nfp_net_rx_queue_release_mbufs(rxq);
-               rte_free(rxq->rxbufs);
-               rte_free(rxq);
-       }
-}
-
-static void
-nfp_net_reset_rx_queue(struct nfp_net_rxq *rxq)
-{
-       nfp_net_rx_queue_release_mbufs(rxq);
-       rxq->rd_p = 0;
-       rxq->nb_rx_hold = 0;
-}
-
-static void
-nfp_net_tx_queue_release_mbufs(struct nfp_net_txq *txq)
-{
-       unsigned i;
-
-       if (txq->txbufs == NULL)
-               return;
-
-       for (i = 0; i < txq->tx_count; i++) {
-               if (txq->txbufs[i].mbuf) {
-                       rte_pktmbuf_free(txq->txbufs[i].mbuf);
-                       txq->txbufs[i].mbuf = NULL;
-               }
-       }
-}
-
-static void
-nfp_net_tx_queue_release(void *tx_queue)
-{
-       struct nfp_net_txq *txq = tx_queue;
-
-       if (txq) {
-               nfp_net_tx_queue_release_mbufs(txq);
-               rte_free(txq->txbufs);
-               rte_free(txq);
-       }
-}
-
-static void
-nfp_net_reset_tx_queue(struct nfp_net_txq *txq)
-{
-       nfp_net_tx_queue_release_mbufs(txq);
-       txq->wr_p = 0;
-       txq->rd_p = 0;
-}
+static int nfp_pf_init(struct rte_pci_device *pci_dev);
+static int nfp_pf_secondary_init(struct rte_pci_device *pci_dev);
+static int nfp_net_pf_read_mac(struct nfp_pf_dev *pf_dev, int port);
+static int nfp_pci_uninit(struct rte_eth_dev *eth_dev);
+static int nfp_init_phyports(struct nfp_pf_dev *pf_dev);
+static int nfp_net_stop(struct rte_eth_dev *dev);
+static int nfp_fw_setup(struct rte_pci_device *dev,
+                       struct nfp_cpp *cpp,
+                       struct nfp_eth_table *nfp_eth_table,
+                       struct nfp_hwinfo *hwinfo);
 
 static int
 __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update)
@@ -334,7 +74,7 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update)
        uint32_t new;
        struct timespec wait;
 
-       PMD_DRV_LOG(DEBUG, "Writing to the configuration queue (%p)...\n",
+       PMD_DRV_LOG(DEBUG, "Writing to the configuration queue (%p)...",
                    hw->qcp_cfg);
 
        if (hw->qcp_cfg == NULL)
@@ -345,7 +85,7 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update)
        wait.tv_sec = 0;
        wait.tv_nsec = 1000000;
 
-       PMD_DRV_LOG(DEBUG, "Polling for update ack...\n");
+       PMD_DRV_LOG(DEBUG, "Polling for update ack...");
 
        /* Poll update field, waiting for NFP to ack the config */
        for (cnt = 0; ; cnt++) {
@@ -363,7 +103,7 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update)
                }
                nanosleep(&wait, 0); /* waiting for a 1ms */
        }
-       PMD_DRV_LOG(DEBUG, "Ack DONE\n");
+       PMD_DRV_LOG(DEBUG, "Ack DONE");
        return 0;
 }
 
@@ -376,12 +116,12 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update)
  * Write the update word to the BAR and ping the reconfig queue. Then poll
  * until the firmware has acknowledged the update by zeroing the update word.
  */
-static int
+int
 nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t ctrl, uint32_t update)
 {
        uint32_t err;
 
-       PMD_DRV_LOG(DEBUG, "nfp_net_reconfig: ctrl=%08x update=%08x\n",
+       PMD_DRV_LOG(DEBUG, "nfp_net_reconfig: ctrl=%08x update=%08x",
                    ctrl, update);
 
        rte_spinlock_lock(&hw->reconfig_lock);
@@ -412,14 +152,12 @@ nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t ctrl, uint32_t update)
  * before any other function in the Ethernet API. This function can
  * also be re-invoked when a device is in the stopped state.
  */
-static int
+int
 nfp_net_configure(struct rte_eth_dev *dev)
 {
        struct rte_eth_conf *dev_conf;
        struct rte_eth_rxmode *rxmode;
        struct rte_eth_txmode *txmode;
-       uint32_t new_ctrl = 0;
-       uint32_t update = 0;
        struct nfp_net_hw *hw;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
@@ -438,6 +176,9 @@ nfp_net_configure(struct rte_eth_dev *dev)
        rxmode = &dev_conf->rxmode;
        txmode = &dev_conf->txmode;
 
+       if (rxmode->mq_mode & ETH_MQ_RX_RSS_FLAG)
+               rxmode->offloads |= DEV_RX_OFFLOAD_RSS_HASH;
+
        /* Checking TX mode */
        if (txmode->mq_mode) {
                PMD_INIT_LOG(INFO, "TX mq_mode DCB and VMDq not supported");
@@ -445,101 +186,16 @@ nfp_net_configure(struct rte_eth_dev *dev)
        }
 
        /* Checking RX mode */
-       if (rxmode->mq_mode & ETH_MQ_RX_RSS) {
-               if (hw->cap & NFP_NET_CFG_CTRL_RSS) {
-                       update = NFP_NET_CFG_UPDATE_RSS;
-                       new_ctrl = NFP_NET_CFG_CTRL_RSS;
-               } else {
-                       PMD_INIT_LOG(INFO, "RSS not supported");
-                       return -EINVAL;
-               }
-       }
-
-       if (rxmode->split_hdr_size) {
-               PMD_INIT_LOG(INFO, "rxmode does not support split header");
-               return -EINVAL;
-       }
-
-       if (rxmode->hw_ip_checksum) {
-               if (hw->cap & NFP_NET_CFG_CTRL_RXCSUM) {
-                       new_ctrl |= NFP_NET_CFG_CTRL_RXCSUM;
-               } else {
-                       PMD_INIT_LOG(INFO, "RXCSUM not supported");
-                       return -EINVAL;
-               }
-       }
-
-       if (rxmode->hw_vlan_filter) {
-               PMD_INIT_LOG(INFO, "VLAN filter not supported");
-               return -EINVAL;
-       }
-
-       if (rxmode->hw_vlan_strip) {
-               if (hw->cap & NFP_NET_CFG_CTRL_RXVLAN) {
-                       new_ctrl |= NFP_NET_CFG_CTRL_RXVLAN;
-               } else {
-                       PMD_INIT_LOG(INFO, "hw vlan strip not supported");
-                       return -EINVAL;
-               }
-       }
-
-       if (rxmode->hw_vlan_extend) {
-               PMD_INIT_LOG(INFO, "VLAN extended not supported");
-               return -EINVAL;
-       }
-
-       if (rxmode->jumbo_frame)
-               hw->mtu = rxmode->max_rx_pkt_len;
-
-       if (!rxmode->hw_strip_crc)
-               PMD_INIT_LOG(INFO, "HW does strip CRC and it is not configurable");
-
-       if (rxmode->enable_scatter) {
-               PMD_INIT_LOG(INFO, "Scatter not supported");
+       if (rxmode->mq_mode & ETH_MQ_RX_RSS &&
+           !(hw->cap & NFP_NET_CFG_CTRL_RSS)) {
+               PMD_INIT_LOG(INFO, "RSS not supported");
                return -EINVAL;
        }
 
-       /* If next capabilities are supported, configure them by default */
-
-       /* VLAN insertion */
-       if (hw->cap & NFP_NET_CFG_CTRL_TXVLAN)
-               new_ctrl |= NFP_NET_CFG_CTRL_TXVLAN;
-
-       /* L2 broadcast */
-       if (hw->cap & NFP_NET_CFG_CTRL_L2BC)
-               new_ctrl |= NFP_NET_CFG_CTRL_L2BC;
-
-       /* L2 multicast */
-       if (hw->cap & NFP_NET_CFG_CTRL_L2MC)
-               new_ctrl |= NFP_NET_CFG_CTRL_L2MC;
-
-       /* TX checksum offload */
-       if (hw->cap & NFP_NET_CFG_CTRL_TXCSUM)
-               new_ctrl |= NFP_NET_CFG_CTRL_TXCSUM;
-
-       /* LSO offload */
-       if (hw->cap & NFP_NET_CFG_CTRL_LSO)
-               new_ctrl |= NFP_NET_CFG_CTRL_LSO;
-
-       /* RX gather */
-       if (hw->cap & NFP_NET_CFG_CTRL_GATHER)
-               new_ctrl |= NFP_NET_CFG_CTRL_GATHER;
-
-       if (!new_ctrl)
-               return 0;
-
-       update |= NFP_NET_CFG_UPDATE_GEN;
-
-       nn_cfg_writel(hw, NFP_NET_CFG_CTRL, new_ctrl);
-       if (nfp_net_reconfig(hw, new_ctrl, update) < 0)
-               return -EIO;
-
-       hw->ctrl = new_ctrl;
-
        return 0;
 }
 
-static void
+void
 nfp_net_enable_queues(struct rte_eth_dev *dev)
 {
        struct nfp_net_hw *hw;
@@ -563,7 +219,7 @@ nfp_net_enable_queues(struct rte_eth_dev *dev)
        nn_cfg_writeq(hw, NFP_NET_CFG_RXRS_ENABLE, enabled_queues);
 }
 
-static void
+void
 nfp_net_disable_queues(struct rte_eth_dev *dev)
 {
        struct nfp_net_hw *hw;
@@ -588,26 +244,14 @@ nfp_net_disable_queues(struct rte_eth_dev *dev)
        hw->ctrl = new_ctrl;
 }
 
-static int
-nfp_net_rx_freelist_setup(struct rte_eth_dev *dev)
-{
-       int i;
-
-       for (i = 0; i < dev->data->nb_rx_queues; i++) {
-               if (nfp_net_rx_fill_freelist(dev->data->rx_queues[i]) < 0)
-                       return -1;
-       }
-       return 0;
-}
-
-static void
+void
 nfp_net_params_setup(struct nfp_net_hw *hw)
 {
        nn_cfg_writel(hw, NFP_NET_CFG_MTU, hw->mtu);
        nn_cfg_writel(hw, NFP_NET_CFG_FLBUFSZ, hw->flbufsz);
 }
 
-static void
+void
 nfp_net_cfg_queue_setup(struct nfp_net_hw *hw)
 {
        hw->qcp_cfg = hw->tx_bar + NFP_QCP_QUEUE_ADDR_SZ;
@@ -615,64 +259,34 @@ nfp_net_cfg_queue_setup(struct nfp_net_hw *hw)
 
 #define ETH_ADDR_LEN   6
 
-static void
-nfp_eth_copy_mac_reverse(uint8_t *dst, const uint8_t *src)
+void
+nfp_eth_copy_mac(uint8_t *dst, const uint8_t *src)
 {
        int i;
 
        for (i = 0; i < ETH_ADDR_LEN; i++)
-               dst[ETH_ADDR_LEN - i - 1] = src[i];
+               dst[i] = src[i];
 }
 
 static int
-nfp_net_pf_read_mac(struct nfp_net_hw *hw, int port)
+nfp_net_pf_read_mac(struct nfp_pf_dev *pf_dev, int port)
 {
-       union eth_table_entry *entry;
-       int idx, i;
-
-       idx = port;
-       entry = hw->eth_table;
+       struct nfp_eth_table *nfp_eth_table;
+       struct nfp_net_hw *hw = NULL;
 
-       /* Reading NFP ethernet table obtained before */
-       for (i = 0; i < NSP_ETH_MAX_COUNT; i++) {
-               if (!(entry->port & NSP_ETH_PORT_LANES_MASK)) {
-                       /* port not in use */
-                       entry++;
-                       continue;
-               }
-               if (idx == 0)
-                       break;
-               idx--;
-               entry++;
-       }
+       /* Grab a pointer to the correct physical port */
+       hw = pf_dev->ports[port];
 
-       if (i == NSP_ETH_MAX_COUNT)
-               return -EINVAL;
+       nfp_eth_table = nfp_eth_read_ports(pf_dev->cpp);
 
-       /*
-        * hw points to port0 private data. We need hw now pointing to
-        * right port.
-        */
-       hw += port;
-       nfp_eth_copy_mac_reverse((uint8_t *)&hw->mac_addr,
-                                (uint8_t *)&entry->mac_addr);
+       nfp_eth_copy_mac((uint8_t *)&hw->mac_addr,
+                        (uint8_t *)&nfp_eth_table->ports[port].mac_addr);
 
+       free(nfp_eth_table);
        return 0;
 }
 
-static void
-nfp_net_vf_read_mac(struct nfp_net_hw *hw)
-{
-       uint32_t tmp;
-
-       tmp = rte_be_to_cpu_32(nn_cfg_readl(hw, NFP_NET_CFG_MACADDR));
-       memcpy(&hw->mac_addr[0], &tmp, sizeof(struct ether_addr));
-
-       tmp = rte_be_to_cpu_32(nn_cfg_readl(hw, NFP_NET_CFG_MACADDR + 4));
-       memcpy(&hw->mac_addr[4], &tmp, 2);
-}
-
-static void
+void
 nfp_net_write_mac(struct nfp_net_hw *hw, uint8_t *mac)
 {
        uint32_t mac0 = *(uint32_t *)mac;
@@ -686,7 +300,41 @@ nfp_net_write_mac(struct nfp_net_hw *hw, uint8_t *mac)
                  hw->ctrl_bar + NFP_NET_CFG_MACADDR + 6);
 }
 
-static int
+int
+nfp_set_mac_addr(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr)
+{
+       struct nfp_net_hw *hw;
+       uint32_t update, ctrl;
+
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       if ((hw->ctrl & NFP_NET_CFG_CTRL_ENABLE) &&
+           !(hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR)) {
+               PMD_INIT_LOG(INFO, "MAC address unable to change when"
+                                 " port enabled");
+               return -EBUSY;
+       }
+
+       if ((hw->ctrl & NFP_NET_CFG_CTRL_ENABLE) &&
+           !(hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR))
+               return -EBUSY;
+
+       /* Writing new MAC to the specific port BAR address */
+       nfp_net_write_mac(hw, (uint8_t *)mac_addr);
+
+       /* Signal the NIC about the change */
+       update = NFP_NET_CFG_UPDATE_MACADDR;
+       ctrl = hw->ctrl;
+       if ((hw->ctrl & NFP_NET_CFG_CTRL_ENABLE) &&
+           (hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR))
+               ctrl |= NFP_NET_CFG_CTRL_LIVE_ADDR;
+       if (nfp_net_reconfig(hw, ctrl, update) < 0) {
+               PMD_INIT_LOG(INFO, "MAC address update failed");
+               return -EIO;
+       }
+       return 0;
+}
+
+int
 nfp_configure_rx_interrupt(struct rte_eth_dev *dev,
                           struct rte_intr_handle *intr_handle)
 {
@@ -720,7 +368,7 @@ nfp_configure_rx_interrupt(struct rte_eth_dev *dev,
                        */
                        nn_cfg_writeb(hw, NFP_NET_CFG_RXR_VEC(i), i + 1);
                        intr_handle->intr_vec[i] = i + 1;
-                       PMD_INIT_LOG(DEBUG, "intr_vec[%d]= %d\n", i,
+                       PMD_INIT_LOG(DEBUG, "intr_vec[%d]= %d", i,
                                            intr_handle->intr_vec[i]);
                }
        }
@@ -730,42 +378,103 @@ nfp_configure_rx_interrupt(struct rte_eth_dev *dev,
        return 0;
 }
 
-static int
-nfp_net_start(struct rte_eth_dev *dev)
+uint32_t
+nfp_check_offloads(struct rte_eth_dev *dev)
 {
-       struct rte_pci_device *pci_dev = RTE_ETH_DEV_TO_PCI(dev);
-       struct rte_intr_handle *intr_handle = &pci_dev->intr_handle;
-       uint32_t new_ctrl, update = 0;
        struct nfp_net_hw *hw;
-       uint32_t intr_vector;
-       int ret;
+       struct rte_eth_conf *dev_conf;
+       struct rte_eth_rxmode *rxmode;
+       struct rte_eth_txmode *txmode;
+       uint32_t ctrl = 0;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
-       PMD_INIT_LOG(DEBUG, "Start");
+       dev_conf = &dev->data->dev_conf;
+       rxmode = &dev_conf->rxmode;
+       txmode = &dev_conf->txmode;
 
-       /* Disabling queues just in case... */
-       nfp_net_disable_queues(dev);
+       if (rxmode->offloads & DEV_RX_OFFLOAD_IPV4_CKSUM) {
+               if (hw->cap & NFP_NET_CFG_CTRL_RXCSUM)
+                       ctrl |= NFP_NET_CFG_CTRL_RXCSUM;
+       }
 
-       /* Writing configuration parameters in the device */
-       nfp_net_params_setup(hw);
+       if (rxmode->offloads & DEV_RX_OFFLOAD_VLAN_STRIP) {
+               if (hw->cap & NFP_NET_CFG_CTRL_RXVLAN)
+                       ctrl |= NFP_NET_CFG_CTRL_RXVLAN;
+       }
 
-       /* Enabling the required queues in the device */
-       nfp_net_enable_queues(dev);
+       if (rxmode->offloads & DEV_RX_OFFLOAD_JUMBO_FRAME)
+               hw->mtu = rxmode->max_rx_pkt_len;
 
-       /* check and configure queue intr-vector mapping */
-       if (dev->data->dev_conf.intr_conf.rxq != 0) {
-               if (hw->pf_multiport_enabled) {
-                       PMD_INIT_LOG(ERR, "PMD rx interrupt is not supported "
-                                         "with NFP multiport PF");
-                               return -EINVAL;
-               }
-               if (intr_handle->type == RTE_INTR_HANDLE_UIO) {
-                       /*
-                        * Better not to share LSC with RX interrupts.
-                        * Unregistering LSC interrupt handler
-                        */
-                       rte_intr_callback_unregister(&pci_dev->intr_handle,
+       if (txmode->offloads & DEV_TX_OFFLOAD_VLAN_INSERT)
+               ctrl |= NFP_NET_CFG_CTRL_TXVLAN;
+
+       /* L2 broadcast */
+       if (hw->cap & NFP_NET_CFG_CTRL_L2BC)
+               ctrl |= NFP_NET_CFG_CTRL_L2BC;
+
+       /* L2 multicast */
+       if (hw->cap & NFP_NET_CFG_CTRL_L2MC)
+               ctrl |= NFP_NET_CFG_CTRL_L2MC;
+
+       /* TX checksum offload */
+       if (txmode->offloads & DEV_TX_OFFLOAD_IPV4_CKSUM ||
+           txmode->offloads & DEV_TX_OFFLOAD_UDP_CKSUM ||
+           txmode->offloads & DEV_TX_OFFLOAD_TCP_CKSUM)
+               ctrl |= NFP_NET_CFG_CTRL_TXCSUM;
+
+       /* LSO offload */
+       if (txmode->offloads & DEV_TX_OFFLOAD_TCP_TSO) {
+               if (hw->cap & NFP_NET_CFG_CTRL_LSO)
+                       ctrl |= NFP_NET_CFG_CTRL_LSO;
+               else
+                       ctrl |= NFP_NET_CFG_CTRL_LSO2;
+       }
+
+       /* RX gather */
+       if (txmode->offloads & DEV_TX_OFFLOAD_MULTI_SEGS)
+               ctrl |= NFP_NET_CFG_CTRL_GATHER;
+
+       return ctrl;
+}
+
+static int
+nfp_net_start(struct rte_eth_dev *dev)
+{
+       struct rte_pci_device *pci_dev = RTE_ETH_DEV_TO_PCI(dev);
+       struct rte_intr_handle *intr_handle = &pci_dev->intr_handle;
+       uint32_t new_ctrl, update = 0;
+       struct nfp_net_hw *hw;
+       struct nfp_pf_dev *pf_dev;
+       struct rte_eth_conf *dev_conf;
+       struct rte_eth_rxmode *rxmode;
+       uint32_t intr_vector;
+       int ret;
+
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       pf_dev = NFP_NET_DEV_PRIVATE_TO_PF(dev->data->dev_private);
+
+       PMD_INIT_LOG(DEBUG, "Start");
+
+       /* Disabling queues just in case... */
+       nfp_net_disable_queues(dev);
+
+       /* Enabling the required queues in the device */
+       nfp_net_enable_queues(dev);
+
+       /* check and configure queue intr-vector mapping */
+       if (dev->data->dev_conf.intr_conf.rxq != 0) {
+               if (pf_dev->multiport) {
+                       PMD_INIT_LOG(ERR, "PMD rx interrupt is not supported "
+                                         "with NFP multiport PF");
+                               return -EINVAL;
+               }
+               if (intr_handle->type == RTE_INTR_HANDLE_UIO) {
+                       /*
+                        * Better not to share LSC with RX interrupts.
+                        * Unregistering LSC interrupt handler
+                        */
+                       rte_intr_callback_unregister(&pci_dev->intr_handle,
                                nfp_net_dev_interrupt_handler, (void *)dev);
 
                        if (dev->data->nb_rx_queues > 1) {
@@ -784,8 +493,22 @@ nfp_net_start(struct rte_eth_dev *dev)
 
        rte_intr_enable(intr_handle);
 
+       new_ctrl = nfp_check_offloads(dev);
+
+       /* Writing configuration parameters in the device */
+       nfp_net_params_setup(hw);
+
+       dev_conf = &dev->data->dev_conf;
+       rxmode = &dev_conf->rxmode;
+
+       if (rxmode->mq_mode & ETH_MQ_RX_RSS) {
+               nfp_net_rss_config_default(dev);
+               update |= NFP_NET_CFG_UPDATE_RSS;
+               new_ctrl |= NFP_NET_CFG_CTRL_RSS;
+       }
+
        /* Enable device */
-       new_ctrl = hw->ctrl | NFP_NET_CFG_CTRL_ENABLE;
+       new_ctrl |= NFP_NET_CFG_CTRL_ENABLE;
 
        update |= NFP_NET_CFG_UPDATE_GEN | NFP_NET_CFG_UPDATE_RING;
 
@@ -797,7 +520,7 @@ nfp_net_start(struct rte_eth_dev *dev)
                return -EIO;
 
        /*
-        * Allocating rte mbuffs for configured rx queues.
+        * Allocating rte mbufs for configured rx queues.
         * This requires queues being enabled before
         */
        if (nfp_net_rx_freelist_setup(dev) < 0) {
@@ -805,9 +528,14 @@ nfp_net_start(struct rte_eth_dev *dev)
                goto error;
        }
 
-       if (hw->is_pf)
-               /* Configure the physical port up */
-               nfp_nsp_eth_config(hw->nspu_desc, hw->pf_port_idx, 1);
+       if (hw->is_phyport) {
+               if (rte_eal_process_type() == RTE_PROC_PRIMARY)
+                       /* Configure the physical port up */
+                       nfp_eth_set_configured(hw->cpp, hw->nfp_idx, 1);
+               else
+                       nfp_eth_set_configured(dev->process_private,
+                                              hw->nfp_idx, 1);
+       }
 
        hw->ctrl = new_ctrl;
 
@@ -833,7 +561,7 @@ error:
 }
 
 /* Stop device: disable rx and tx functions to allow for reconfiguring. */
-static void
+static int
 nfp_net_stop(struct rte_eth_dev *dev)
 {
        int i;
@@ -856,19 +584,71 @@ nfp_net_stop(struct rte_eth_dev *dev)
                        (struct nfp_net_rxq *)dev->data->rx_queues[i]);
        }
 
-       if (hw->is_pf)
+       if (hw->is_phyport) {
+               if (rte_eal_process_type() == RTE_PROC_PRIMARY)
+                       /* Configure the physical port down */
+                       nfp_eth_set_configured(hw->cpp, hw->nfp_idx, 0);
+               else
+                       nfp_eth_set_configured(dev->process_private,
+                                              hw->nfp_idx, 0);
+       }
+
+       return 0;
+}
+
+/* Set the link up. */
+static int
+nfp_net_set_link_up(struct rte_eth_dev *dev)
+{
+       struct nfp_net_hw *hw;
+
+       PMD_DRV_LOG(DEBUG, "Set link up");
+
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+
+       if (!hw->is_phyport)
+               return -ENOTSUP;
+
+       if (rte_eal_process_type() == RTE_PROC_PRIMARY)
                /* Configure the physical port down */
-               nfp_nsp_eth_config(hw->nspu_desc, hw->pf_port_idx, 0);
+               return nfp_eth_set_configured(hw->cpp, hw->nfp_idx, 1);
+       else
+               return nfp_eth_set_configured(dev->process_private,
+                                             hw->nfp_idx, 1);
+}
+
+/* Set the link down. */
+static int
+nfp_net_set_link_down(struct rte_eth_dev *dev)
+{
+       struct nfp_net_hw *hw;
+
+       PMD_DRV_LOG(DEBUG, "Set link down");
+
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+
+       if (!hw->is_phyport)
+               return -ENOTSUP;
+
+       if (rte_eal_process_type() == RTE_PROC_PRIMARY)
+               /* Configure the physical port down */
+               return nfp_eth_set_configured(hw->cpp, hw->nfp_idx, 0);
+       else
+               return nfp_eth_set_configured(dev->process_private,
+                                             hw->nfp_idx, 0);
 }
 
 /* Reset and stop device. The device can not be restarted. */
-static void
+static int
 nfp_net_close(struct rte_eth_dev *dev)
 {
        struct nfp_net_hw *hw;
        struct rte_pci_device *pci_dev;
        int i;
 
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return 0;
+
        PMD_INIT_LOG(DEBUG, "Close");
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
@@ -892,8 +672,34 @@ nfp_net_close(struct rte_eth_dev *dev)
                        (struct nfp_net_rxq *)dev->data->rx_queues[i]);
        }
 
+       /* Only free PF resources after all physical ports have been closed */
+       if (pci_dev->id.device_id == PCI_DEVICE_ID_NFP4000_PF_NIC ||
+           pci_dev->id.device_id == PCI_DEVICE_ID_NFP6000_PF_NIC) {
+               struct nfp_pf_dev *pf_dev;
+               pf_dev = NFP_NET_DEV_PRIVATE_TO_PF(dev->data->dev_private);
+
+               /* Mark this port as unused and free device priv resources*/
+               nn_cfg_writeb(hw, NFP_NET_CFG_LSC, 0xff);
+               pf_dev->ports[hw->idx] = NULL;
+               rte_eth_dev_release_port(dev);
+
+               for (i = 0; i < pf_dev->total_phyports; i++) {
+                       /* Check to see if ports are still in use */
+                       if (pf_dev->ports[i])
+                               return 0;
+               }
+
+               /* Now it is safe to free all PF resources */
+               PMD_INIT_LOG(INFO, "Freeing PF resources");
+               nfp_cpp_area_free(pf_dev->ctrl_area);
+               nfp_cpp_area_free(pf_dev->hwqueues_area);
+               free(pf_dev->hwinfo);
+               free(pf_dev->sym_tbl);
+               nfp_cpp_free(pf_dev->cpp);
+               rte_free(pf_dev);
+       }
+
        rte_intr_disable(&pci_dev->intr_handle);
-       nn_cfg_writeb(hw, NFP_NET_CFG_LSC, 0xff);
 
        /* unregister callback func from eal lib */
        rte_intr_callback_unregister(&pci_dev->intr_handle,
@@ -904,26 +710,29 @@ nfp_net_close(struct rte_eth_dev *dev)
         * The ixgbe PMD driver disables the pcie master on the
         * device. The i40e does not...
         */
+
+       return 0;
 }
 
-static void
+int
 nfp_net_promisc_enable(struct rte_eth_dev *dev)
 {
        uint32_t new_ctrl, update = 0;
        struct nfp_net_hw *hw;
+       int ret;
 
-       PMD_DRV_LOG(DEBUG, "Promiscuous mode enable\n");
+       PMD_DRV_LOG(DEBUG, "Promiscuous mode enable");
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
        if (!(hw->cap & NFP_NET_CFG_CTRL_PROMISC)) {
                PMD_INIT_LOG(INFO, "Promiscuous mode not supported");
-               return;
+               return -ENOTSUP;
        }
 
        if (hw->ctrl & NFP_NET_CFG_CTRL_PROMISC) {
-               PMD_DRV_LOG(INFO, "Promiscuous mode already enabled\n");
-               return;
+               PMD_DRV_LOG(INFO, "Promiscuous mode already enabled");
+               return 0;
        }
 
        new_ctrl = hw->ctrl | NFP_NET_CFG_CTRL_PROMISC;
@@ -933,23 +742,27 @@ nfp_net_promisc_enable(struct rte_eth_dev *dev)
         * DPDK sets promiscuous mode on just after this call assuming
         * it can not fail ...
         */
-       if (nfp_net_reconfig(hw, new_ctrl, update) < 0)
-               return;
+       ret = nfp_net_reconfig(hw, new_ctrl, update);
+       if (ret < 0)
+               return ret;
 
        hw->ctrl = new_ctrl;
+
+       return 0;
 }
 
-static void
+int
 nfp_net_promisc_disable(struct rte_eth_dev *dev)
 {
        uint32_t new_ctrl, update = 0;
        struct nfp_net_hw *hw;
+       int ret;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
        if ((hw->ctrl & NFP_NET_CFG_CTRL_PROMISC) == 0) {
-               PMD_DRV_LOG(INFO, "Promiscuous mode already disabled\n");
-               return;
+               PMD_DRV_LOG(INFO, "Promiscuous mode already disabled");
+               return 0;
        }
 
        new_ctrl = hw->ctrl & ~NFP_NET_CFG_CTRL_PROMISC;
@@ -959,10 +772,13 @@ nfp_net_promisc_disable(struct rte_eth_dev *dev)
         * DPDK sets promiscuous mode off just before this call
         * assuming it can not fail ...
         */
-       if (nfp_net_reconfig(hw, new_ctrl, update) < 0)
-               return;
+       ret = nfp_net_reconfig(hw, new_ctrl, update);
+       if (ret < 0)
+               return ret;
 
        hw->ctrl = new_ctrl;
+
+       return 0;
 }
 
 /*
@@ -971,12 +787,13 @@ nfp_net_promisc_disable(struct rte_eth_dev *dev)
  * Wait to complete is needed as it can take up to 9 seconds to get the Link
  * status.
  */
-static int
+int
 nfp_net_link_update(struct rte_eth_dev *dev, __rte_unused int wait_to_complete)
 {
        struct nfp_net_hw *hw;
-       struct rte_eth_link link, old;
+       struct rte_eth_link link;
        uint32_t nn_link_status;
+       int ret;
 
        static const uint32_t ls_to_ethtool[] = {
                [NFP_NET_CFG_STS_LINK_RATE_UNSUPPORTED] = ETH_SPEED_NUM_NONE,
@@ -989,13 +806,10 @@ nfp_net_link_update(struct rte_eth_dev *dev, __rte_unused int wait_to_complete)
                [NFP_NET_CFG_STS_LINK_RATE_100G]        = ETH_SPEED_NUM_100G,
        };
 
-       PMD_DRV_LOG(DEBUG, "Link update\n");
+       PMD_DRV_LOG(DEBUG, "Link update");
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
-       memset(&old, 0, sizeof(old));
-       nfp_net_dev_atomic_read_link_status(dev, &old);
-
        nn_link_status = nn_cfg_readl(hw, NFP_NET_CFG_STS);
 
        memset(&link, 0, sizeof(struct rte_eth_link));
@@ -1013,19 +827,17 @@ nfp_net_link_update(struct rte_eth_dev *dev, __rte_unused int wait_to_complete)
        else
                link.link_speed = ls_to_ethtool[nn_link_status];
 
-       if (old.link_status != link.link_status) {
-               nfp_net_dev_atomic_write_link_status(dev, &link);
+       ret = rte_eth_linkstatus_set(dev, &link);
+       if (ret == 0) {
                if (link.link_status)
-                       PMD_DRV_LOG(INFO, "NIC Link is Up\n");
+                       PMD_DRV_LOG(INFO, "NIC Link is Up");
                else
-                       PMD_DRV_LOG(INFO, "NIC Link is Down\n");
-               return 0;
+                       PMD_DRV_LOG(INFO, "NIC Link is Down");
        }
-
-       return -1;
+       return ret;
 }
 
-static int
+int
 nfp_net_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 {
        int i;
@@ -1120,7 +932,7 @@ nfp_net_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
        return -EINVAL;
 }
 
-static void
+int
 nfp_net_stats_reset(struct rte_eth_dev *dev)
 {
        int i;
@@ -1181,19 +993,20 @@ nfp_net_stats_reset(struct rte_eth_dev *dev)
 
        hw->eth_stats_base.imissed =
                nn_cfg_readq(hw, NFP_NET_CFG_STATS_RX_DISCARDS);
+
+       return 0;
 }
 
-static void
+int
 nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
 {
        struct nfp_net_hw *hw;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
-       dev_info->pci_dev = RTE_ETH_DEV_TO_PCI(dev);
        dev_info->max_rx_queues = (uint16_t)hw->max_rx_queues;
        dev_info->max_tx_queues = (uint16_t)hw->max_tx_queues;
-       dev_info->min_rx_bufsize = ETHER_MIN_MTU;
+       dev_info->min_rx_bufsize = RTE_ETHER_MIN_MTU;
        dev_info->max_rx_pktlen = hw->max_mtu;
        /* Next should change when PF support is implemented */
        dev_info->max_mac_addrs = 1;
@@ -1214,6 +1027,12 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
                                             DEV_TX_OFFLOAD_UDP_CKSUM |
                                             DEV_TX_OFFLOAD_TCP_CKSUM;
 
+       if (hw->cap & NFP_NET_CFG_CTRL_LSO_ANY)
+               dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO;
+
+       if (hw->cap & NFP_NET_CFG_CTRL_GATHER)
+               dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_MULTI_SEGS;
+
        dev_info->default_rxconf = (struct rte_eth_rxconf) {
                .rx_thresh = {
                        .pthresh = DEFAULT_RX_PTHRESH,
@@ -1232,27 +1051,47 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
                },
                .tx_free_thresh = DEFAULT_TX_FREE_THRESH,
                .tx_rs_thresh = DEFAULT_TX_RSBIT_THRESH,
-               .txq_flags = ETH_TXQ_FLAGS_NOMULTSEGS |
-                            ETH_TXQ_FLAGS_NOOFFLOADS,
        };
 
-       dev_info->flow_type_rss_offloads = ETH_RSS_NONFRAG_IPV4_TCP |
-                                          ETH_RSS_NONFRAG_IPV4_UDP |
-                                          ETH_RSS_NONFRAG_IPV6_TCP |
-                                          ETH_RSS_NONFRAG_IPV6_UDP;
+       dev_info->rx_desc_lim = (struct rte_eth_desc_lim) {
+               .nb_max = NFP_NET_MAX_RX_DESC,
+               .nb_min = NFP_NET_MIN_RX_DESC,
+               .nb_align = NFP_ALIGN_RING_DESC,
+       };
+
+       dev_info->tx_desc_lim = (struct rte_eth_desc_lim) {
+               .nb_max = NFP_NET_MAX_TX_DESC,
+               .nb_min = NFP_NET_MIN_TX_DESC,
+               .nb_align = NFP_ALIGN_RING_DESC,
+               .nb_seg_max = NFP_TX_MAX_SEG,
+               .nb_mtu_seg_max = NFP_TX_MAX_MTU_SEG,
+       };
+
+       /* All NFP devices support jumbo frames */
+       dev_info->rx_offload_capa |= DEV_RX_OFFLOAD_JUMBO_FRAME;
 
-       dev_info->reta_size = NFP_NET_CFG_RSS_ITBL_SZ;
-       dev_info->hash_key_size = NFP_NET_CFG_RSS_KEY_SZ;
+       if (hw->cap & NFP_NET_CFG_CTRL_RSS) {
+               dev_info->rx_offload_capa |= DEV_RX_OFFLOAD_RSS_HASH;
 
-       dev_info->speed_capa = ETH_SPEED_NUM_1G | ETH_LINK_SPEED_10G |
-                              ETH_SPEED_NUM_25G | ETH_SPEED_NUM_40G |
-                              ETH_SPEED_NUM_50G | ETH_LINK_SPEED_100G;
+               dev_info->flow_type_rss_offloads = ETH_RSS_IPV4 |
+                                                  ETH_RSS_NONFRAG_IPV4_TCP |
+                                                  ETH_RSS_NONFRAG_IPV4_UDP |
+                                                  ETH_RSS_IPV6 |
+                                                  ETH_RSS_NONFRAG_IPV6_TCP |
+                                                  ETH_RSS_NONFRAG_IPV6_UDP;
 
-       if (hw->cap & NFP_NET_CFG_CTRL_LSO)
-               dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO;
+               dev_info->reta_size = NFP_NET_CFG_RSS_ITBL_SZ;
+               dev_info->hash_key_size = NFP_NET_CFG_RSS_KEY_SZ;
+       }
+
+       dev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G |
+                              ETH_LINK_SPEED_25G | ETH_LINK_SPEED_40G |
+                              ETH_LINK_SPEED_50G | ETH_LINK_SPEED_100G;
+
+       return 0;
 }
 
-static const uint32_t *
+const uint32_t *
 nfp_net_supported_ptypes_get(struct rte_eth_dev *dev)
 {
        static const uint32_t ptypes[] = {
@@ -1269,45 +1108,7 @@ nfp_net_supported_ptypes_get(struct rte_eth_dev *dev)
        return NULL;
 }
 
-static uint32_t
-nfp_net_rx_queue_count(struct rte_eth_dev *dev, uint16_t queue_idx)
-{
-       struct nfp_net_rxq *rxq;
-       struct nfp_net_rx_desc *rxds;
-       uint32_t idx;
-       uint32_t count;
-
-       rxq = (struct nfp_net_rxq *)dev->data->rx_queues[queue_idx];
-
-       idx = rxq->rd_p;
-
-       count = 0;
-
-       /*
-        * Other PMDs are just checking the DD bit in intervals of 4
-        * descriptors and counting all four if the first has the DD
-        * bit on. Of course, this is not accurate but can be good for
-        * performance. But ideally that should be done in descriptors
-        * chunks belonging to the same cache line
-        */
-
-       while (count < rxq->rx_count) {
-               rxds = &rxq->rxds[idx];
-               if ((rxds->rxd.meta_len_dd & PCIE_DESC_RX_DD) == 0)
-                       break;
-
-               count++;
-               idx++;
-
-               /* Wrapping? */
-               if ((idx) == rxq->rx_count)
-                       idx = 0;
-       }
-
-       return count;
-}
-
-static int
+int
 nfp_rx_queue_intr_enable(struct rte_eth_dev *dev, uint16_t queue_id)
 {
        struct rte_pci_device *pci_dev;
@@ -1327,7 +1128,7 @@ nfp_rx_queue_intr_enable(struct rte_eth_dev *dev, uint16_t queue_id)
        return 0;
 }
 
-static int
+int
 nfp_rx_queue_intr_disable(struct rte_eth_dev *dev, uint16_t queue_id)
 {
        struct rte_pci_device *pci_dev;
@@ -1352,20 +1153,19 @@ nfp_net_dev_link_status_print(struct rte_eth_dev *dev)
        struct rte_pci_device *pci_dev = RTE_ETH_DEV_TO_PCI(dev);
        struct rte_eth_link link;
 
-       memset(&link, 0, sizeof(link));
-       nfp_net_dev_atomic_read_link_status(dev, &link);
+       rte_eth_linkstatus_get(dev, &link);
        if (link.link_status)
-               RTE_LOG(INFO, PMD, "Port %d: Link Up - speed %u Mbps - %s\n",
-                       dev->data->port_id, link.link_speed,
-                       link.link_duplex == ETH_LINK_FULL_DUPLEX
-                       ? "full-duplex" : "half-duplex");
+               PMD_DRV_LOG(INFO, "Port %d: Link Up - speed %u Mbps - %s",
+                           dev->data->port_id, link.link_speed,
+                           link.link_duplex == ETH_LINK_FULL_DUPLEX
+                           ? "full-duplex" : "half-duplex");
        else
-               RTE_LOG(INFO, PMD, " Port %d: Link Down\n",
-                       dev->data->port_id);
+               PMD_DRV_LOG(INFO, " Port %d: Link Down",
+                           dev->data->port_id);
 
-       RTE_LOG(INFO, PMD, "PCI Address: %04d:%02d:%02d:%d\n",
-               pci_dev->addr.domain, pci_dev->addr.bus,
-               pci_dev->addr.devid, pci_dev->addr.function);
+       PMD_DRV_LOG(INFO, "PCI Address: " PCI_PRI_FMT,
+                   pci_dev->addr.domain, pci_dev->addr.bus,
+                   pci_dev->addr.devid, pci_dev->addr.function);
 }
 
 /* Interrupt configuration and handling */
@@ -1388,7 +1188,7 @@ nfp_net_irq_unmask(struct rte_eth_dev *dev)
        if (hw->ctrl & NFP_NET_CFG_CTRL_MSIXAUTO) {
                /* If MSI-X auto-masking is used, clear the entry */
                rte_wmb();
-               rte_intr_enable(&pci_dev->intr_handle);
+               rte_intr_ack(&pci_dev->intr_handle);
        } else {
                /* Make sure all updates are written before un-masking */
                rte_wmb();
@@ -1397,18 +1197,41 @@ nfp_net_irq_unmask(struct rte_eth_dev *dev)
        }
 }
 
+/*
+ * Interrupt handler which shall be registered for alarm callback for delayed
+ * handling specific interrupt to wait for the stable nic state. As the NIC
+ * interrupt state is not stable for nfp after link is just down, it needs
+ * to wait 4 seconds to get the stable status.
+ *
+ * @param handle   Pointer to interrupt handle.
+ * @param param    The address of parameter (struct rte_eth_dev *)
+ *
+ * @return  void
+ */
 static void
+nfp_net_dev_interrupt_delayed_handler(void *param)
+{
+       struct rte_eth_dev *dev = (struct rte_eth_dev *)param;
+
+       nfp_net_link_update(dev, 0);
+       rte_eth_dev_callback_process(dev, RTE_ETH_EVENT_INTR_LSC, NULL);
+
+       nfp_net_dev_link_status_print(dev);
+
+       /* Unmasking */
+       nfp_net_irq_unmask(dev);
+}
+
+void
 nfp_net_dev_interrupt_handler(void *param)
 {
        int64_t timeout;
        struct rte_eth_link link;
        struct rte_eth_dev *dev = (struct rte_eth_dev *)param;
 
-       PMD_DRV_LOG(DEBUG, "We got a LSC interrupt!!!\n");
+       PMD_DRV_LOG(DEBUG, "We got a LSC interrupt!!!");
 
-       /* get the link status */
-       memset(&link, 0, sizeof(link));
-       nfp_net_dev_atomic_read_link_status(dev, &link);
+       rte_eth_linkstatus_get(dev, &link);
 
        nfp_net_link_update(dev, 0);
 
@@ -1425,38 +1248,13 @@ nfp_net_dev_interrupt_handler(void *param)
        if (rte_eal_alarm_set(timeout * 1000,
                              nfp_net_dev_interrupt_delayed_handler,
                              (void *)dev) < 0) {
-               RTE_LOG(ERR, PMD, "Error setting alarm");
+               PMD_INIT_LOG(ERR, "Error setting alarm");
                /* Unmasking */
                nfp_net_irq_unmask(dev);
        }
 }
 
-/*
- * Interrupt handler which shall be registered for alarm callback for delayed
- * handling specific interrupt to wait for the stable nic state. As the NIC
- * interrupt state is not stable for nfp after link is just down, it needs
- * to wait 4 seconds to get the stable status.
- *
- * @param handle   Pointer to interrupt handle.
- * @param param    The address of parameter (struct rte_eth_dev *)
- *
- * @return  void
- */
-static void
-nfp_net_dev_interrupt_delayed_handler(void *param)
-{
-       struct rte_eth_dev *dev = (struct rte_eth_dev *)param;
-
-       nfp_net_link_update(dev, 0);
-       _rte_eth_dev_callback_process(dev, RTE_ETH_EVENT_INTR_LSC, NULL, NULL);
-
-       nfp_net_dev_link_status_print(dev);
-
-       /* Unmasking */
-       nfp_net_irq_unmask(dev);
-}
-
-static int
+int
 nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
 {
        struct nfp_net_hw *hw;
@@ -1464,7 +1262,7 @@ nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
        /* check that mtu is within the allowed range */
-       if ((mtu < ETHER_MIN_MTU) || ((uint32_t)mtu > hw->max_mtu))
+       if (mtu < RTE_ETHER_MIN_MTU || (uint32_t)mtu > hw->max_mtu)
                return -EINVAL;
 
        /* mtu setting is forbidden if port is started */
@@ -1475,10 +1273,10 @@ nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
        }
 
        /* switch to jumbo mode if needed */
-       if ((uint32_t)mtu > ETHER_MAX_LEN)
-               dev->data->dev_conf.rxmode.jumbo_frame = 1;
+       if ((uint32_t)mtu > RTE_ETHER_MTU)
+               dev->data->dev_conf.rxmode.offloads |= DEV_RX_OFFLOAD_JUMBO_FRAME;
        else
-               dev->data->dev_conf.rxmode.jumbo_frame = 0;
+               dev->data->dev_conf.rxmode.offloads &= ~DEV_RX_OFFLOAD_JUMBO_FRAME;
 
        /* update max frame size */
        dev->data->dev_conf.rxmode.max_rx_pkt_len = (uint32_t)mtu;
@@ -1491,885 +1289,53 @@ nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
        return 0;
 }
 
-static int
-nfp_net_rx_queue_setup(struct rte_eth_dev *dev,
-                      uint16_t queue_idx, uint16_t nb_desc,
-                      unsigned int socket_id,
-                      const struct rte_eth_rxconf *rx_conf,
-                      struct rte_mempool *mp)
+int
+nfp_net_vlan_offload_set(struct rte_eth_dev *dev, int mask)
 {
-       const struct rte_memzone *tz;
-       struct nfp_net_rxq *rxq;
+       uint32_t new_ctrl, update;
        struct nfp_net_hw *hw;
+       int ret;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       new_ctrl = 0;
 
-       PMD_INIT_FUNC_TRACE();
+       /* Enable vlan strip if it is not configured yet */
+       if ((mask & ETH_VLAN_STRIP_OFFLOAD) &&
+           !(hw->ctrl & NFP_NET_CFG_CTRL_RXVLAN))
+               new_ctrl = hw->ctrl | NFP_NET_CFG_CTRL_RXVLAN;
 
-       /* Validating number of descriptors */
-       if (((nb_desc * sizeof(struct nfp_net_rx_desc)) % 128) != 0 ||
-           (nb_desc > NFP_NET_MAX_RX_DESC) ||
-           (nb_desc < NFP_NET_MIN_RX_DESC)) {
-               RTE_LOG(ERR, PMD, "Wrong nb_desc value\n");
-               return -EINVAL;
-       }
+       /* Disable vlan strip just if it is configured */
+       if (!(mask & ETH_VLAN_STRIP_OFFLOAD) &&
+           (hw->ctrl & NFP_NET_CFG_CTRL_RXVLAN))
+               new_ctrl = hw->ctrl & ~NFP_NET_CFG_CTRL_RXVLAN;
 
-       /*
-        * Free memory prior to re-allocation if needed. This is the case after
-        * calling nfp_net_stop
-        */
-       if (dev->data->rx_queues[queue_idx]) {
-               nfp_net_rx_queue_release(dev->data->rx_queues[queue_idx]);
-               dev->data->rx_queues[queue_idx] = NULL;
-       }
+       if (new_ctrl == 0)
+               return 0;
 
-       /* Allocating rx queue data structure */
-       rxq = rte_zmalloc_socket("ethdev RX queue", sizeof(struct nfp_net_rxq),
-                                RTE_CACHE_LINE_SIZE, socket_id);
-       if (rxq == NULL)
-               return -ENOMEM;
+       update = NFP_NET_CFG_UPDATE_GEN;
 
-       /* Hw queues mapping based on firmware confifguration */
-       rxq->qidx = queue_idx;
-       rxq->fl_qcidx = queue_idx * hw->stride_rx;
-       rxq->rx_qcidx = rxq->fl_qcidx + (hw->stride_rx - 1);
-       rxq->qcp_fl = hw->rx_bar + NFP_QCP_QUEUE_OFF(rxq->fl_qcidx);
-       rxq->qcp_rx = hw->rx_bar + NFP_QCP_QUEUE_OFF(rxq->rx_qcidx);
-
-       /*
-        * Tracking mbuf size for detecting a potential mbuf overflow due to
-        * RX offset
-        */
-       rxq->mem_pool = mp;
-       rxq->mbuf_size = rxq->mem_pool->elt_size;
-       rxq->mbuf_size -= (sizeof(struct rte_mbuf) + RTE_PKTMBUF_HEADROOM);
-       hw->flbufsz = rxq->mbuf_size;
-
-       rxq->rx_count = nb_desc;
-       rxq->port_id = dev->data->port_id;
-       rxq->rx_free_thresh = rx_conf->rx_free_thresh;
-       rxq->crc_len = (uint8_t) ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
-                                 : ETHER_CRC_LEN);
-       rxq->drop_en = rx_conf->rx_drop_en;
-
-       /*
-        * Allocate RX ring hardware descriptors. A memzone large enough to
-        * handle the maximum ring size is allocated in order to allow for
-        * resizing in later calls to the queue setup function.
-        */
-       tz = rte_eth_dma_zone_reserve(dev, "rx_ring", queue_idx,
-                                  sizeof(struct nfp_net_rx_desc) *
-                                  NFP_NET_MAX_RX_DESC, NFP_MEMZONE_ALIGN,
-                                  socket_id);
-
-       if (tz == NULL) {
-               RTE_LOG(ERR, PMD, "Error allocatig rx dma\n");
-               nfp_net_rx_queue_release(rxq);
-               return -ENOMEM;
-       }
-
-       /* Saving physical and virtual addresses for the RX ring */
-       rxq->dma = (uint64_t)tz->iova;
-       rxq->rxds = (struct nfp_net_rx_desc *)tz->addr;
-
-       /* mbuf pointers array for referencing mbufs linked to RX descriptors */
-       rxq->rxbufs = rte_zmalloc_socket("rxq->rxbufs",
-                                        sizeof(*rxq->rxbufs) * nb_desc,
-                                        RTE_CACHE_LINE_SIZE, socket_id);
-       if (rxq->rxbufs == NULL) {
-               nfp_net_rx_queue_release(rxq);
-               return -ENOMEM;
-       }
-
-       PMD_RX_LOG(DEBUG, "rxbufs=%p hw_ring=%p dma_addr=0x%" PRIx64 "\n",
-                  rxq->rxbufs, rxq->rxds, (unsigned long int)rxq->dma);
-
-       nfp_net_reset_rx_queue(rxq);
-
-       dev->data->rx_queues[queue_idx] = rxq;
-       rxq->hw = hw;
-
-       /*
-        * Telling the HW about the physical address of the RX ring and number
-        * of descriptors in log2 format
-        */
-       nn_cfg_writeq(hw, NFP_NET_CFG_RXR_ADDR(queue_idx), rxq->dma);
-       nn_cfg_writeb(hw, NFP_NET_CFG_RXR_SZ(queue_idx), rte_log2_u32(nb_desc));
-
-       return 0;
-}
-
-static int
-nfp_net_rx_fill_freelist(struct nfp_net_rxq *rxq)
-{
-       struct nfp_net_rx_buff *rxe = rxq->rxbufs;
-       uint64_t dma_addr;
-       unsigned i;
-
-       PMD_RX_LOG(DEBUG, "nfp_net_rx_fill_freelist for %u descriptors\n",
-                  rxq->rx_count);
-
-       for (i = 0; i < rxq->rx_count; i++) {
-               struct nfp_net_rx_desc *rxd;
-               struct rte_mbuf *mbuf = rte_pktmbuf_alloc(rxq->mem_pool);
-
-               if (mbuf == NULL) {
-                       RTE_LOG(ERR, PMD, "RX mbuf alloc failed queue_id=%u\n",
-                               (unsigned)rxq->qidx);
-                       return -ENOMEM;
-               }
-
-               dma_addr = rte_cpu_to_le_64(RTE_MBUF_DMA_ADDR_DEFAULT(mbuf));
-
-               rxd = &rxq->rxds[i];
-               rxd->fld.dd = 0;
-               rxd->fld.dma_addr_hi = (dma_addr >> 32) & 0xff;
-               rxd->fld.dma_addr_lo = dma_addr & 0xffffffff;
-               rxe[i].mbuf = mbuf;
-               PMD_RX_LOG(DEBUG, "[%d]: %" PRIx64 "\n", i, dma_addr);
-       }
-
-       /* Make sure all writes are flushed before telling the hardware */
-       rte_wmb();
-
-       /* Not advertising the whole ring as the firmware gets confused if so */
-       PMD_RX_LOG(DEBUG, "Increment FL write pointer in %u\n",
-                  rxq->rx_count - 1);
-
-       nfp_qcp_ptr_add(rxq->qcp_fl, NFP_QCP_WRITE_PTR, rxq->rx_count - 1);
-
-       return 0;
-}
-
-static int
-nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
-                      uint16_t nb_desc, unsigned int socket_id,
-                      const struct rte_eth_txconf *tx_conf)
-{
-       const struct rte_memzone *tz;
-       struct nfp_net_txq *txq;
-       uint16_t tx_free_thresh;
-       struct nfp_net_hw *hw;
-
-       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-
-       PMD_INIT_FUNC_TRACE();
-
-       /* Validating number of descriptors */
-       if (((nb_desc * sizeof(struct nfp_net_tx_desc)) % 128) != 0 ||
-           (nb_desc > NFP_NET_MAX_TX_DESC) ||
-           (nb_desc < NFP_NET_MIN_TX_DESC)) {
-               RTE_LOG(ERR, PMD, "Wrong nb_desc value\n");
-               return -EINVAL;
-       }
-
-       tx_free_thresh = (uint16_t)((tx_conf->tx_free_thresh) ?
-                                   tx_conf->tx_free_thresh :
-                                   DEFAULT_TX_FREE_THRESH);
-
-       if (tx_free_thresh > (nb_desc)) {
-               RTE_LOG(ERR, PMD,
-                       "tx_free_thresh must be less than the number of TX "
-                       "descriptors. (tx_free_thresh=%u port=%d "
-                       "queue=%d)\n", (unsigned int)tx_free_thresh,
-                       dev->data->port_id, (int)queue_idx);
-               return -(EINVAL);
-       }
-
-       /*
-        * Free memory prior to re-allocation if needed. This is the case after
-        * calling nfp_net_stop
-        */
-       if (dev->data->tx_queues[queue_idx]) {
-               PMD_TX_LOG(DEBUG, "Freeing memory prior to re-allocation %d\n",
-                          queue_idx);
-               nfp_net_tx_queue_release(dev->data->tx_queues[queue_idx]);
-               dev->data->tx_queues[queue_idx] = NULL;
-       }
-
-       /* Allocating tx queue data structure */
-       txq = rte_zmalloc_socket("ethdev TX queue", sizeof(struct nfp_net_txq),
-                                RTE_CACHE_LINE_SIZE, socket_id);
-       if (txq == NULL) {
-               RTE_LOG(ERR, PMD, "Error allocating tx dma\n");
-               return -ENOMEM;
-       }
-
-       /*
-        * Allocate TX ring hardware descriptors. A memzone large enough to
-        * handle the maximum ring size is allocated in order to allow for
-        * resizing in later calls to the queue setup function.
-        */
-       tz = rte_eth_dma_zone_reserve(dev, "tx_ring", queue_idx,
-                                  sizeof(struct nfp_net_tx_desc) *
-                                  NFP_NET_MAX_TX_DESC, NFP_MEMZONE_ALIGN,
-                                  socket_id);
-       if (tz == NULL) {
-               RTE_LOG(ERR, PMD, "Error allocating tx dma\n");
-               nfp_net_tx_queue_release(txq);
-               return -ENOMEM;
-       }
-
-       txq->tx_count = nb_desc;
-       txq->tx_free_thresh = tx_free_thresh;
-       txq->tx_pthresh = tx_conf->tx_thresh.pthresh;
-       txq->tx_hthresh = tx_conf->tx_thresh.hthresh;
-       txq->tx_wthresh = tx_conf->tx_thresh.wthresh;
-
-       /* queue mapping based on firmware configuration */
-       txq->qidx = queue_idx;
-       txq->tx_qcidx = queue_idx * hw->stride_tx;
-       txq->qcp_q = hw->tx_bar + NFP_QCP_QUEUE_OFF(txq->tx_qcidx);
-
-       txq->port_id = dev->data->port_id;
-       txq->txq_flags = tx_conf->txq_flags;
-
-       /* Saving physical and virtual addresses for the TX ring */
-       txq->dma = (uint64_t)tz->iova;
-       txq->txds = (struct nfp_net_tx_desc *)tz->addr;
-
-       /* mbuf pointers array for referencing mbufs linked to TX descriptors */
-       txq->txbufs = rte_zmalloc_socket("txq->txbufs",
-                                        sizeof(*txq->txbufs) * nb_desc,
-                                        RTE_CACHE_LINE_SIZE, socket_id);
-       if (txq->txbufs == NULL) {
-               nfp_net_tx_queue_release(txq);
-               return -ENOMEM;
-       }
-       PMD_TX_LOG(DEBUG, "txbufs=%p hw_ring=%p dma_addr=0x%" PRIx64 "\n",
-                  txq->txbufs, txq->txds, (unsigned long int)txq->dma);
-
-       nfp_net_reset_tx_queue(txq);
-
-       dev->data->tx_queues[queue_idx] = txq;
-       txq->hw = hw;
-
-       /*
-        * Telling the HW about the physical address of the TX ring and number
-        * of descriptors in log2 format
-        */
-       nn_cfg_writeq(hw, NFP_NET_CFG_TXR_ADDR(queue_idx), txq->dma);
-       nn_cfg_writeb(hw, NFP_NET_CFG_TXR_SZ(queue_idx), rte_log2_u32(nb_desc));
-
-       return 0;
-}
-
-/* nfp_net_tx_tso - Set TX descriptor for TSO */
-static inline void
-nfp_net_tx_tso(struct nfp_net_txq *txq, struct nfp_net_tx_desc *txd,
-              struct rte_mbuf *mb)
-{
-       uint64_t ol_flags;
-       struct nfp_net_hw *hw = txq->hw;
-
-       if (!(hw->cap & NFP_NET_CFG_CTRL_LSO))
-               goto clean_txd;
-
-       ol_flags = mb->ol_flags;
-
-       if (!(ol_flags & PKT_TX_TCP_SEG))
-               goto clean_txd;
-
-       txd->l4_offset = mb->l2_len + mb->l3_len + mb->l4_len;
-       txd->lso = rte_cpu_to_le_16(mb->tso_segsz);
-       txd->flags = PCIE_DESC_TX_LSO;
-       return;
-
-clean_txd:
-       txd->flags = 0;
-       txd->l4_offset = 0;
-       txd->lso = 0;
-}
-
-/* nfp_net_tx_cksum - Set TX CSUM offload flags in TX descriptor */
-static inline void
-nfp_net_tx_cksum(struct nfp_net_txq *txq, struct nfp_net_tx_desc *txd,
-                struct rte_mbuf *mb)
-{
-       uint64_t ol_flags;
-       struct nfp_net_hw *hw = txq->hw;
-
-       if (!(hw->cap & NFP_NET_CFG_CTRL_TXCSUM))
-               return;
-
-       ol_flags = mb->ol_flags;
-
-       /* IPv6 does not need checksum */
-       if (ol_flags & PKT_TX_IP_CKSUM)
-               txd->flags |= PCIE_DESC_TX_IP4_CSUM;
-
-       switch (ol_flags & PKT_TX_L4_MASK) {
-       case PKT_TX_UDP_CKSUM:
-               txd->flags |= PCIE_DESC_TX_UDP_CSUM;
-               break;
-       case PKT_TX_TCP_CKSUM:
-               txd->flags |= PCIE_DESC_TX_TCP_CSUM;
-               break;
-       }
-
-       if (ol_flags & (PKT_TX_IP_CKSUM | PKT_TX_L4_MASK))
-               txd->flags |= PCIE_DESC_TX_CSUM;
-}
-
-/* nfp_net_rx_cksum - set mbuf checksum flags based on RX descriptor flags */
-static inline void
-nfp_net_rx_cksum(struct nfp_net_rxq *rxq, struct nfp_net_rx_desc *rxd,
-                struct rte_mbuf *mb)
-{
-       struct nfp_net_hw *hw = rxq->hw;
-
-       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RXCSUM))
-               return;
-
-       /* If IPv4 and IP checksum error, fail */
-       if ((rxd->rxd.flags & PCIE_DESC_RX_IP4_CSUM) &&
-           !(rxd->rxd.flags & PCIE_DESC_RX_IP4_CSUM_OK))
-               mb->ol_flags |= PKT_RX_IP_CKSUM_BAD;
-
-       /* If neither UDP nor TCP return */
-       if (!(rxd->rxd.flags & PCIE_DESC_RX_TCP_CSUM) &&
-           !(rxd->rxd.flags & PCIE_DESC_RX_UDP_CSUM))
-               return;
-
-       if ((rxd->rxd.flags & PCIE_DESC_RX_TCP_CSUM) &&
-           !(rxd->rxd.flags & PCIE_DESC_RX_TCP_CSUM_OK))
-               mb->ol_flags |= PKT_RX_L4_CKSUM_BAD;
-
-       if ((rxd->rxd.flags & PCIE_DESC_RX_UDP_CSUM) &&
-           !(rxd->rxd.flags & PCIE_DESC_RX_UDP_CSUM_OK))
-               mb->ol_flags |= PKT_RX_L4_CKSUM_BAD;
-}
-
-#define NFP_HASH_OFFSET      ((uint8_t *)mbuf->buf_addr + mbuf->data_off - 4)
-#define NFP_HASH_TYPE_OFFSET ((uint8_t *)mbuf->buf_addr + mbuf->data_off - 8)
-
-#define NFP_DESC_META_LEN(d) (d->rxd.meta_len_dd & PCIE_DESC_RX_META_LEN_MASK)
-
-/*
- * nfp_net_set_hash - Set mbuf hash data
- *
- * The RSS hash and hash-type are pre-pended to the packet data.
- * Extract and decode it and set the mbuf fields.
- */
-static inline void
-nfp_net_set_hash(struct nfp_net_rxq *rxq, struct nfp_net_rx_desc *rxd,
-                struct rte_mbuf *mbuf)
-{
-       struct nfp_net_hw *hw = rxq->hw;
-       uint8_t *meta_offset;
-       uint32_t meta_info;
-       uint32_t hash = 0;
-       uint32_t hash_type = 0;
-
-       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS))
-               return;
-
-       if (NFD_CFG_MAJOR_VERSION_of(hw->ver) <= 3) {
-               if (!(rxd->rxd.flags & PCIE_DESC_RX_RSS))
-                       return;
-
-               hash = rte_be_to_cpu_32(*(uint32_t *)NFP_HASH_OFFSET);
-               hash_type = rte_be_to_cpu_32(*(uint32_t *)NFP_HASH_TYPE_OFFSET);
-
-       } else if (NFP_DESC_META_LEN(rxd)) {
-               /*
-                * new metadata api:
-                * <----  32 bit  ----->
-                * m    field type word
-                * e     data field #2
-                * t     data field #1
-                * a     data field #0
-                * ====================
-                *    packet data
-                *
-                * Field type word contains up to 8 4bit field types
-                * A 4bit field type refers to a data field word
-                * A data field word can have several 4bit field types
-                */
-               meta_offset = rte_pktmbuf_mtod(mbuf, uint8_t *);
-               meta_offset -= NFP_DESC_META_LEN(rxd);
-               meta_info = rte_be_to_cpu_32(*(uint32_t *)meta_offset);
-               meta_offset += 4;
-               /* NFP PMD just supports metadata for hashing */
-               switch (meta_info & NFP_NET_META_FIELD_MASK) {
-               case NFP_NET_META_HASH:
-                       /* next field type is about the hash type */
-                       meta_info >>= NFP_NET_META_FIELD_SIZE;
-                       /* hash value is in the data field */
-                       hash = rte_be_to_cpu_32(*(uint32_t *)meta_offset);
-                       hash_type = meta_info & NFP_NET_META_FIELD_MASK;
-                       break;
-               default:
-                       /* Unsupported metadata can be a performance issue */
-                       return;
-               }
-       } else {
-               return;
-       }
-
-       mbuf->hash.rss = hash;
-       mbuf->ol_flags |= PKT_RX_RSS_HASH;
-
-       switch (hash_type) {
-       case NFP_NET_RSS_IPV4:
-               mbuf->packet_type |= RTE_PTYPE_INNER_L3_IPV4;
-               break;
-       case NFP_NET_RSS_IPV6:
-               mbuf->packet_type |= RTE_PTYPE_INNER_L3_IPV6;
-               break;
-       case NFP_NET_RSS_IPV6_EX:
-               mbuf->packet_type |= RTE_PTYPE_INNER_L3_IPV6_EXT;
-               break;
-       default:
-               mbuf->packet_type |= RTE_PTYPE_INNER_L4_MASK;
-       }
-}
-
-static inline void
-nfp_net_mbuf_alloc_failed(struct nfp_net_rxq *rxq)
-{
-       rte_eth_devices[rxq->port_id].data->rx_mbuf_alloc_failed++;
-}
-
-#define NFP_DESC_META_LEN(d) (d->rxd.meta_len_dd & PCIE_DESC_RX_META_LEN_MASK)
-
-/*
- * RX path design:
- *
- * There are some decissions to take:
- * 1) How to check DD RX descriptors bit
- * 2) How and when to allocate new mbufs
- *
- * Current implementation checks just one single DD bit each loop. As each
- * descriptor is 8 bytes, it is likely a good idea to check descriptors in
- * a single cache line instead. Tests with this change have not shown any
- * performance improvement but it requires further investigation. For example,
- * depending on which descriptor is next, the number of descriptors could be
- * less than 8 for just checking those in the same cache line. This implies
- * extra work which could be counterproductive by itself. Indeed, last firmware
- * changes are just doing this: writing several descriptors with the DD bit
- * for saving PCIe bandwidth and DMA operations from the NFP.
- *
- * Mbuf allocation is done when a new packet is received. Then the descriptor
- * is automatically linked with the new mbuf and the old one is given to the
- * user. The main drawback with this design is mbuf allocation is heavier than
- * using bulk allocations allowed by DPDK with rte_mempool_get_bulk. From the
- * cache point of view it does not seem allocating the mbuf early on as we are
- * doing now have any benefit at all. Again, tests with this change have not
- * shown any improvement. Also, rte_mempool_get_bulk returns all or nothing
- * so looking at the implications of this type of allocation should be studied
- * deeply
- */
-
-static uint16_t
-nfp_net_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)
-{
-       struct nfp_net_rxq *rxq;
-       struct nfp_net_rx_desc *rxds;
-       struct nfp_net_rx_buff *rxb;
-       struct nfp_net_hw *hw;
-       struct rte_mbuf *mb;
-       struct rte_mbuf *new_mb;
-       uint16_t nb_hold;
-       uint64_t dma_addr;
-       int avail;
-
-       rxq = rx_queue;
-       if (unlikely(rxq == NULL)) {
-               /*
-                * DPDK just checks the queue is lower than max queues
-                * enabled. But the queue needs to be configured
-                */
-               RTE_LOG_DP(ERR, PMD, "RX Bad queue\n");
-               return -EINVAL;
-       }
-
-       hw = rxq->hw;
-       avail = 0;
-       nb_hold = 0;
-
-       while (avail < nb_pkts) {
-               rxb = &rxq->rxbufs[rxq->rd_p];
-               if (unlikely(rxb == NULL)) {
-                       RTE_LOG_DP(ERR, PMD, "rxb does not exist!\n");
-                       break;
-               }
-
-               /*
-                * Memory barrier to ensure that we won't do other
-                * reads before the DD bit.
-                */
-               rte_rmb();
-
-               rxds = &rxq->rxds[rxq->rd_p];
-               if ((rxds->rxd.meta_len_dd & PCIE_DESC_RX_DD) == 0)
-                       break;
-
-               /*
-                * We got a packet. Let's alloc a new mbuff for refilling the
-                * free descriptor ring as soon as possible
-                */
-               new_mb = rte_pktmbuf_alloc(rxq->mem_pool);
-               if (unlikely(new_mb == NULL)) {
-                       RTE_LOG_DP(DEBUG, PMD,
-                       "RX mbuf alloc failed port_id=%u queue_id=%u\n",
-                               rxq->port_id, (unsigned int)rxq->qidx);
-                       nfp_net_mbuf_alloc_failed(rxq);
-                       break;
-               }
-
-               nb_hold++;
-
-               /*
-                * Grab the mbuff and refill the descriptor with the
-                * previously allocated mbuff
-                */
-               mb = rxb->mbuf;
-               rxb->mbuf = new_mb;
-
-               PMD_RX_LOG(DEBUG, "Packet len: %u, mbuf_size: %u\n",
-                          rxds->rxd.data_len, rxq->mbuf_size);
-
-               /* Size of this segment */
-               mb->data_len = rxds->rxd.data_len - NFP_DESC_META_LEN(rxds);
-               /* Size of the whole packet. We just support 1 segment */
-               mb->pkt_len = rxds->rxd.data_len - NFP_DESC_META_LEN(rxds);
-
-               if (unlikely((mb->data_len + hw->rx_offset) >
-                            rxq->mbuf_size)) {
-                       /*
-                        * This should not happen and the user has the
-                        * responsibility of avoiding it. But we have
-                        * to give some info about the error
-                        */
-                       RTE_LOG_DP(ERR, PMD,
-                               "mbuf overflow likely due to the RX offset.\n"
-                               "\t\tYour mbuf size should have extra space for"
-                               " RX offset=%u bytes.\n"
-                               "\t\tCurrently you just have %u bytes available"
-                               " but the received packet is %u bytes long",
-                               hw->rx_offset,
-                               rxq->mbuf_size - hw->rx_offset,
-                               mb->data_len);
-                       return -EINVAL;
-               }
-
-               /* Filling the received mbuff with packet info */
-               if (hw->rx_offset)
-                       mb->data_off = RTE_PKTMBUF_HEADROOM + hw->rx_offset;
-               else
-                       mb->data_off = RTE_PKTMBUF_HEADROOM +
-                                      NFP_DESC_META_LEN(rxds);
-
-               /* No scatter mode supported */
-               mb->nb_segs = 1;
-               mb->next = NULL;
-
-               /* Checking the RSS flag */
-               nfp_net_set_hash(rxq, rxds, mb);
-
-               /* Checking the checksum flag */
-               nfp_net_rx_cksum(rxq, rxds, mb);
-
-               if ((rxds->rxd.flags & PCIE_DESC_RX_VLAN) &&
-                   (hw->ctrl & NFP_NET_CFG_CTRL_RXVLAN)) {
-                       mb->vlan_tci = rte_cpu_to_le_32(rxds->rxd.vlan);
-                       mb->ol_flags |= PKT_RX_VLAN | PKT_RX_VLAN_STRIPPED;
-               }
-
-               /* Adding the mbuff to the mbuff array passed by the app */
-               rx_pkts[avail++] = mb;
-
-               /* Now resetting and updating the descriptor */
-               rxds->vals[0] = 0;
-               rxds->vals[1] = 0;
-               dma_addr = rte_cpu_to_le_64(RTE_MBUF_DMA_ADDR_DEFAULT(new_mb));
-               rxds->fld.dd = 0;
-               rxds->fld.dma_addr_hi = (dma_addr >> 32) & 0xff;
-               rxds->fld.dma_addr_lo = dma_addr & 0xffffffff;
-
-               rxq->rd_p++;
-               if (unlikely(rxq->rd_p == rxq->rx_count)) /* wrapping?*/
-                       rxq->rd_p = 0;
-       }
-
-       if (nb_hold == 0)
-               return nb_hold;
-
-       PMD_RX_LOG(DEBUG, "RX  port_id=%u queue_id=%u, %d packets received\n",
-                  rxq->port_id, (unsigned int)rxq->qidx, nb_hold);
-
-       nb_hold += rxq->nb_rx_hold;
-
-       /*
-        * FL descriptors needs to be written before incrementing the
-        * FL queue WR pointer
-        */
-       rte_wmb();
-       if (nb_hold > rxq->rx_free_thresh) {
-               PMD_RX_LOG(DEBUG, "port=%u queue=%u nb_hold=%u avail=%u\n",
-                          rxq->port_id, (unsigned int)rxq->qidx,
-                          (unsigned)nb_hold, (unsigned)avail);
-               nfp_qcp_ptr_add(rxq->qcp_fl, NFP_QCP_WRITE_PTR, nb_hold);
-               nb_hold = 0;
-       }
-       rxq->nb_rx_hold = nb_hold;
-
-       return avail;
-}
-
-/*
- * nfp_net_tx_free_bufs - Check for descriptors with a complete
- * status
- * @txq: TX queue to work with
- * Returns number of descriptors freed
- */
-int
-nfp_net_tx_free_bufs(struct nfp_net_txq *txq)
-{
-       uint32_t qcp_rd_p;
-       int todo;
-
-       PMD_TX_LOG(DEBUG, "queue %u. Check for descriptor with a complete"
-                  " status\n", txq->qidx);
-
-       /* Work out how many packets have been sent */
-       qcp_rd_p = nfp_qcp_read(txq->qcp_q, NFP_QCP_READ_PTR);
-
-       if (qcp_rd_p == txq->rd_p) {
-               PMD_TX_LOG(DEBUG, "queue %u: It seems harrier is not sending "
-                          "packets (%u, %u)\n", txq->qidx,
-                          qcp_rd_p, txq->rd_p);
-               return 0;
-       }
-
-       if (qcp_rd_p > txq->rd_p)
-               todo = qcp_rd_p - txq->rd_p;
-       else
-               todo = qcp_rd_p + txq->tx_count - txq->rd_p;
-
-       PMD_TX_LOG(DEBUG, "qcp_rd_p %u, txq->rd_p: %u, qcp->rd_p: %u\n",
-                  qcp_rd_p, txq->rd_p, txq->rd_p);
-
-       if (todo == 0)
-               return todo;
-
-       txq->rd_p += todo;
-       if (unlikely(txq->rd_p >= txq->tx_count))
-               txq->rd_p -= txq->tx_count;
-
-       return todo;
-}
-
-/* Leaving always free descriptors for avoiding wrapping confusion */
-static inline
-uint32_t nfp_free_tx_desc(struct nfp_net_txq *txq)
-{
-       if (txq->wr_p >= txq->rd_p)
-               return txq->tx_count - (txq->wr_p - txq->rd_p) - 8;
-       else
-               return txq->rd_p - txq->wr_p - 8;
-}
-
-/*
- * nfp_net_txq_full - Check if the TX queue free descriptors
- * is below tx_free_threshold
- *
- * @txq: TX queue to check
- *
- * This function uses the host copy* of read/write pointers
- */
-static inline
-uint32_t nfp_net_txq_full(struct nfp_net_txq *txq)
-{
-       return (nfp_free_tx_desc(txq) < txq->tx_free_thresh);
-}
-
-static uint16_t
-nfp_net_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
-{
-       struct nfp_net_txq *txq;
-       struct nfp_net_hw *hw;
-       struct nfp_net_tx_desc *txds, txd;
-       struct rte_mbuf *pkt;
-       uint64_t dma_addr;
-       int pkt_size, dma_size;
-       uint16_t free_descs, issued_descs;
-       struct rte_mbuf **lmbuf;
-       int i;
-
-       txq = tx_queue;
-       hw = txq->hw;
-       txds = &txq->txds[txq->wr_p];
-
-       PMD_TX_LOG(DEBUG, "working for queue %u at pos %d and %u packets\n",
-                  txq->qidx, txq->wr_p, nb_pkts);
-
-       if ((nfp_free_tx_desc(txq) < nb_pkts) || (nfp_net_txq_full(txq)))
-               nfp_net_tx_free_bufs(txq);
-
-       free_descs = (uint16_t)nfp_free_tx_desc(txq);
-       if (unlikely(free_descs == 0))
-               return 0;
-
-       pkt = *tx_pkts;
-
-       i = 0;
-       issued_descs = 0;
-       PMD_TX_LOG(DEBUG, "queue: %u. Sending %u packets\n",
-                  txq->qidx, nb_pkts);
-       /* Sending packets */
-       while ((i < nb_pkts) && free_descs) {
-               /* Grabbing the mbuf linked to the current descriptor */
-               lmbuf = &txq->txbufs[txq->wr_p].mbuf;
-               /* Warming the cache for releasing the mbuf later on */
-               RTE_MBUF_PREFETCH_TO_FREE(*lmbuf);
-
-               pkt = *(tx_pkts + i);
-
-               if (unlikely((pkt->nb_segs > 1) &&
-                            !(hw->cap & NFP_NET_CFG_CTRL_GATHER))) {
-                       PMD_INIT_LOG(INFO, "NFP_NET_CFG_CTRL_GATHER not set");
-                       rte_panic("Multisegment packet unsupported\n");
-               }
-
-               /* Checking if we have enough descriptors */
-               if (unlikely(pkt->nb_segs > free_descs))
-                       goto xmit_end;
-
-               /*
-                * Checksum and VLAN flags just in the first descriptor for a
-                * multisegment packet, but TSO info needs to be in all of them.
-                */
-               txd.data_len = pkt->pkt_len;
-               nfp_net_tx_tso(txq, &txd, pkt);
-               nfp_net_tx_cksum(txq, &txd, pkt);
-
-               if ((pkt->ol_flags & PKT_TX_VLAN_PKT) &&
-                   (hw->cap & NFP_NET_CFG_CTRL_TXVLAN)) {
-                       txd.flags |= PCIE_DESC_TX_VLAN;
-                       txd.vlan = pkt->vlan_tci;
-               }
-
-               /*
-                * mbuf data_len is the data in one segment and pkt_len data
-                * in the whole packet. When the packet is just one segment,
-                * then data_len = pkt_len
-                */
-               pkt_size = pkt->pkt_len;
-
-               while (pkt) {
-                       /* Copying TSO, VLAN and cksum info */
-                       *txds = txd;
-
-                       /* Releasing mbuf used by this descriptor previously*/
-                       if (*lmbuf)
-                               rte_pktmbuf_free_seg(*lmbuf);
-
-                       /*
-                        * Linking mbuf with descriptor for being released
-                        * next time descriptor is used
-                        */
-                       *lmbuf = pkt;
-
-                       dma_size = pkt->data_len;
-                       dma_addr = rte_mbuf_data_iova(pkt);
-                       PMD_TX_LOG(DEBUG, "Working with mbuf at dma address:"
-                                  "%" PRIx64 "\n", dma_addr);
-
-                       /* Filling descriptors fields */
-                       txds->dma_len = dma_size;
-                       txds->data_len = txd.data_len;
-                       txds->dma_addr_hi = (dma_addr >> 32) & 0xff;
-                       txds->dma_addr_lo = (dma_addr & 0xffffffff);
-                       ASSERT(free_descs > 0);
-                       free_descs--;
-
-                       txq->wr_p++;
-                       if (unlikely(txq->wr_p == txq->tx_count)) /* wrapping?*/
-                               txq->wr_p = 0;
-
-                       pkt_size -= dma_size;
-                       if (!pkt_size)
-                               /* End of packet */
-                               txds->offset_eop |= PCIE_DESC_TX_EOP;
-                       else
-                               txds->offset_eop &= PCIE_DESC_TX_OFFSET_MASK;
-
-                       pkt = pkt->next;
-                       /* Referencing next free TX descriptor */
-                       txds = &txq->txds[txq->wr_p];
-                       lmbuf = &txq->txbufs[txq->wr_p].mbuf;
-                       issued_descs++;
-               }
-               i++;
-       }
-
-xmit_end:
-       /* Increment write pointers. Force memory write before we let HW know */
-       rte_wmb();
-       nfp_qcp_ptr_add(txq->qcp_q, NFP_QCP_WRITE_PTR, issued_descs);
-
-       return i;
-}
-
-static int
-nfp_net_vlan_offload_set(struct rte_eth_dev *dev, int mask)
-{
-       uint32_t new_ctrl, update;
-       struct nfp_net_hw *hw;
-       int ret;
-
-       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-       new_ctrl = 0;
-
-       if ((mask & ETH_VLAN_FILTER_OFFLOAD) ||
-           (mask & ETH_VLAN_EXTEND_OFFLOAD))
-               RTE_LOG(INFO, PMD, "No support for ETH_VLAN_FILTER_OFFLOAD or"
-                       " ETH_VLAN_EXTEND_OFFLOAD");
-
-       /* Enable vlan strip if it is not configured yet */
-       if ((mask & ETH_VLAN_STRIP_OFFLOAD) &&
-           !(hw->ctrl & NFP_NET_CFG_CTRL_RXVLAN))
-               new_ctrl = hw->ctrl | NFP_NET_CFG_CTRL_RXVLAN;
-
-       /* Disable vlan strip just if it is configured */
-       if (!(mask & ETH_VLAN_STRIP_OFFLOAD) &&
-           (hw->ctrl & NFP_NET_CFG_CTRL_RXVLAN))
-               new_ctrl = hw->ctrl & ~NFP_NET_CFG_CTRL_RXVLAN;
-
-       if (new_ctrl == 0)
-               return 0;
-
-       update = NFP_NET_CFG_UPDATE_GEN;
-
-       ret = nfp_net_reconfig(hw, new_ctrl, update);
-       if (!ret)
-               hw->ctrl = new_ctrl;
+       ret = nfp_net_reconfig(hw, new_ctrl, update);
+       if (!ret)
+               hw->ctrl = new_ctrl;
 
        return ret;
 }
 
-/* Update Redirection Table(RETA) of Receive Side Scaling of Ethernet device */
 static int
-nfp_net_reta_update(struct rte_eth_dev *dev,
+nfp_net_rss_reta_write(struct rte_eth_dev *dev,
                    struct rte_eth_rss_reta_entry64 *reta_conf,
                    uint16_t reta_size)
 {
        uint32_t reta, mask;
        int i, j;
        int idx, shift;
-       uint32_t update;
        struct nfp_net_hw *hw =
                NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 
-       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS))
-               return -EINVAL;
-
        if (reta_size != NFP_NET_CFG_RSS_ITBL_SZ) {
-               RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
+               PMD_DRV_LOG(ERR, "The size of hash lookup table configured "
                        "(%d) doesn't match the number hardware can supported "
-                       "(%d)\n", reta_size, NFP_NET_CFG_RSS_ITBL_SZ);
+                       "(%d)", reta_size, NFP_NET_CFG_RSS_ITBL_SZ);
                return -EINVAL;
        }
 
@@ -2402,6 +1368,26 @@ nfp_net_reta_update(struct rte_eth_dev *dev,
                nn_cfg_writel(hw, NFP_NET_CFG_RSS_ITBL + (idx * 64) + shift,
                              reta);
        }
+       return 0;
+}
+
+/* Update Redirection Table(RETA) of Receive Side Scaling of Ethernet device */
+int
+nfp_net_reta_update(struct rte_eth_dev *dev,
+                   struct rte_eth_rss_reta_entry64 *reta_conf,
+                   uint16_t reta_size)
+{
+       struct nfp_net_hw *hw =
+               NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       uint32_t update;
+       int ret;
+
+       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS))
+               return -EINVAL;
+
+       ret = nfp_net_rss_reta_write(dev, reta_conf, reta_size);
+       if (ret != 0)
+               return ret;
 
        update = NFP_NET_CFG_UPDATE_RSS;
 
@@ -2412,7 +1398,7 @@ nfp_net_reta_update(struct rte_eth_dev *dev,
 }
 
  /* Query Redirection Table(RETA) of Receive Side Scaling of Ethernet device. */
-static int
+int
 nfp_net_reta_query(struct rte_eth_dev *dev,
                   struct rte_eth_rss_reta_entry64 *reta_conf,
                   uint16_t reta_size)
@@ -2422,52 +1408,99 @@ nfp_net_reta_query(struct rte_eth_dev *dev,
        uint32_t reta;
        struct nfp_net_hw *hw;
 
-       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+
+       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS))
+               return -EINVAL;
+
+       if (reta_size != NFP_NET_CFG_RSS_ITBL_SZ) {
+               PMD_DRV_LOG(ERR, "The size of hash lookup table configured "
+                       "(%d) doesn't match the number hardware can supported "
+                       "(%d)", reta_size, NFP_NET_CFG_RSS_ITBL_SZ);
+               return -EINVAL;
+       }
+
+       /*
+        * Reading Redirection Table. There are 128 8bit-entries which can be
+        * manage as 32 32bit-entries
+        */
+       for (i = 0; i < reta_size; i += 4) {
+               /* Handling 4 RSS entries per loop */
+               idx = i / RTE_RETA_GROUP_SIZE;
+               shift = i % RTE_RETA_GROUP_SIZE;
+               mask = (uint8_t)((reta_conf[idx].mask >> shift) & 0xF);
+
+               if (!mask)
+                       continue;
+
+               reta = nn_cfg_readl(hw, NFP_NET_CFG_RSS_ITBL + (idx * 64) +
+                                   shift);
+               for (j = 0; j < 4; j++) {
+                       if (!(mask & (0x1 << j)))
+                               continue;
+                       reta_conf[idx].reta[shift + j] =
+                               (uint8_t)((reta >> (8 * j)) & 0xF);
+               }
+       }
+       return 0;
+}
+
+static int
+nfp_net_rss_hash_write(struct rte_eth_dev *dev,
+                       struct rte_eth_rss_conf *rss_conf)
+{
+       struct nfp_net_hw *hw;
+       uint64_t rss_hf;
+       uint32_t cfg_rss_ctrl = 0;
+       uint8_t key;
+       int i;
+
+       hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+
+       /* Writing the key byte a byte */
+       for (i = 0; i < rss_conf->rss_key_len; i++) {
+               memcpy(&key, &rss_conf->rss_key[i], 1);
+               nn_cfg_writeb(hw, NFP_NET_CFG_RSS_KEY + i, key);
+       }
+
+       rss_hf = rss_conf->rss_hf;
+
+       if (rss_hf & ETH_RSS_IPV4)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV4;
+
+       if (rss_hf & ETH_RSS_NONFRAG_IPV4_TCP)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV4_TCP;
+
+       if (rss_hf & ETH_RSS_NONFRAG_IPV4_UDP)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV4_UDP;
+
+       if (rss_hf & ETH_RSS_IPV6)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV6;
+
+       if (rss_hf & ETH_RSS_NONFRAG_IPV6_TCP)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV6_TCP;
 
-       if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS))
-               return -EINVAL;
+       if (rss_hf & ETH_RSS_NONFRAG_IPV6_UDP)
+               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV6_UDP;
 
-       if (reta_size != NFP_NET_CFG_RSS_ITBL_SZ) {
-               RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
-                       "(%d) doesn't match the number hardware can supported "
-                       "(%d)\n", reta_size, NFP_NET_CFG_RSS_ITBL_SZ);
-               return -EINVAL;
-       }
+       cfg_rss_ctrl |= NFP_NET_CFG_RSS_MASK;
+       cfg_rss_ctrl |= NFP_NET_CFG_RSS_TOEPLITZ;
 
-       /*
-        * Reading Redirection Table. There are 128 8bit-entries which can be
-        * manage as 32 32bit-entries
-        */
-       for (i = 0; i < reta_size; i += 4) {
-               /* Handling 4 RSS entries per loop */
-               idx = i / RTE_RETA_GROUP_SIZE;
-               shift = i % RTE_RETA_GROUP_SIZE;
-               mask = (uint8_t)((reta_conf[idx].mask >> shift) & 0xF);
+       /* configuring where to apply the RSS hash */
+       nn_cfg_writel(hw, NFP_NET_CFG_RSS_CTRL, cfg_rss_ctrl);
 
-               if (!mask)
-                       continue;
+       /* Writing the key size */
+       nn_cfg_writeb(hw, NFP_NET_CFG_RSS_KEY_SZ, rss_conf->rss_key_len);
 
-               reta = nn_cfg_readl(hw, NFP_NET_CFG_RSS_ITBL + (idx * 64) +
-                                   shift);
-               for (j = 0; j < 4; j++) {
-                       if (!(mask & (0x1 << j)))
-                               continue;
-                       reta_conf->reta[shift + j] =
-                               (uint8_t)((reta >> (8 * j)) & 0xF);
-               }
-       }
        return 0;
 }
 
-static int
+int
 nfp_net_rss_hash_update(struct rte_eth_dev *dev,
                        struct rte_eth_rss_conf *rss_conf)
 {
        uint32_t update;
-       uint32_t cfg_rss_ctrl = 0;
-       uint8_t key;
        uint64_t rss_hf;
-       int i;
        struct nfp_net_hw *hw;
 
        hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
@@ -2477,41 +1510,18 @@ nfp_net_rss_hash_update(struct rte_eth_dev *dev,
        /* Checking if RSS is enabled */
        if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS)) {
                if (rss_hf != 0) { /* Enable RSS? */
-                       RTE_LOG(ERR, PMD, "RSS unsupported\n");
+                       PMD_DRV_LOG(ERR, "RSS unsupported");
                        return -EINVAL;
                }
                return 0; /* Nothing to do */
        }
 
        if (rss_conf->rss_key_len > NFP_NET_CFG_RSS_KEY_SZ) {
-               RTE_LOG(ERR, PMD, "hash key too long\n");
+               PMD_DRV_LOG(ERR, "hash key too long");
                return -EINVAL;
        }
 
-       if (rss_hf & ETH_RSS_IPV4)
-               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV4 |
-                               NFP_NET_CFG_RSS_IPV4_TCP |
-                               NFP_NET_CFG_RSS_IPV4_UDP;
-
-       if (rss_hf & ETH_RSS_IPV6)
-               cfg_rss_ctrl |= NFP_NET_CFG_RSS_IPV6 |
-                               NFP_NET_CFG_RSS_IPV6_TCP |
-                               NFP_NET_CFG_RSS_IPV6_UDP;
-
-       cfg_rss_ctrl |= NFP_NET_CFG_RSS_MASK;
-       cfg_rss_ctrl |= NFP_NET_CFG_RSS_TOEPLITZ;
-
-       /* configuring where to apply the RSS hash */
-       nn_cfg_writel(hw, NFP_NET_CFG_RSS_CTRL, cfg_rss_ctrl);
-
-       /* Writing the key byte a byte */
-       for (i = 0; i < rss_conf->rss_key_len; i++) {
-               memcpy(&key, &rss_conf->rss_key[i], 1);
-               nn_cfg_writeb(hw, NFP_NET_CFG_RSS_KEY + i, key);
-       }
-
-       /* Writing the key size */
-       nn_cfg_writeb(hw, NFP_NET_CFG_RSS_KEY_SZ, rss_conf->rss_key_len);
+       nfp_net_rss_hash_write(dev, rss_conf);
 
        update = NFP_NET_CFG_UPDATE_RSS;
 
@@ -2521,7 +1531,7 @@ nfp_net_rss_hash_update(struct rte_eth_dev *dev,
        return 0;
 }
 
-static int
+int
 nfp_net_rss_hash_conf_get(struct rte_eth_dev *dev,
                          struct rte_eth_rss_conf *rss_conf)
 {
@@ -2557,6 +1567,9 @@ nfp_net_rss_hash_conf_get(struct rte_eth_dev *dev,
        if (cfg_rss_ctrl & NFP_NET_CFG_RSS_IPV6)
                rss_hf |= ETH_RSS_NONFRAG_IPV4_UDP | ETH_RSS_NONFRAG_IPV6_UDP;
 
+       /* Propagate current RSS hash functions to caller */
+       rss_conf->rss_hf = rss_hf;
+
        /* Reading the key size */
        rss_conf->rss_key_len = nn_cfg_readl(hw, NFP_NET_CFG_RSS_KEY_SZ);
 
@@ -2569,11 +1582,54 @@ nfp_net_rss_hash_conf_get(struct rte_eth_dev *dev,
        return 0;
 }
 
+int
+nfp_net_rss_config_default(struct rte_eth_dev *dev)
+{
+       struct rte_eth_conf *dev_conf;
+       struct rte_eth_rss_conf rss_conf;
+       struct rte_eth_rss_reta_entry64 nfp_reta_conf[2];
+       uint16_t rx_queues = dev->data->nb_rx_queues;
+       uint16_t queue;
+       int i, j, ret;
+
+       PMD_DRV_LOG(INFO, "setting default RSS conf for %u queues",
+               rx_queues);
+
+       nfp_reta_conf[0].mask = ~0x0;
+       nfp_reta_conf[1].mask = ~0x0;
+
+       queue = 0;
+       for (i = 0; i < 0x40; i += 8) {
+               for (j = i; j < (i + 8); j++) {
+                       nfp_reta_conf[0].reta[j] = queue;
+                       nfp_reta_conf[1].reta[j] = queue++;
+                       queue %= rx_queues;
+               }
+       }
+       ret = nfp_net_rss_reta_write(dev, nfp_reta_conf, 0x80);
+       if (ret != 0)
+               return ret;
+
+       dev_conf = &dev->data->dev_conf;
+       if (!dev_conf) {
+               PMD_DRV_LOG(INFO, "wrong rss conf");
+               return -EINVAL;
+       }
+       rss_conf = dev_conf->rx_adv_conf.rss_conf;
+
+       ret = nfp_net_rss_hash_write(dev, &rss_conf);
+
+       return ret;
+}
+
+
 /* Initialise and register driver with DPDK Application */
 static const struct eth_dev_ops nfp_net_eth_dev_ops = {
        .dev_configure          = nfp_net_configure,
        .dev_start              = nfp_net_start,
        .dev_stop               = nfp_net_stop,
+       .dev_set_link_up        = nfp_net_set_link_up,
+       .dev_set_link_down      = nfp_net_set_link_down,
        .dev_close              = nfp_net_close,
        .promiscuous_enable     = nfp_net_promisc_enable,
        .promiscuous_disable    = nfp_net_promisc_disable,
@@ -2583,6 +1639,7 @@ static const struct eth_dev_ops nfp_net_eth_dev_ops = {
        .dev_infos_get          = nfp_net_infos_get,
        .dev_supported_ptypes_get = nfp_net_supported_ptypes_get,
        .mtu_set                = nfp_net_dev_mtu_set,
+       .mac_addr_set           = nfp_set_mac_addr,
        .vlan_offload_set       = nfp_net_vlan_offload_set,
        .reta_update            = nfp_net_reta_update,
        .reta_query             = nfp_net_reta_query,
@@ -2590,85 +1647,64 @@ static const struct eth_dev_ops nfp_net_eth_dev_ops = {
        .rss_hash_conf_get      = nfp_net_rss_hash_conf_get,
        .rx_queue_setup         = nfp_net_rx_queue_setup,
        .rx_queue_release       = nfp_net_rx_queue_release,
-       .rx_queue_count         = nfp_net_rx_queue_count,
        .tx_queue_setup         = nfp_net_tx_queue_setup,
        .tx_queue_release       = nfp_net_tx_queue_release,
        .rx_queue_intr_enable   = nfp_rx_queue_intr_enable,
        .rx_queue_intr_disable  = nfp_rx_queue_intr_disable,
 };
 
-/*
- * All eth_dev created got its private data, but before nfp_net_init, that
- * private data is referencing private data for all the PF ports. This is due
- * to how the vNIC bars are mapped based on first port, so all ports need info
- * about port 0 private data. Inside nfp_net_init the private data pointer is
- * changed to the right address for each port once the bars have been mapped.
- *
- * This functions helps to find out which port and therefore which offset
- * inside the private data array to use.
- */
-static int
-get_pf_port_number(char *name)
-{
-       char *pf_str = name;
-       int size = 0;
-
-       while ((*pf_str != '_') && (*pf_str != '\0') && (size++ < 30))
-               pf_str++;
-
-       if (size == 30)
-               /*
-                * This should not happen at all and it would mean major
-                * implementation fault.
-                */
-               rte_panic("nfp_net: problem with pf device name\n");
-
-       /* Expecting _portX with X within [0,7] */
-       pf_str += 5;
-
-       return (int)strtol(pf_str, NULL, 10);
-}
 
 static int
 nfp_net_init(struct rte_eth_dev *eth_dev)
 {
        struct rte_pci_device *pci_dev;
-       struct nfp_net_hw *hw, *hwport0;
+       struct nfp_pf_dev *pf_dev;
+       struct nfp_net_hw *hw;
 
        uint64_t tx_bar_off = 0, rx_bar_off = 0;
        uint32_t start_q;
        int stride = 4;
-
-       nspu_desc_t *nspu_desc = NULL;
-       uint64_t bar_offset;
        int port = 0;
+       int err;
 
        PMD_INIT_FUNC_TRACE();
 
        pci_dev = RTE_ETH_DEV_TO_PCI(eth_dev);
 
+       /* Use backpointer here to the PF of this eth_dev */
+       pf_dev = NFP_NET_DEV_PRIVATE_TO_PF(eth_dev->data->dev_private);
+
+       /* NFP can not handle DMA addresses requiring more than 40 bits */
+       if (rte_mem_check_dma_mask(40)) {
+               RTE_LOG(ERR, PMD, "device %s can not be used:",
+                                  pci_dev->device.name);
+               RTE_LOG(ERR, PMD, "\trestricted dma mask to 40 bits!\n");
+               return -ENODEV;
+       };
+
        if ((pci_dev->id.device_id == PCI_DEVICE_ID_NFP4000_PF_NIC) ||
            (pci_dev->id.device_id == PCI_DEVICE_ID_NFP6000_PF_NIC)) {
-               port = get_pf_port_number(eth_dev->data->name);
+               port = ((struct nfp_net_hw *)eth_dev->data->dev_private)->idx;
                if (port < 0 || port > 7) {
-                       RTE_LOG(ERR, PMD, "Port value is wrong\n");
+                       PMD_DRV_LOG(ERR, "Port value is wrong");
                        return -ENODEV;
                }
 
-               PMD_INIT_LOG(DEBUG, "Working with PF port value %d\n", port);
+               /* Use PF array of physical ports to get pointer to
+                * this specific port
+                */
+               hw = pf_dev->ports[port];
 
-               /* This points to port 0 private data */
-               hwport0 = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
+               PMD_INIT_LOG(DEBUG, "Working with physical port number: %d, "
+                                   "NFP internal port number: %d",
+                                   port, hw->nfp_idx);
 
-               /* This points to the specific port private data */
-               hw = &hwport0[port];
-               hw->pf_port_idx = port;
        } else {
                hw = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
-               hwport0 = 0;
        }
 
        eth_dev->dev_ops = &nfp_net_eth_dev_ops;
+       eth_dev->rx_queue_count = nfp_net_rx_queue_count;
        eth_dev->rx_pkt_burst = &nfp_net_recv_pkts;
        eth_dev->tx_pkt_burst = &nfp_net_xmit_pkts;
 
@@ -2690,38 +1726,26 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
 
        hw->ctrl_bar = (uint8_t *)pci_dev->mem_resource[0].addr;
        if (hw->ctrl_bar == NULL) {
-               RTE_LOG(ERR, PMD,
-                       "hw->ctrl_bar is NULL. BAR0 not configured\n");
+               PMD_DRV_LOG(ERR,
+                       "hw->ctrl_bar is NULL. BAR0 not configured");
                return -ENODEV;
        }
 
-       if (hw->is_pf && port == 0) {
-               nspu_desc = hw->nspu_desc;
-
-               if (nfp_nsp_map_ctrl_bar(nspu_desc, &bar_offset) != 0) {
-                       /*
-                        * A firmware should be there after PF probe so this
-                        * should not happen.
+       if (hw->is_phyport) {
+               if (port == 0) {
+                       hw->ctrl_bar = pf_dev->ctrl_bar;
+               } else {
+                       if (!pf_dev->ctrl_bar)
+                               return -ENODEV;
+                       /* Use port offset in pf ctrl_bar for this
+                        * ports control bar
                         */
-                       RTE_LOG(ERR, PMD, "PF BAR symbol resolution failed\n");
-                       return -ENODEV;
+                       hw->ctrl_bar = pf_dev->ctrl_bar +
+                                      (port * NFP_PF_CSR_SLICE_SIZE);
                }
-
-               /* vNIC PF control BAR is a subset of PF PCI device BAR */
-               hw->ctrl_bar += bar_offset;
-               PMD_INIT_LOG(DEBUG, "ctrl bar: %p\n", hw->ctrl_bar);
-       }
-
-       if (port > 0) {
-               if (!hwport0->ctrl_bar)
-                       return -ENODEV;
-
-               /* address based on port0 offset */
-               hw->ctrl_bar = hwport0->ctrl_bar +
-                              (port * NFP_PF_CSR_SLICE_SIZE);
        }
 
-       PMD_INIT_LOG(DEBUG, "ctrl bar: %p\n", hw->ctrl_bar);
+       PMD_INIT_LOG(DEBUG, "ctrl bar: %p", hw->ctrl_bar);
 
        hw->max_rx_queues = nn_cfg_readl(hw, NFP_NET_CFG_MAX_RXRINGS);
        hw->max_tx_queues = nn_cfg_readl(hw, NFP_NET_CFG_MAX_TXRINGS);
@@ -2732,36 +1756,22 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
        case PCI_DEVICE_ID_NFP6000_PF_NIC:
        case PCI_DEVICE_ID_NFP6000_VF_NIC:
                start_q = nn_cfg_readl(hw, NFP_NET_CFG_START_TXQ);
-               tx_bar_off = NFP_PCIE_QUEUE(start_q);
+               tx_bar_off = (uint64_t)start_q * NFP_QCP_QUEUE_ADDR_SZ;
                start_q = nn_cfg_readl(hw, NFP_NET_CFG_START_RXQ);
-               rx_bar_off = NFP_PCIE_QUEUE(start_q);
+               rx_bar_off = (uint64_t)start_q * NFP_QCP_QUEUE_ADDR_SZ;
                break;
        default:
-               RTE_LOG(ERR, PMD, "nfp_net: no device ID matching\n");
-               return -ENODEV;
+               PMD_DRV_LOG(ERR, "nfp_net: no device ID matching");
+               err = -ENODEV;
+               goto dev_err_ctrl_map;
        }
 
-       PMD_INIT_LOG(DEBUG, "tx_bar_off: 0x%" PRIx64 "\n", tx_bar_off);
-       PMD_INIT_LOG(DEBUG, "rx_bar_off: 0x%" PRIx64 "\n", rx_bar_off);
-
-       if (hw->is_pf && port == 0) {
-               /* configure access to tx/rx vNIC BARs */
-               nfp_nsp_map_queues_bar(nspu_desc, &bar_offset);
-               PMD_INIT_LOG(DEBUG, "tx/rx bar_offset: %" PRIx64 "\n",
-                                   bar_offset);
-               hwport0->hw_queues = (uint8_t *)pci_dev->mem_resource[0].addr;
+       PMD_INIT_LOG(DEBUG, "tx_bar_off: 0x%" PRIx64 "", tx_bar_off);
+       PMD_INIT_LOG(DEBUG, "rx_bar_off: 0x%" PRIx64 "", rx_bar_off);
 
-               /* vNIC PF tx/rx BARs are a subset of PF PCI device */
-               hwport0->hw_queues += bar_offset;
-
-               /* Lets seize the chance to read eth table from hw */
-               if (nfp_nsp_eth_read_table(nspu_desc, &hw->eth_table))
-                       return -ENODEV;
-       }
-
-       if (hw->is_pf) {
-               hw->tx_bar = hwport0->hw_queues + tx_bar_off;
-               hw->rx_bar = hwport0->hw_queues + rx_bar_off;
+       if (hw->is_phyport) {
+               hw->tx_bar = pf_dev->hw_queues + tx_bar_off;
+               hw->rx_bar = pf_dev->hw_queues + rx_bar_off;
                eth_dev->data->dev_private = hw;
        } else {
                hw->tx_bar = (uint8_t *)pci_dev->mem_resource[2].addr +
@@ -2779,16 +1789,22 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
        hw->ver = nn_cfg_readl(hw, NFP_NET_CFG_VERSION);
        hw->cap = nn_cfg_readl(hw, NFP_NET_CFG_CAP);
        hw->max_mtu = nn_cfg_readl(hw, NFP_NET_CFG_MAX_MTU);
-       hw->mtu = ETHER_MTU;
+       hw->mtu = RTE_ETHER_MTU;
+
+       /* VLAN insertion is incompatible with LSOv2 */
+       if (hw->cap & NFP_NET_CFG_CTRL_LSO2)
+               hw->cap &= ~NFP_NET_CFG_CTRL_TXVLAN;
 
        if (NFD_CFG_MAJOR_VERSION_of(hw->ver) < 2)
                hw->rx_offset = NFP_NET_RX_OFFSET;
        else
                hw->rx_offset = nn_cfg_readl(hw, NFP_NET_CFG_RX_OFFSET_ADDR);
 
-       PMD_INIT_LOG(INFO, "VER: %#x, Maximum supported MTU: %d",
-                    hw->ver, hw->max_mtu);
-       PMD_INIT_LOG(INFO, "CAP: %#x, %s%s%s%s%s%s%s%s%s%s%s", hw->cap,
+       PMD_INIT_LOG(INFO, "VER: %u.%u, Maximum supported MTU: %d",
+                          NFD_CFG_MAJOR_VERSION_of(hw->ver),
+                          NFD_CFG_MINOR_VERSION_of(hw->ver), hw->max_mtu);
+
+       PMD_INIT_LOG(INFO, "CAP: %#x, %s%s%s%s%s%s%s%s%s%s%s%s%s%s", hw->cap,
                     hw->cap & NFP_NET_CFG_CTRL_PROMISC ? "PROMISC " : "",
                     hw->cap & NFP_NET_CFG_CTRL_L2BC    ? "L2BCFILT " : "",
                     hw->cap & NFP_NET_CFG_CTRL_L2MC    ? "L2MCFILT " : "",
@@ -2798,8 +1814,11 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
                     hw->cap & NFP_NET_CFG_CTRL_TXVLAN  ? "TXVLAN "  : "",
                     hw->cap & NFP_NET_CFG_CTRL_SCATTER ? "SCATTER " : "",
                     hw->cap & NFP_NET_CFG_CTRL_GATHER  ? "GATHER "  : "",
+                    hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR ? "LIVE_ADDR "  : "",
                     hw->cap & NFP_NET_CFG_CTRL_LSO     ? "TSO "     : "",
-                    hw->cap & NFP_NET_CFG_CTRL_RSS     ? "RSS "     : "");
+                    hw->cap & NFP_NET_CFG_CTRL_LSO2     ? "TSOv2 "     : "",
+                    hw->cap & NFP_NET_CFG_CTRL_RSS     ? "RSS "     : "",
+                    hw->cap & NFP_NET_CFG_CTRL_RSS2     ? "RSSv2 "     : "");
 
        hw->ctrl = 0;
 
@@ -2813,29 +1832,37 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
        rte_spinlock_init(&hw->reconfig_lock);
 
        /* Allocating memory for mac addr */
-       eth_dev->data->mac_addrs = rte_zmalloc("mac_addr", ETHER_ADDR_LEN, 0);
+       eth_dev->data->mac_addrs = rte_zmalloc("mac_addr",
+                                              RTE_ETHER_ADDR_LEN, 0);
        if (eth_dev->data->mac_addrs == NULL) {
                PMD_INIT_LOG(ERR, "Failed to space for MAC address");
-               return -ENOMEM;
+               err = -ENOMEM;
+               goto dev_err_queues_map;
        }
 
-       if (hw->is_pf) {
-               nfp_net_pf_read_mac(hwport0, port);
+       if (hw->is_phyport) {
+               nfp_net_pf_read_mac(pf_dev, port);
                nfp_net_write_mac(hw, (uint8_t *)&hw->mac_addr);
-       } else {
-               nfp_net_vf_read_mac(hw);
        }
 
-       if (!is_valid_assigned_ether_addr((struct ether_addr *)&hw->mac_addr)) {
+       if (!rte_is_valid_assigned_ether_addr(
+                   (struct rte_ether_addr *)&hw->mac_addr)) {
+               PMD_INIT_LOG(INFO, "Using random mac address for port %d",
+                                  port);
                /* Using random mac addresses for VFs */
-               eth_random_addr(&hw->mac_addr[0]);
+               rte_eth_random_addr(&hw->mac_addr[0]);
                nfp_net_write_mac(hw, (uint8_t *)&hw->mac_addr);
        }
 
        /* Copying mac address to DPDK eth_dev struct */
-       ether_addr_copy((struct ether_addr *)hw->mac_addr,
+       rte_ether_addr_copy((struct rte_ether_addr *)hw->mac_addr,
                        &eth_dev->data->mac_addrs[0]);
 
+       if (!(hw->cap & NFP_NET_CFG_CTRL_LIVE_ADDR))
+               eth_dev->data->dev_flags |= RTE_ETH_DEV_NOLIVE_MAC_ADDR;
+
+       eth_dev->data->dev_flags |= RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS;
+
        PMD_INIT_LOG(INFO, "port %d VendorID=0x%x DeviceID=0x%x "
                     "mac=%02x:%02x:%02x:%02x:%02x:%02x",
                     eth_dev->data->port_id, pci_dev->id.vendor_id,
@@ -2843,152 +1870,446 @@ nfp_net_init(struct rte_eth_dev *eth_dev)
                     hw->mac_addr[0], hw->mac_addr[1], hw->mac_addr[2],
                     hw->mac_addr[3], hw->mac_addr[4], hw->mac_addr[5]);
 
-       /* Registering LSC interrupt handler */
-       rte_intr_callback_register(&pci_dev->intr_handle,
-                                  nfp_net_dev_interrupt_handler,
-                                  (void *)eth_dev);
+       if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
+               /* Registering LSC interrupt handler */
+               rte_intr_callback_register(&pci_dev->intr_handle,
+                                          nfp_net_dev_interrupt_handler,
+                                          (void *)eth_dev);
+               /* Telling the firmware about the LSC interrupt entry */
+               nn_cfg_writeb(hw, NFP_NET_CFG_LSC, NFP_NET_IRQ_LSC_IDX);
+               /* Recording current stats counters values */
+               nfp_net_stats_reset(eth_dev);
+       }
+
+       return 0;
+
+dev_err_queues_map:
+               nfp_cpp_area_free(hw->hwqueues_area);
+dev_err_ctrl_map:
+               nfp_cpp_area_free(hw->ctrl_area);
 
-       /* Telling the firmware about the LSC interrupt entry */
-       nn_cfg_writeb(hw, NFP_NET_CFG_LSC, NFP_NET_IRQ_LSC_IDX);
+       return err;
+}
 
-       /* Recording current stats counters values */
-       nfp_net_stats_reset(eth_dev);
+#define DEFAULT_FW_PATH       "/lib/firmware/netronome"
 
+static int
+nfp_fw_upload(struct rte_pci_device *dev, struct nfp_nsp *nsp, char *card)
+{
+       struct nfp_cpp *cpp = nsp->cpp;
+       void *fw_buf;
+       char fw_name[125];
+       char serial[40];
+       size_t fsize;
+
+       /* Looking for firmware file in order of priority */
+
+       /* First try to find a firmware image specific for this device */
+       snprintf(serial, sizeof(serial),
+                       "serial-%02x-%02x-%02x-%02x-%02x-%02x-%02x-%02x",
+               cpp->serial[0], cpp->serial[1], cpp->serial[2], cpp->serial[3],
+               cpp->serial[4], cpp->serial[5], cpp->interface >> 8,
+               cpp->interface & 0xff);
+
+       snprintf(fw_name, sizeof(fw_name), "%s/%s.nffw", DEFAULT_FW_PATH,
+                       serial);
+       PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name);
+       if (rte_firmware_read(fw_name, &fw_buf, &fsize) == 0)
+               goto load_fw;
+
+       /* Then try the PCI name */
+       snprintf(fw_name, sizeof(fw_name), "%s/pci-%s.nffw", DEFAULT_FW_PATH,
+                       dev->device.name);
+       PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name);
+       if (rte_firmware_read(fw_name, &fw_buf, &fsize) == 0)
+               goto load_fw;
+
+       /* Finally try the card type and media */
+       snprintf(fw_name, sizeof(fw_name), "%s/%s", DEFAULT_FW_PATH, card);
+       PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name);
+       if (rte_firmware_read(fw_name, &fw_buf, &fsize) < 0) {
+               PMD_DRV_LOG(INFO, "Firmware file %s not found.", fw_name);
+               return -ENOENT;
+       }
+
+load_fw:
+       PMD_DRV_LOG(INFO, "Firmware file found at %s with size: %zu",
+               fw_name, fsize);
+
+       PMD_DRV_LOG(INFO, "Uploading the firmware ...");
+       nfp_nsp_load_fw(nsp, fw_buf, fsize);
+       PMD_DRV_LOG(INFO, "Done");
+
+       free(fw_buf);
        return 0;
 }
 
 static int
-nfp_pf_create_dev(struct rte_pci_device *dev, int port, int ports,
-                 nfpu_desc_t *nfpu_desc, void **priv)
+nfp_fw_setup(struct rte_pci_device *dev, struct nfp_cpp *cpp,
+            struct nfp_eth_table *nfp_eth_table, struct nfp_hwinfo *hwinfo)
+{
+       struct nfp_nsp *nsp;
+       const char *nfp_fw_model;
+       char card_desc[100];
+       int err = 0;
+
+       nfp_fw_model = nfp_hwinfo_lookup(hwinfo, "assembly.partno");
+
+       if (nfp_fw_model) {
+               PMD_DRV_LOG(INFO, "firmware model found: %s", nfp_fw_model);
+       } else {
+               PMD_DRV_LOG(ERR, "firmware model NOT found");
+               return -EIO;
+       }
+
+       if (nfp_eth_table->count == 0 || nfp_eth_table->count > 8) {
+               PMD_DRV_LOG(ERR, "NFP ethernet table reports wrong ports: %u",
+                      nfp_eth_table->count);
+               return -EIO;
+       }
+
+       PMD_DRV_LOG(INFO, "NFP ethernet port table reports %u ports",
+                          nfp_eth_table->count);
+
+       PMD_DRV_LOG(INFO, "Port speed: %u", nfp_eth_table->ports[0].speed);
+
+       snprintf(card_desc, sizeof(card_desc), "nic_%s_%dx%d.nffw",
+                       nfp_fw_model, nfp_eth_table->count,
+                       nfp_eth_table->ports[0].speed / 1000);
+
+       nsp = nfp_nsp_open(cpp);
+       if (!nsp) {
+               PMD_DRV_LOG(ERR, "NFP error when obtaining NSP handle");
+               return -EIO;
+       }
+
+       nfp_nsp_device_soft_reset(nsp);
+       err = nfp_fw_upload(dev, nsp, card_desc);
+
+       nfp_nsp_close(nsp);
+       return err;
+}
+
+static int nfp_init_phyports(struct nfp_pf_dev *pf_dev)
 {
-       struct rte_eth_dev *eth_dev;
        struct nfp_net_hw *hw;
-       char *port_name;
-       int ret;
+       struct rte_eth_dev *eth_dev;
+       struct nfp_eth_table *nfp_eth_table = NULL;
+       int ret = 0;
+       int i;
+
+       nfp_eth_table = nfp_eth_read_ports(pf_dev->cpp);
+       if (!nfp_eth_table) {
+               PMD_INIT_LOG(ERR, "Error reading NFP ethernet table");
+               ret = -EIO;
+               goto error;
+       }
 
-       port_name = rte_zmalloc("nfp_pf_port_name", 100, 0);
-       if (!port_name)
-               return -ENOMEM;
+       /* Loop through all physical ports on PF */
+       for (i = 0; i < pf_dev->total_phyports; i++) {
+               const unsigned int numa_node = rte_socket_id();
+               char port_name[RTE_ETH_NAME_MAX_LEN];
 
-       if (ports > 1)
-               sprintf(port_name, "%s_port%d", dev->device.name, port);
-       else
-               sprintf(port_name, "%s", dev->device.name);
+               snprintf(port_name, sizeof(port_name), "%s_port%d",
+                        pf_dev->pci_dev->device.name, i);
 
-       eth_dev = rte_eth_dev_allocate(port_name);
-       if (!eth_dev)
-               return -ENOMEM;
+               /* Allocate a eth_dev for this phyport */
+               eth_dev = rte_eth_dev_allocate(port_name);
+               if (!eth_dev) {
+                       ret = -ENODEV;
+                       goto port_cleanup;
+               }
 
-       if (port == 0) {
-               *priv = rte_zmalloc(port_name,
-                                   sizeof(struct nfp_net_adapter) * ports,
-                                   RTE_CACHE_LINE_SIZE);
-               if (!*priv) {
+               /* Allocate memory for this phyport */
+               eth_dev->data->dev_private =
+                       rte_zmalloc_socket(port_name, sizeof(struct nfp_net_hw),
+                                          RTE_CACHE_LINE_SIZE, numa_node);
+               if (!eth_dev->data->dev_private) {
+                       ret = -ENOMEM;
                        rte_eth_dev_release_port(eth_dev);
-                       return -ENOMEM;
+                       goto port_cleanup;
                }
-       }
 
-       eth_dev->data->dev_private = *priv;
+               hw = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
 
-       /*
-        * dev_private pointing to port0 dev_private because we need
-        * to configure vNIC bars based on port0 at nfp_net_init.
-        * Then dev_private is adjusted per port.
-        */
-       hw = (struct nfp_net_hw *)(eth_dev->data->dev_private) + port;
-       hw->nspu_desc = nfpu_desc->nspu;
-       hw->nfpu_desc = nfpu_desc;
-       hw->is_pf = 1;
-       if (ports > 1)
-               hw->pf_multiport_enabled = 1;
+               /* Add this device to the PF's array of physical ports */
+               pf_dev->ports[i] = hw;
 
-       eth_dev->device = &dev->device;
-       rte_eth_copy_pci_info(eth_dev, dev);
+               hw->pf_dev = pf_dev;
+               hw->cpp = pf_dev->cpp;
+               hw->eth_dev = eth_dev;
+               hw->idx = i;
+               hw->nfp_idx = nfp_eth_table->ports[i].index;
+               hw->is_phyport = true;
 
-       ret = nfp_net_init(eth_dev);
+               eth_dev->device = &pf_dev->pci_dev->device;
 
-       if (ret)
-               rte_eth_dev_release_port(eth_dev);
+               /* ctrl/tx/rx BAR mappings and remaining init happens in
+                * nfp_net_init
+                */
+               ret = nfp_net_init(eth_dev);
+
+               if (ret) {
+                       ret = -ENODEV;
+                       goto port_cleanup;
+               }
+
+               rte_eth_dev_probing_finish(eth_dev);
 
-       rte_free(port_name);
+       } /* End loop, all ports on this PF */
+       ret = 0;
+       goto eth_table_cleanup;
 
+port_cleanup:
+       for (i = 0; i < pf_dev->total_phyports; i++) {
+               if (pf_dev->ports[i] && pf_dev->ports[i]->eth_dev) {
+                       struct rte_eth_dev *tmp_dev;
+                       tmp_dev = pf_dev->ports[i]->eth_dev;
+                       rte_eth_dev_release_port(tmp_dev);
+                       pf_dev->ports[i] = NULL;
+               }
+       }
+eth_table_cleanup:
+       free(nfp_eth_table);
+error:
        return ret;
 }
 
-static int nfp_pf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
-                           struct rte_pci_device *dev)
+static int nfp_pf_init(struct rte_pci_device *pci_dev)
 {
-       nfpu_desc_t *nfpu_desc;
-       nspu_desc_t *nspu_desc;
-       uint64_t offset_symbol;
-       uint8_t *bar_offset;
-       int major, minor;
+       struct nfp_pf_dev *pf_dev = NULL;
+       struct nfp_cpp *cpp;
+       struct nfp_hwinfo *hwinfo;
+       struct nfp_rtsym_table *sym_tbl;
+       struct nfp_eth_table *nfp_eth_table = NULL;
+       char name[RTE_ETH_NAME_MAX_LEN];
        int total_ports;
-       void *priv = 0;
        int ret = -ENODEV;
-       int i;
+       int err;
 
-       if (!dev)
+       if (!pci_dev)
                return ret;
 
-       nfpu_desc = rte_malloc("nfp nfpu", sizeof(nfpu_desc_t), 0);
-       if (!nfpu_desc)
-               return -ENOMEM;
+       /*
+        * When device bound to UIO, the device could be used, by mistake,
+        * by two DPDK apps, and the UIO driver does not avoid it. This
+        * could lead to a serious problem when configuring the NFP CPP
+        * interface. Here we avoid this telling to the CPP init code to
+        * use a lock file if UIO is being used.
+        */
+       if (pci_dev->kdrv == RTE_PCI_KDRV_VFIO)
+               cpp = nfp_cpp_from_device_name(pci_dev, 0);
+       else
+               cpp = nfp_cpp_from_device_name(pci_dev, 1);
 
-       if (nfpu_open(dev, nfpu_desc, 0) < 0) {
-               RTE_LOG(ERR, PMD,
-                       "nfpu_open failed\n");
-               goto nfpu_error;
+       if (!cpp) {
+               PMD_INIT_LOG(ERR, "A CPP handle can not be obtained");
+               ret = -EIO;
+               goto error;
        }
 
-       nspu_desc = nfpu_desc->nspu;
+       hwinfo = nfp_hwinfo_read(cpp);
+       if (!hwinfo) {
+               PMD_INIT_LOG(ERR, "Error reading hwinfo table");
+               ret = -EIO;
+               goto error;
+       }
 
+       nfp_eth_table = nfp_eth_read_ports(cpp);
+       if (!nfp_eth_table) {
+               PMD_INIT_LOG(ERR, "Error reading NFP ethernet table");
+               ret = -EIO;
+               goto hwinfo_cleanup;
+       }
 
-       /* Check NSP ABI version */
-       if (nfp_nsp_get_abi_version(nspu_desc, &major, &minor) < 0) {
-               RTE_LOG(INFO, PMD, "NFP NSP not present\n");
-               goto error;
+       if (nfp_fw_setup(pci_dev, cpp, nfp_eth_table, hwinfo)) {
+               PMD_INIT_LOG(ERR, "Error when uploading firmware");
+               ret = -EIO;
+               goto eth_table_cleanup;
        }
-       PMD_INIT_LOG(INFO, "nspu ABI version: %d.%d\n", major, minor);
 
-       if ((major == 0) && (minor < 20)) {
-               RTE_LOG(INFO, PMD, "NFP NSP ABI version too old. Required 0.20 or higher\n");
-               goto error;
+       /* Now the symbol table should be there */
+       sym_tbl = nfp_rtsym_table_read(cpp);
+       if (!sym_tbl) {
+               PMD_INIT_LOG(ERR, "Something is wrong with the firmware"
+                               " symbol table");
+               ret = -EIO;
+               goto eth_table_cleanup;
        }
 
-       ret = nfp_nsp_fw_setup(nspu_desc, "nfd_cfg_pf0_num_ports",
-                              &offset_symbol);
-       if (ret)
-               goto error;
+       total_ports = nfp_rtsym_read_le(sym_tbl, "nfd_cfg_pf0_num_ports", &err);
+       if (total_ports != (int)nfp_eth_table->count) {
+               PMD_DRV_LOG(ERR, "Inconsistent number of ports");
+               ret = -EIO;
+               goto sym_tbl_cleanup;
+       }
 
-       bar_offset = (uint8_t *)dev->mem_resource[0].addr;
-       bar_offset += offset_symbol;
-       total_ports = (uint32_t)*bar_offset;
-       PMD_INIT_LOG(INFO, "Total pf ports: %d\n", total_ports);
+       PMD_INIT_LOG(INFO, "Total physical ports: %d", total_ports);
 
        if (total_ports <= 0 || total_ports > 8) {
-               RTE_LOG(ERR, PMD, "nfd_cfg_pf0_num_ports symbol with wrong value");
+               PMD_INIT_LOG(ERR, "nfd_cfg_pf0_num_ports symbol with wrong value");
                ret = -ENODEV;
-               goto error;
+               goto sym_tbl_cleanup;
+       }
+       /* Allocate memory for the PF "device" */
+       snprintf(name, sizeof(name), "nfp_pf%d", 0);
+       pf_dev = rte_zmalloc(name, sizeof(*pf_dev), 0);
+       if (!pf_dev) {
+               ret = -ENOMEM;
+               goto sym_tbl_cleanup;
        }
 
-       for (i = 0; i < total_ports; i++) {
-               ret = nfp_pf_create_dev(dev, i, total_ports, nfpu_desc, &priv);
-               if (ret)
-                       goto error;
+       /* Populate the newly created PF device */
+       pf_dev->cpp = cpp;
+       pf_dev->hwinfo = hwinfo;
+       pf_dev->sym_tbl = sym_tbl;
+       pf_dev->total_phyports = total_ports;
+
+       if (total_ports > 1)
+               pf_dev->multiport = true;
+
+       pf_dev->pci_dev = pci_dev;
+
+       /* Map the symbol table */
+       pf_dev->ctrl_bar = nfp_rtsym_map(pf_dev->sym_tbl, "_pf0_net_bar0",
+                                    pf_dev->total_phyports * 32768,
+                                    &pf_dev->ctrl_area);
+       if (!pf_dev->ctrl_bar) {
+               PMD_INIT_LOG(ERR, "nfp_rtsym_map fails for _pf0_net_ctrl_bar");
+               ret = -EIO;
+               goto pf_cleanup;
+       }
+
+       PMD_INIT_LOG(DEBUG, "ctrl bar: %p", pf_dev->ctrl_bar);
+
+       /* configure access to tx/rx vNIC BARs */
+       pf_dev->hw_queues = nfp_cpp_map_area(pf_dev->cpp, 0, 0,
+                                             NFP_PCIE_QUEUE(0),
+                                             NFP_QCP_QUEUE_AREA_SZ,
+                                             &pf_dev->hwqueues_area);
+       if (!pf_dev->hw_queues) {
+               PMD_INIT_LOG(ERR, "nfp_rtsym_map fails for net.qc");
+               ret = -EIO;
+               goto ctrl_area_cleanup;
+       }
+
+       PMD_INIT_LOG(DEBUG, "tx/rx bar address: 0x%p", pf_dev->hw_queues);
+
+       /* Initialize and prep physical ports now
+        * This will loop through all physical ports
+        */
+       ret = nfp_init_phyports(pf_dev);
+       if (ret) {
+               PMD_INIT_LOG(ERR, "Could not create physical ports");
+               goto hwqueues_cleanup;
        }
 
+       /* register the CPP bridge service here for primary use */
+       nfp_register_cpp_service(pf_dev->cpp);
+
        return 0;
 
+hwqueues_cleanup:
+       nfp_cpp_area_free(pf_dev->hwqueues_area);
+ctrl_area_cleanup:
+       nfp_cpp_area_free(pf_dev->ctrl_area);
+pf_cleanup:
+       rte_free(pf_dev);
+sym_tbl_cleanup:
+       free(sym_tbl);
+eth_table_cleanup:
+       free(nfp_eth_table);
+hwinfo_cleanup:
+       free(hwinfo);
 error:
-       nfpu_close(nfpu_desc);
-nfpu_error:
-       rte_free(nfpu_desc);
-
        return ret;
 }
 
+/*
+ * When attaching to the NFP4000/6000 PF on a secondary process there
+ * is no need to initialize the PF again. Only minimal work is required
+ * here
+ */
+static int nfp_pf_secondary_init(struct rte_pci_device *pci_dev)
+{
+       struct nfp_cpp *cpp;
+       struct nfp_rtsym_table *sym_tbl;
+       int total_ports;
+       int i;
+       int err;
+
+       if (!pci_dev)
+               return -ENODEV;
+
+       /*
+        * When device bound to UIO, the device could be used, by mistake,
+        * by two DPDK apps, and the UIO driver does not avoid it. This
+        * could lead to a serious problem when configuring the NFP CPP
+        * interface. Here we avoid this telling to the CPP init code to
+        * use a lock file if UIO is being used.
+        */
+       if (pci_dev->kdrv == RTE_PCI_KDRV_VFIO)
+               cpp = nfp_cpp_from_device_name(pci_dev, 0);
+       else
+               cpp = nfp_cpp_from_device_name(pci_dev, 1);
+
+       if (!cpp) {
+               PMD_INIT_LOG(ERR, "A CPP handle can not be obtained");
+               return -EIO;
+       }
+
+       /*
+        * We don't have access to the PF created in the primary process
+        * here so we have to read the number of ports from firmware
+        */
+       sym_tbl = nfp_rtsym_table_read(cpp);
+       if (!sym_tbl) {
+               PMD_INIT_LOG(ERR, "Something is wrong with the firmware"
+                               " symbol table");
+               return -EIO;
+       }
+
+       total_ports = nfp_rtsym_read_le(sym_tbl, "nfd_cfg_pf0_num_ports", &err);
+
+       for (i = 0; i < total_ports; i++) {
+               struct rte_eth_dev *eth_dev;
+               char port_name[RTE_ETH_NAME_MAX_LEN];
+
+               snprintf(port_name, sizeof(port_name), "%s_port%d",
+                        pci_dev->device.name, i);
+
+               PMD_DRV_LOG(DEBUG, "Secondary attaching to port %s",
+                   port_name);
+               eth_dev = rte_eth_dev_attach_secondary(port_name);
+               if (!eth_dev) {
+                       RTE_LOG(ERR, EAL,
+                       "secondary process attach failed, "
+                       "ethdev doesn't exist");
+                       return -ENODEV;
+               }
+               eth_dev->process_private = cpp;
+               eth_dev->dev_ops = &nfp_net_eth_dev_ops;
+               eth_dev->rx_queue_count = nfp_net_rx_queue_count;
+               eth_dev->rx_pkt_burst = &nfp_net_recv_pkts;
+               eth_dev->tx_pkt_burst = &nfp_net_xmit_pkts;
+               rte_eth_dev_probing_finish(eth_dev);
+       }
+
+       /* Register the CPP bridge service for the secondary too */
+       nfp_register_cpp_service(cpp);
+
+       return 0;
+}
+
+static int nfp_pf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
+                           struct rte_pci_device *dev)
+{
+       if (rte_eal_process_type() == RTE_PROC_PRIMARY)
+               return nfp_pf_init(dev);
+       else
+               return nfp_pf_secondary_init(dev);
+}
+
 static const struct rte_pci_id pci_id_nfp_pf_net_map[] = {
        {
                RTE_PCI_DEVICE(PCI_VENDOR_ID_NETRONOME,
@@ -3003,42 +2324,32 @@ static const struct rte_pci_id pci_id_nfp_pf_net_map[] = {
        },
 };
 
-static const struct rte_pci_id pci_id_nfp_vf_net_map[] = {
-       {
-               RTE_PCI_DEVICE(PCI_VENDOR_ID_NETRONOME,
-                              PCI_DEVICE_ID_NFP6000_VF_NIC)
-       },
-       {
-               .vendor_id = 0,
-       },
-};
-
-static int eth_nfp_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
-       struct rte_pci_device *pci_dev)
+static int nfp_pci_uninit(struct rte_eth_dev *eth_dev)
 {
-       return rte_eth_dev_pci_generic_probe(pci_dev,
-               sizeof(struct nfp_net_adapter), nfp_net_init);
+       struct rte_pci_device *pci_dev;
+       uint16_t port_id;
+
+       pci_dev = RTE_ETH_DEV_TO_PCI(eth_dev);
+
+       if (pci_dev->id.device_id == PCI_DEVICE_ID_NFP4000_PF_NIC ||
+           pci_dev->id.device_id == PCI_DEVICE_ID_NFP6000_PF_NIC) {
+               /* Free up all physical ports under PF */
+               RTE_ETH_FOREACH_DEV_OF(port_id, &pci_dev->device)
+                       rte_eth_dev_close(port_id);
+               /*
+                * Ports can be closed and freed but hotplugging is not
+                * currently supported
+                */
+               return -ENOTSUP;
+       }
+
+       /* VF cleanup, just free private port data */
+       return nfp_net_close(eth_dev);
 }
 
 static int eth_nfp_pci_remove(struct rte_pci_device *pci_dev)
 {
-       struct rte_eth_dev *eth_dev;
-       struct nfp_net_hw *hw, *hwport0;
-       int port = 0;
-
-       eth_dev = rte_eth_dev_allocated(pci_dev->device.name);
-       if ((pci_dev->id.device_id == PCI_DEVICE_ID_NFP4000_PF_NIC) ||
-           (pci_dev->id.device_id == PCI_DEVICE_ID_NFP6000_PF_NIC)) {
-               port = get_pf_port_number(eth_dev->data->name);
-               hwport0 = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
-               hw = &hwport0[port];
-       } else {
-               hw = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
-       }
-       /* hotplug is not possible with multiport PF */
-       if (hw->pf_multiport_enabled)
-               return -ENOTSUP;
-       return rte_eth_dev_pci_generic_remove(pci_dev, NULL);
+       return rte_eth_dev_pci_generic_remove(pci_dev, nfp_pci_uninit);
 }
 
 static struct rte_pci_driver rte_nfp_net_pf_pmd = {
@@ -3048,20 +2359,11 @@ static struct rte_pci_driver rte_nfp_net_pf_pmd = {
        .remove = eth_nfp_pci_remove,
 };
 
-static struct rte_pci_driver rte_nfp_net_vf_pmd = {
-       .id_table = pci_id_nfp_vf_net_map,
-       .drv_flags = RTE_PCI_DRV_NEED_MAPPING | RTE_PCI_DRV_INTR_LSC,
-       .probe = eth_nfp_pci_probe,
-       .remove = eth_nfp_pci_remove,
-};
-
 RTE_PMD_REGISTER_PCI(net_nfp_pf, rte_nfp_net_pf_pmd);
-RTE_PMD_REGISTER_PCI(net_nfp_vf, rte_nfp_net_vf_pmd);
 RTE_PMD_REGISTER_PCI_TABLE(net_nfp_pf, pci_id_nfp_pf_net_map);
-RTE_PMD_REGISTER_PCI_TABLE(net_nfp_vf, pci_id_nfp_vf_net_map);
 RTE_PMD_REGISTER_KMOD_DEP(net_nfp_pf, "* igb_uio | uio_pci_generic | vfio");
-RTE_PMD_REGISTER_KMOD_DEP(net_nfp_vf, "* igb_uio | uio_pci_generic | vfio");
-
+RTE_LOG_REGISTER_SUFFIX(nfp_logtype_init, init, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(nfp_logtype_driver, driver, NOTICE);
 /*
  * Local variables:
  * c-file-style: "Linux"