Re: [Mesa-dev] [PATCH] r600: shader CF_OP_VTX also doesn't use the EOP bit.

2017-11-16 Thread Dave Airlie
On 17 November 2017 at 08:00, Gert Wollny wrote: > Am Mittwoch, den 15.11.2017, 11:28 +1000 schrieb Dave Airlie: >> On 15 November 2017 at 04:50, Gert Wollny >> wrote: >> > Although the EOP bit is documented for the vertex fetch clause, it >> > is not >> > properly interpreted. As a result the pi

Re: [Mesa-dev] [PATCH] r600: shader CF_OP_VTX also doesn't use the EOP bit.

2017-11-16 Thread Gert Wollny
Am Mittwoch, den 15.11.2017, 11:28 +1000 schrieb Dave Airlie: > On 15 November 2017 at 04:50, Gert Wollny > wrote: > > Although the EOP bit is documented for the vertex fetch clause, it > > is not > > properly interpreted. As a result the piglit > >   spec/arb_tessellation_shader/execution/trivial

Re: [Mesa-dev] [PATCH] r600: shader CF_OP_VTX also doesn't use the EOP bit.

2017-11-15 Thread Gert Wollny
Am Mittwoch, den 15.11.2017, 11:28 +1000 schrieb Dave Airlie: > > I'll likely apply this as well at some point, that test however hangs > my cayman with or without my patch (and your patch doesn't do > anything on cayman). BTW: valgrind showed that there was some problem in sb in the parser (bran

Re: [Mesa-dev] [PATCH] r600: shader CF_OP_VTX also doesn't use the EOP bit.

2017-11-14 Thread Dave Airlie
On 15 November 2017 at 04:50, Gert Wollny wrote: > Although the EOP bit is documented for the vertex fetch clause, it is not > properly interpreted. As a result the piglit > spec/arb_tessellation_shader/execution/trivial-tess-gs_no-gs-inputs > creates a TESS_EVAL shader that does not have an EOP

[Mesa-dev] [PATCH] r600: shader CF_OP_VTX also doesn't use the EOP bit.

2017-11-14 Thread Gert Wollny
Although the EOP bit is documented for the vertex fetch clause, it is not properly interpreted. As a result the piglit spec/arb_tessellation_shader/execution/trivial-tess-gs_no-gs-inputs creates a TESS_EVAL shader that does not have an EOP clause, which might result in a GPU lockup. This patch f