On Tue, Apr 04, 2017 at 05:50:10PM +0200, Maarten Lankhorst wrote:
> Op 29-03-17 om 16:21 schreef ville.syrjala@xxxxxxxxxxxxxxx:
> > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxxxxx>
> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
> > Cc: Uwe Kleine-König <uwe@xxxxxxxxxxxxxxxxx>
> > Cc: Rafael Ristovski <rafael.ristovski@xxxxxxxxx>
> > Fixes: a5509abda48e ("drm/i915: Fix legacy cursor vs. watermarks for ILK-BDW")
> > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
Thanks. Patch pushed to dinq.
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
