On Wed, Apr 13, 2022 at 07:55:24AM +0200, Christoph Hellwig wrote: > On Tue, Apr 12, 2022 at 12:53:28PM -0300, Jason Gunthorpe wrote: > > All callers have a struct vfio_device trivially available, pass it in > > directly and avoid calling the expensive vfio_group_get_from_dev(). > > Instead of bothering the drivers with the notifiers at all, the two > notifier_blocks should move into struct vfio_device, and the > vfio_ops should just grow two new dma_unmap and set_kvm methods. > > This will isolate the drives from the whole notifiers mess and it's > boilerplate code. I already looked into that for a while, it is a real mess too because of how the notifiers are used by the current drivers, eg gvt assumes the notifier is called during its open_device callback to setup its kvm. The unmap is less convoluted and I might still try to do that.. For this series I prefer to leave it alone Jason
