lkml.org 
[lkml]   [2023]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v6] vfio/cdx: add support for CDX bus
On Mon, May 29, 2023 at 01:36:30PM +0530, Nipun Gupta wrote:
>
>
> On 5/25/2023 1:18 AM, Alex Williamson wrote:
> >
> > On Wed, 24 May 2023 10:45:29 -0600
> > Alex Williamson <alex.williamson@redhat.com> wrote:
> >
> > > On Wed, 17 May 2023 15:27:18 +0530
> > > Nipun Gupta <nipun.gupta@amd.com> 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?

You should send it as a rc fixup for your already merged CDX bus stuff

Jason

\
 
 \ /
  Last update: 2023-06-01 16:15    [W:0.042 / U:0.984 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site