drm/vc4: Add a proper short-circut path for legacy cursor updates.
authorEric Anholt <eric@anholt.net>
Wed, 30 Dec 2015 19:50:22 +0000 (11:50 -0800)
committerEric Anholt <eric@anholt.net>
Tue, 16 Feb 2016 19:24:08 +0000 (11:24 -0800)
commit6674a904d68041d982ffb284d2827410765a097a
treeb3abe75650091eb7bbdd2dd98d5a3e0fb1bde246
parent5c6799942003df91801b1d2277bba34d71f99603
drm/vc4: Add a proper short-circut path for legacy cursor updates.

Previously, on every modeset we would allocate new display list
memory, recompute changed planes, write all of them to the new memory,
and pointed scanout at the new list (which will latch approximately at
the next line of scanout).  We let
drm_atomic_helper_wait_for_vblanks() decide whether we needed to wait
for a vblank after a modeset before cleaning up the old state and
letting the next modeset proceed, and on legacy cursor updates we
wouldn't wait.  If you moved the cursor fast enough, we could
potentially wrap around the display list memory area and overwrite the
existing display list while it was still being scanned out, resulting
in the HVS scanning out garbage or just halting.

Instead of making cursor updates wait for scanout to move to the new
display list area (which introduces significant cursor lag in X), we
just rewrite our current display list.

Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/vc4/vc4_kms.c
drivers/gpu/drm/vc4/vc4_plane.c