test/kvargs: fix invalid cases check
[dpdk.git] / drivers / bus / pci / pci_common.c
index 41f8fc2..3f55420 100644 (file)
@@ -169,8 +169,22 @@ rte_pci_probe_one_driver(struct rte_pci_driver *dr,
         * This needs to be before rte_pci_map_device(), as it enables to use
         * driver flags for adjusting configuration.
         */
-       if (!already_probed)
+       if (!already_probed) {
+               enum rte_iova_mode dev_iova_mode;
+               enum rte_iova_mode iova_mode;
+
+               dev_iova_mode = pci_device_iova_mode(dr, dev);
+               iova_mode = rte_eal_iova_mode();
+               if (dev_iova_mode != RTE_IOVA_DC &&
+                   dev_iova_mode != iova_mode) {
+                       RTE_LOG(ERR, EAL, "  Expecting '%s' IOVA mode but current mode is '%s', not initializing\n",
+                               dev_iova_mode == RTE_IOVA_PA ? "PA" : "VA",
+                               iova_mode == RTE_IOVA_PA ? "PA" : "VA");
+                       return -EINVAL;
+               }
+
                dev->driver = dr;
+       }
 
        if (!already_probed && (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING)) {
                /* map resources for devices that use igb_uio */
@@ -233,6 +247,7 @@ rte_pci_detach_dev(struct rte_pci_device *dev)
 
        /* clear driver structure */
        dev->driver = NULL;
+       dev->device.driver = NULL;
 
        if (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING)
                /* unmap resources for devices that use igb_uio */
@@ -243,7 +258,7 @@ rte_pci_detach_dev(struct rte_pci_device *dev)
 
 /*
  * If vendor/device ID match, call the probe() function of all
- * registered driver for the given device. Return -1 if initialization
+ * registered driver for the given device. Return < 0 if initialization
  * failed, return 1 if no driver is found for this device.
  */
 static int
@@ -253,13 +268,13 @@ pci_probe_all_drivers(struct rte_pci_device *dev)
        int rc = 0;
 
        if (dev == NULL)
-               return -1;
+               return -EINVAL;
 
        FOREACH_DRIVER_ON_PCIBUS(dr) {
                rc = rte_pci_probe_one_driver(dr, dev);
                if (rc < 0)
                        /* negative value is an error */
-                       return -1;
+                       return rc;
                if (rc > 0)
                        /* positive value means driver doesn't support it */
                        continue;
@@ -522,11 +537,141 @@ pci_unplug(struct rte_device *dev)
        ret = rte_pci_detach_dev(pdev);
        if (ret == 0) {
                rte_pci_remove_device(pdev);
+               rte_devargs_remove(dev->devargs);
                free(pdev);
        }
        return ret;
 }
 
+static int
+pci_dma_map(struct rte_device *dev, void *addr, uint64_t iova, size_t len)
+{
+       struct rte_pci_device *pdev = RTE_DEV_TO_PCI(dev);
+
+       if (!pdev || !pdev->driver) {
+               rte_errno = EINVAL;
+               return -1;
+       }
+       if (pdev->driver->dma_map)
+               return pdev->driver->dma_map(pdev, addr, iova, len);
+       /**
+        *  In case driver don't provides any specific mapping
+        *  try fallback to VFIO.
+        */
+       if (pdev->kdrv == RTE_KDRV_VFIO)
+               return rte_vfio_container_dma_map
+                               (RTE_VFIO_DEFAULT_CONTAINER_FD, (uintptr_t)addr,
+                                iova, len);
+       rte_errno = ENOTSUP;
+       return -1;
+}
+
+static int
+pci_dma_unmap(struct rte_device *dev, void *addr, uint64_t iova, size_t len)
+{
+       struct rte_pci_device *pdev = RTE_DEV_TO_PCI(dev);
+
+       if (!pdev || !pdev->driver) {
+               rte_errno = EINVAL;
+               return -1;
+       }
+       if (pdev->driver->dma_unmap)
+               return pdev->driver->dma_unmap(pdev, addr, iova, len);
+       /**
+        *  In case driver don't provides any specific mapping
+        *  try fallback to VFIO.
+        */
+       if (pdev->kdrv == RTE_KDRV_VFIO)
+               return rte_vfio_container_dma_unmap
+                               (RTE_VFIO_DEFAULT_CONTAINER_FD, (uintptr_t)addr,
+                                iova, len);
+       rte_errno = ENOTSUP;
+       return -1;
+}
+
+static bool
+pci_ignore_device(const struct rte_pci_device *dev)
+{
+       struct rte_devargs *devargs = dev->device.devargs;
+
+       switch (rte_pci_bus.bus.conf.scan_mode) {
+       case RTE_BUS_SCAN_WHITELIST:
+               if (devargs && devargs->policy == RTE_DEV_WHITELISTED)
+                       return false;
+               break;
+       case RTE_BUS_SCAN_UNDEFINED:
+       case RTE_BUS_SCAN_BLACKLIST:
+               if (devargs == NULL ||
+                   devargs->policy != RTE_DEV_BLACKLISTED)
+                       return false;
+               break;
+       }
+       return true;
+}
+
+enum rte_iova_mode
+rte_pci_get_iommu_class(void)
+{
+       enum rte_iova_mode iova_mode = RTE_IOVA_DC;
+       const struct rte_pci_device *dev;
+       const struct rte_pci_driver *drv;
+       bool devices_want_va = false;
+       bool devices_want_pa = false;
+       int iommu_no_va = -1;
+
+       FOREACH_DEVICE_ON_PCIBUS(dev) {
+               /*
+                * We can check this only once, because the IOMMU hardware is
+                * the same for all of them.
+                */
+               if (iommu_no_va == -1)
+                       iommu_no_va = pci_device_iommu_support_va(dev)
+                                       ? 0 : 1;
+               if (pci_ignore_device(dev))
+                       continue;
+               if (dev->kdrv == RTE_KDRV_UNKNOWN ||
+                   dev->kdrv == RTE_KDRV_NONE)
+                       continue;
+               FOREACH_DRIVER_ON_PCIBUS(drv) {
+                       enum rte_iova_mode dev_iova_mode;
+
+                       if (!rte_pci_match(drv, dev))
+                               continue;
+
+                       dev_iova_mode = pci_device_iova_mode(drv, dev);
+                       RTE_LOG(DEBUG, EAL, "PCI driver %s for device "
+                               PCI_PRI_FMT " wants IOVA as '%s'\n",
+                               drv->driver.name,
+                               dev->addr.domain, dev->addr.bus,
+                               dev->addr.devid, dev->addr.function,
+                               dev_iova_mode == RTE_IOVA_DC ? "DC" :
+                               (dev_iova_mode == RTE_IOVA_PA ? "PA" : "VA"));
+                       if (dev_iova_mode == RTE_IOVA_PA)
+                               devices_want_pa = true;
+                       else if (dev_iova_mode == RTE_IOVA_VA)
+                               devices_want_va = true;
+               }
+       }
+       if (iommu_no_va == 1) {
+               iova_mode = RTE_IOVA_PA;
+               if (devices_want_va) {
+                       RTE_LOG(WARNING, EAL, "Some devices want 'VA' but IOMMU does not support 'VA'.\n");
+                       RTE_LOG(WARNING, EAL, "The devices that want 'VA' won't initialize.\n");
+               }
+       } else if (devices_want_va && !devices_want_pa) {
+               iova_mode = RTE_IOVA_VA;
+       } else if (devices_want_pa && !devices_want_va) {
+               iova_mode = RTE_IOVA_PA;
+       } else {
+               iova_mode = RTE_IOVA_DC;
+               if (devices_want_va) {
+                       RTE_LOG(WARNING, EAL, "Some devices want 'VA' but forcing 'DC' because other devices want 'PA'.\n");
+                       RTE_LOG(WARNING, EAL, "Depending on the final decision by the EAL, not all devices may be able to initialize.\n");
+               }
+       }
+       return iova_mode;
+}
+
 struct rte_pci_bus rte_pci_bus = {
        .bus = {
                .scan = rte_pci_scan,
@@ -535,6 +680,8 @@ struct rte_pci_bus rte_pci_bus = {
                .plug = pci_plug,
                .unplug = pci_unplug,
                .parse = pci_parse,
+               .dma_map = pci_dma_map,
+               .dma_unmap = pci_dma_unmap,
                .get_iommu_class = rte_pci_get_iommu_class,
                .dev_iterate = rte_pci_dev_iterate,
                .hot_unplug_handler = pci_hot_unplug_handler,