On Mon, 5 Nov 2012 15:20:53 +0200, ville.syrjala at linux.intel.com wrote: > From: Ville Syrj??l?? <ville.syrjala at linux.intel.com> > > intel_pipe_set_base() never actually waited for any pending page flips > on the CRTC. It looks like it tried to, by calling intel_finish_fb() on > the current front buffer. But the pending flips were actually tracked > in the BO of the previous front buffer, so the call to intel_finish_fb() > never did anything useful. > > intel_crtc_wait_for_pending_flips() is the current _working_ way to wait > for pending page flips. So use it in intel_pipe_set_base() too. Some > refactoring was necessary to avoid locking struct_mutex twice. > > Signed-off-by: Ville Syrj??l?? <ville.syrjala at linux.intel.com> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> -Chris -- Chris Wilson, Intel Open Source Technology Centre
