drm/i915: Move non-phys cursors into the GTT
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 2 Jun 2010 07:41:44 +0000 (08:41 +0100)
committerEric Anholt <eric@anholt.net>
Wed, 2 Jun 2010 21:35:07 +0000 (14:35 -0700)
commit382fe70fddf54114802c935264f1d5baf8d3d174
treef751b1d8019843711358244714e0eff7a3422c4e
parent0d7168bcf45fa5b6307726091ea77fd4ab16d1ab
drm/i915: Move non-phys cursors into the GTT

Cursors need to be in the GTT domain when being accessed by the GPU.
Previously this was a fortuitous byproduct of userspace using pwrite()
to upload the image data into the cursor. The redundant clflush was
removed in commit 9b8c4a and so the image was no longer being flushed
out of the caches into main memory. One could also devise a scenario
where the cursor was rendered by the GPU, prior to being attached as the
cursor, resulting in similar corruption due to the missing MI_FLUSH.

Fixes:

  Bug 28335 - Cursor corruption caused by commit 9b8c4a0b21
  https://bugs.freedesktop.org/show_bug.cgi?id=28335

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Tested-by: Arkadiusz Miƛkiewicz <arekm@maven.pl>
Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/i915/intel_display.c