On Tue, Feb 15, 2022 at 04:00:35PM +0000, Joao Martins wrote: > On 2/14/22 14:06, Jason Gunthorpe wrote: > > On Mon, Feb 14, 2022 at 01:34:15PM +0000, Joao Martins wrote: > > > >> [*] apparently we need to write an invalid entry first, invalidate the {IO}TLB > >> and then write the new valid entry. Not sure I understood correctly that this > >> is the 'break-before-make' thingie. > > > > Doesn't that explode if the invalid entry is DMA'd to? > > > Yes, IIUC. Also, the manual has this note: Heh, sounds like "this doesn't work" to me :) > > Like I said, I'd prefer we not build more on the VFIO type 1 code > > until we have a conclusion for iommufd.. > > > > I didn't quite understand what you mean by conclusion. If people are dead-set against doing iommufd, then lets abandon the idea and go back to hacking up vfio. > If by conclusion you mean the whole thing to be merged, how can the work be > broken up to pieces if we busy-waiting on the new subsystem? Or maybe you meant > in terms of direction... I think go ahead and build it on top of iommufd, start working out the API details, etc. I think once the direction is concluded the new APIs will go forward. > > While returning the dirty data looks straight forward, it is hard to > > see an obvious path to enabling and controlling the system iommu the > > way vfio is now. > > It seems strange to have a whole UAPI for userspace [*] meant to > return dirty data to userspace, when dirty right now means the whole > pinned page set and so copying the whole guest ... Yes, the whole thing is only partially implemented, and doesn't have any in-kernel user. It is another place holder for an implementation to come someday. > Hence my thinking was that the patches /if small/ would let us see how dirty > tracking might work for iommu kAPI (and iommufd) too. It could be tried, but I think if you go into there you will find it quickly turns quite complicated to address all the edge cases. Eg what do you do if you have a mdev present after you turn on system tracking? What if the mdev is using a PASID? What about hotplug of new VFIO devices? Remember, dirty tracking for vfio is totally useless without also having vfio device migration. Do you already have a migration capable device to use with this? > Would it be better to do more iterative steps (when possible) as opposed to > scratch and rebuild VFIO type1 IOMMU handling? Possibly, but every thing that gets added has to be carried over to the new code too, and energy has to be expended trying to figure out how the half implemented stuff should work while finishing it. At the very least we must decide what to do with device-provided dirty tracking before the VFIO type1 stuff can be altered to use the system IOMMU. This is very much like the migration FSM, the only appeal is the existing qemu implementation of the protocol. Jason