[Intel-gfx] [PATCH v2] drm/i915: Make legacy cursor updates more unsynced

Ville Syrjälä ville.syrjala at linux.intel.com
Tue Apr 4 16:55:44 UTC 2017


On Tue, Apr 04, 2017 at 05:50:10PM +0200, Maarten Lankhorst wrote:
> Op 29-03-17 om 16:21 schreef ville.syrjala at linux.intel.com:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > We're clearing the legacy_cursor_update flag before calling
> > drm_atomic_helper_setup_commit() which means the helper will
> > wait for the flip to complete before cleaning up the framebuffers.
> > That's not what we want for the legacy cursor, so let's clear
> > the flag after setting up the commit.
> >
> > Also toss in a FIXME about solving these problems in a nicer
> > way using the fabled vblank workers.
> >
> > v2: Also unsync with legacy page flips
> >
> > Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> > Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> > Cc: Uwe Kleine-König <uwe at kleine-koenig.org>
> > Cc: Rafael Ristovski <rafael.ristovski at gmail.com>
> > Fixes: a5509abda48e ("drm/i915: Fix legacy cursor vs. watermarks for ILK-BDW")
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>

Thanks. Patch pushed to dinq.

-- 
Ville Syrjälä
Intel OTC


More information about the Intel-gfx mailing list