> From: Xu, Quan > Sent: Wednesday, June 29, 2016 2:00 PM > > From: Quan Xu <quan...@intel.com> > > A struct pci_dev* instead of SBDF is stored inside struct > pci_ats_dev and parameter to *_ats_device(). > > Also use ats_dev for "struct pci_ats_dev" variable, while > pdev (_pdev, if there is already a pdev) for "struct pci_dev" > in the scope of IOMMU. > > Signed-off-by: Quan Xu <quan...@intel.com> > > CC: Jan Beulich <jbeul...@suse.com> > CC: Kevin Tian <kevin.t...@intel.com> > CC: Feng Wu <feng...@intel.com> > CC: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> >
Acked-by: Kevin Tian <kevin.t...@intel.com>, with one minor comment: > diff --git a/xen/drivers/passthrough/amd/iommu_cmd.c > b/xen/drivers/passthrough/amd/iommu_cmd.c > index 7c9d9be..934977a 100644 > --- a/xen/drivers/passthrough/amd/iommu_cmd.c > +++ b/xen/drivers/passthrough/amd/iommu_cmd.c > @@ -294,28 +294,27 @@ void amd_iommu_flush_iotlb(u8 devfn, const struct > pci_dev > *pdev, > if ( !ats_enabled ) > return; > > - ats_pdev = get_ats_device(pdev->seg, pdev->bus, pdev->devfn); > + ats_pdev = get_ats_device(pdev); > if ( ats_pdev == NULL ) > return; > To align with other places, better to change ats_pdev to ats_dev too. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
