net/nfp: avoid unplug if multiport
authorAlejandro Lucero <alejandro.lucero@netronome.com>
Tue, 31 Oct 2017 11:04:52 +0000 (11:04 +0000)
committerFerruh Yigit <ferruh.yigit@intel.com>
Wed, 1 Nov 2017 21:17:06 +0000 (22:17 +0100)
PF multiport support has some limitations like not allowing plugging or
unplugging of single ports as they all belong to same PCI device.

A previous patch for removing detachable flag introduced a wrong check.

Fixes: 00a3d8104ac5 ("ethdev: remove detachable device flag")

Signed-off-by: Alejandro Lucero <alejandro.lucero@netronome.com>
drivers/net/nfp/nfp_net.c

index e9001f4..83dec06 100644 (file)
@@ -3029,7 +3029,7 @@ static int eth_nfp_pci_remove(struct rte_pci_device *pci_dev)
                hw = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private);
        }
        /* hotplug is not possible with multiport PF */
-       if (!hw->pf_multiport_enabled)
+       if (hw->pf_multiport_enabled)
                return -ENOTSUP;
        return rte_eth_dev_pci_generic_remove(pci_dev, NULL);
 }