On Thursday 19 February 2009 11:37:01 Chris Wilson wrote: > With a few additional suggestions by Jesse, I've managed to get > tear-free compositing working on i915. Here's the diff on top of the > original patch (though obviously this is just a suggestion, still need > to prevent multiple pending flips to the same plane and ensure that the > old buffer is eventually unpinned, and you might choose to drop the > mutex around the wait_for_vblank ;-):
Yeah, looks pretty reasonable. Here's what I've been working with. It adds a couple of more changes (and slightly different cleanups) over your version: - added a seqno to wait for - wait for flip before queuing another but it's missing thew 915 changes you added (btw did I get the pitch wrong? or are you submitting a different value for your objects?). I added a new seqno since it's possible (even likely) that the next vblank won't actually reflect the flip, if the GPU is busy processing a large batchbuffer when the ring command goes in for example. Also it might be a good idea to wait for any previous flip before queuing a new one. Anyway still tracking down some issues with the X side, but it seems like it's approaching readiness. Thanks, -- Jesse Barnes, Intel Open Source Technology Center ------------------------------------------------------------------------------ Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
