lkml.org 
[lkml]   [2022]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v5 09/14] PCI: portdrv: Suppress kernel DMA ownership auto-claiming
    On Tue, Jan 04, 2022 at 09:56:39AM +0800, Lu Baolu wrote:
    > If a switch lacks ACS P2P Request Redirect, a device below the switch can
    > bypass the IOMMU and DMA directly to other devices below the switch, so
    > all the downstream devices must be in the same IOMMU group as the switch
    > itself.

    Help me think through what's going on here. IIUC, we put devices in
    the same IOMMU group when they can interfere with each other in any
    way (DMA, config access, etc).

    (We said "DMA" above, but I guess this would also apply to config
    requests, right?)

    *This* patch doesn't check for any ACS features. Can you connect the
    dots for me? I guess the presence or absence of P2P Request Redirect
    determines the size of the IOMMU group. And the following says
    something about what is allowed in the group? And .no_kernel_api_dma
    allows an exception to the general rule?

    > The existing vfio framework allows the portdrv driver to be bound
    > to the bridge while its downstream devices are assigned to user space.

    I.e., the existing VFIO framework allows a switch to be in the same
    IOMMU group as the devices below it, even though the switch has a
    kernel driver and the other devices may have userspace drivers?

    Is this a function of VFIO design or of the IOMMU driver?

    > The pci_dma_configure() marks the iommu_group as containing only devices
    > with kernel drivers that manage DMA. Avoid this default behavior for the
    > portdrv driver in order for compatibility with the current vfio policy.

    I assume "IOMMU group" means the same as "iommu_group"; maybe we can
    use one of them consistently?

    > Suggested-by: Jason Gunthorpe <jgg@nvidia.com>
    > Suggested-by: Kevin Tian <kevin.tian@intel.com>
    > Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
    > ---
    > drivers/pci/pcie/portdrv_pci.c | 2 ++
    > 1 file changed, 2 insertions(+)
    >
    > diff --git a/drivers/pci/pcie/portdrv_pci.c b/drivers/pci/pcie/portdrv_pci.c
    > index 35eca6277a96..2116f821c005 100644
    > --- a/drivers/pci/pcie/portdrv_pci.c
    > +++ b/drivers/pci/pcie/portdrv_pci.c
    > @@ -202,6 +202,8 @@ static struct pci_driver pcie_portdriver = {
    >
    > .err_handler = &pcie_portdrv_err_handler,
    >
    > + .no_kernel_api_dma = true,
    > +
    > .driver.pm = PCIE_PORTDRV_PM_OPS,
    > };
    >
    > --
    > 2.25.1
    >

    \
     
     \ /
      Last update: 2022-01-04 18:07    [W:4.302 / U:0.168 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site