ethdev: remove useless check in detach capability
authorGaetan Rivet <gaetan.rivet@6wind.com>
Wed, 26 Jul 2017 13:35:53 +0000 (15:35 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 31 Jul 2017 12:08:25 +0000 (14:08 +0200)
This capability is not bound to which driver is handling the device, but
whether the bus is able to unplug it.

This check is already performed in rte_eal_dev_detach, there is no need
to do it in the ethdev layer.

Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
lib/librte_ether/rte_ethdev.c

index 8c365ed..805ef63 100644 (file)
@@ -367,16 +367,6 @@ rte_eth_dev_is_detachable(uint8_t port_id)
 
        RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -EINVAL);
 
-       switch (rte_eth_devices[port_id].data->kdrv) {
-       case RTE_KDRV_IGB_UIO:
-       case RTE_KDRV_UIO_GENERIC:
-       case RTE_KDRV_NIC_UIO:
-       case RTE_KDRV_NONE:
-       case RTE_KDRV_VFIO:
-               break;
-       default:
-               return -ENOTSUP;
-       }
        dev_flags = rte_eth_devices[port_id].data->dev_flags;
        if ((dev_flags & RTE_ETH_DEV_DETACHABLE) &&
                (!(dev_flags & RTE_ETH_DEV_BONDED_SLAVE)))