[Mesa-dev] [PATCH] r600g/sb: Enable SB for geometry shaders

[Glenn Kennard]

On Tue, 24 Mar 2015 17:21:35 +0100, Dieter Nützel <Dieter at nuetzel-hh.de>  
wrote:

> Am 20.03.2015 14:13, schrieb Glenn Kennard:
>> Add SV_GEOMETRY_EMIT special variable type to track the
>> implicit dependencies between CUT/EMIT_VERTEX/MEM_RING
>> instructions so GCM/scheduler doesn't reorder them.
>>  Mark emit instructions as unkillable so DCE doesn't eat them.
>>  Signed-off-by: Glenn Kennard <glenn.kennard at gmail.com>
>> ---
>> The hangs with SB on geometry shaders were all due to the CUT/EMIT
>> instructions either being DCE:d or emitted out of order from the
>> memory ring writes, so the hardware stalled forever waiting for
>> completed primitives.
>>  Tested only on a Turks so far, but should behave the same across
>> all R600 generations.
>
> Hello Glenn,
>
> what tests are preferred?
> Starting with a Turks XT here, too and could do some tests on RV730  
> (AGP) then.
>
> -Dieter

Just the usual piglit regression testing, at this point it's been tested  
on a Turks XT, and a RV770. A R6xx card and some VLIW4 gpu would complete  
the coverage needed.


/Glenn