On Fri, Mar 11, 2016 at 04:46:23AM +0000, Tian, Kevin wrote: > > From: Neo Jia [mailto:cjia@xxxxxxxxxx] > > Sent: Friday, March 11, 2016 12:20 PM > > > > On Thu, Mar 10, 2016 at 11:10:10AM +0800, Jike Song wrote: > > > > > > >> Is it supposed to be the caller who should set > > > >> up IOMMU by DMA api such as dma_map_page(), after calling > > > >> vgpu_dma_do_translate()? > > > >> > > > > > > > > Don't think you need to call dma_map_page here. Once you have the pfn available > > > > to your GPU kernel driver, you can just go ahead to setup the mapping as you > > > > normally do such as calling pci_map_sg and its friends. > > > > > > > > > > Technically it's definitely OK to call DMA API from the caller rather than here, > > > however personally I think it is a bit counter-intuitive: IOMMU page tables > > > should be constructed within the VFIO IOMMU driver. > > > > > > > Hi Jike, > > > > For vGPU, what we have is just a virtual device and a fake IOMMU group, therefore > > the actual interaction with the real GPU should be managed by the GPU vendor driver. > > > > Hi, Neo, > > Seems we have a different thought on this. Regardless of whether it's a virtual/physical > device, imo, VFIO should manage IOMMU configuration. The only difference is: > > - for physical device, VFIO directly invokes IOMMU API to set IOMMU entry (GPA->HPA); > - for virtual device, VFIO invokes kernel DMA APIs which indirectly lead to IOMMU entry > set if CONFIG_IOMMU is enabled in kernel (GPA->IOVA); How does it make any sense for us to do a dma_map_page for a physical device that we don't have any direct interaction with? > > This would provide an unified way to manage the translation in VFIO, and then vendor > specific driver only needs to query and use returned IOVA corresponding to a GPA. > > Doing so has another benefit, to make underlying vGPU driver VMM agnostic. For KVM, > yes we can use pci_map_sg. However for Xen it's different (today Dom0 doesn't see > IOMMU. In the future there'll be a PVIOMMU implementation) so different code path is > required. It's better to abstract such specific knowledge out of vGPU driver, which just > uses whatever dma_addr returned by other agent (VFIO here, or another Xen specific > agent) in a centralized way. > > Alex, what's your opinion on this? > > Thanks > Kevin -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
