drm/i915/gem: Pull phys pread/pwrite implementations to the backend
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 5 Nov 2020 15:49:34 +0000 (15:49 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 5 Nov 2020 19:24:19 +0000 (19:24 +0000)
commit852e1b3644817f071427b83859b889c788a0cf69
tree0421ce3f0edb831d73990fdf07327207d0fb2831
parent0049b688459b846f819b6e51c24cd0781fcfde41
drm/i915/gem: Pull phys pread/pwrite implementations to the backend

Move the specialised interactions with the physical GEM object from the
pread/pwrite ioctl handler into the phys backend.

Currently, if one is able to exhaust the entire aperture and then try to
pwrite into an object not backed by struct page, we accidentally invoked
the phys pwrite handler on a non-phys object; calamitous.

Fixes: c6790dc22312 ("drm/i915: Wean off drm_pci_alloc/drm_pci_free")
Testcase: igt/gem_pwrite/exhaustion
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Cc: stable@vger.kernel.org
Link: https://patchwork.freedesktop.org/patch/msgid/20201105154934.16022-2-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/gem/i915_gem_phys.c
drivers/gpu/drm/i915/i915_gem.c