On 06/13/2011 03:22 PM, Chris Wilson wrote:
On Tue, 14 Jun 2011 00:14:11 -0700, Kenneth Graunke<kenn...@whitecape.org>  
wrote:
On 06/09/2011 12:14 PM, Jesse Barnes wrote:
Found a couple more problems:
    1) MI_DISPLAY_FLIP should take a '1' in the last byte to indicate
       length on IVB
    2) apprently only the BLT ring version of the command actually causes
       interrupts to be generated

With this patch, modetest -v works on my test platform.  Clearly it's in
need of more splitting though.  The case statement should be split into
per-chipset flip command generation.

Signed-off-by: Jesse Barnes<jbar...@virtuousgeek.org>

Reviewed-and-tested-by: Kenneth Graunke<kenn...@whitecape.org>

But it's not the neatest patch in the world ;-)

True :)

Using struct intel_ring_buffer *ring for all generation and switching
between blt/render on demand should be neater. Also we should be able
to do pageflips from either the render or blt ring on gen7, right? That
will be useful to avoid a synchronisation point.
-Chris

I thought so, from reading the docs. But it doesn't seem to work from the render ring...

--Kenneth


_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to