On Fri, 02 Nov 2012 13:29:20 +0000 Chris Wilson <[email protected]> wrote:
> On Thu, 1 Nov 2012 20:06:03 +0200, [email protected] wrote: > > From: Ville Syrjälä <[email protected]> > > > > As per Chris Wilson's suggestion make > > i915_gem_execbuffer_wait_for_flips() go away. > > > > This was used to stall the GPU ring while there are pending > > page flips involving the relevant BO. Ie. while the BO is still > > being scanned out by the display controller. > > > > The recommended alternative is to use the page flip events to > > wait for the page flips to fully complete before reusing the BO > > of the old front buffer. Or use more buffers. > > > > Signed-off-by: Ville Syrjälä <[email protected]> > Reviewed-by: Chris Wilson <[email protected]> > > Needs an ack from either Jesse or Kristian. Yeah I think this is safe. the kernel doesn't need to protect userspace from doing something silly like rendering on top of the current framebuffer. This patch would also address Eric's complaint about the async patches in that the last buffer will always be available for rendering. Generally this won't be an issue anyway, since subsequent rendering will be behind any flips that will replace the current scanout, so you won't see drawing in progress even in the async flip case. -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
