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

[Marc Dietrich]

Am Dienstag, 24. März 2015, 20:05:46 schrieb 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.

I would like to, but "piglit run quick" stalls/crashes the gpu (rs880) too 
often. Maybe you could tell me some special tests to run instead of all.

Marc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150325/6352dc2e/attachment-0001.sig>