On Fri, Jun 17, 2022 at 01:42:42AM -0700, Christoph Hellwig wrote: > On Thu, Jun 16, 2022 at 04:52:09PM -0700, Nicolin Chen wrote: > > + ret = vfio_unpin_pages(&vgpu->vfio_device, gfn << PAGE_SHIFT, npage); > > + drm_WARN_ON(&i915->drm, ret != npage); > > The shifting of gfn seems to happen bother here and in the callers. Sorry. I overlooked this line. I can add another preparatory patches for callers to pass in an IOVA other than "pfn << PAGE_SHIFT", if you think it's necessary: although GVT still does things in PFN, both ap and ccw prepare their PFN lists from IOVAs, which now can be omitted.