Re: [PATCH v6] vfio/cdx: add support for CDX bus

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 5/25/2023 1:18 AM, Alex Williamson wrote:

On Wed, 24 May 2023 10:45:29 -0600
Alex Williamson <alex.williamson@xxxxxxxxxx> wrote:

On Wed, 17 May 2023 15:27:18 +0530
Nipun Gupta <nipun.gupta@xxxxxxx> wrote:


<snip>

+
+MODULE_DEVICE_TABLE(cdx, vfio_cdx_table);
+
+static struct cdx_driver vfio_cdx_driver = {
+   .probe          = vfio_cdx_probe,
+   .remove         = vfio_cdx_remove,
+   .match_id_table = vfio_cdx_table,
+   .driver = {
+           .name   = "vfio-cdx",
+           .owner  = THIS_MODULE,
+   },
+   .driver_managed_dma = true,

Hmm, looks like cdx bus is broken here, there's no actual
implementation of a dma_configure callback and no setup of the IOMMU
default domain for theoretical cdx drivers that might want to use the
DMA API.  Without that, this driver_manged_dma flag doesn't provide any
guarantees to a vfio driver that we have exclusive ownership of the
group.  Please fix, this flag needs to actually have some meaning on
cdx.  Thanks,

Alex

Agree, this change was missed on CDX bus and we are working on fixing this. Shall I send this fix as a commit with this patch?

Thanks,
Nipun



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux