Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-24 Thread Iago Toral
On Mon, 2015-09-21 at 09:21 -0700, Matt Turner wrote: > On Mon, Sep 21, 2015 at 8:00 AM, Iago Toral wrote: > > On Mon, 2015-09-21 at 07:49 -0700, Kenneth Graunke wrote: > >> On Monday, September 21, 2015 09:46:24 AM Mark Janes wrote: > >> > This series hits an assertion on ILK and G45: > >> > > >>

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-22 Thread Iago Toral
On Tue, 2015-09-22 at 08:10 +0200, Iago Toral wrote: > Hi Mark, > > On Mon, 2015-09-21 at 17:45 -0700, Mark Janes wrote: > > Hi Iago, > > > > According to my tests, this patch series fixes the gles2/gles3 > > "functional.uniform_api.random.23" tests in dEQP, on sandybridge. > > > > Do you see th

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Iago Toral
Hi Mark, On Mon, 2015-09-21 at 17:45 -0700, Mark Janes wrote: > Hi Iago, > > According to my tests, this patch series fixes the gles2/gles3 > "functional.uniform_api.random.23" tests in dEQP, on sandybridge. > > Do you see the same results? Should this patch series be applied to the > stable br

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Mark Janes
Hi Iago, According to my tests, this patch series fixes the gles2/gles3 "functional.uniform_api.random.23" tests in dEQP, on sandybridge. Do you see the same results? Should this patch series be applied to the stable branch? thanks, Mark Iago Toral Quiroga writes: > There are some bug repor

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Matt Turner
On Mon, Sep 21, 2015 at 8:00 AM, Iago Toral wrote: > On Mon, 2015-09-21 at 07:49 -0700, Kenneth Graunke wrote: >> On Monday, September 21, 2015 09:46:24 AM Mark Janes wrote: >> > This series hits an assertion on ILK and G45: >> > >> > src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Asse

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Iago Toral
On Mon, 2015-09-21 at 07:49 -0700, Kenneth Graunke wrote: > On Monday, September 21, 2015 09:46:24 AM Mark Janes wrote: > > This series hits an assertion on ILK and G45: > > > > src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Assertion > > `dest.nr < (devinfo->gen == 6 ? 24 : 16)' faile

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Kenneth Graunke
On Monday, September 21, 2015 09:46:24 AM Mark Janes wrote: > This series hits an assertion on ILK and G45: > > src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Assertion > `dest.nr < (devinfo->gen == 6 ? 24 : 16)' failed. > > It triggers about 8k piglit assertions on those platforms.

Re: [Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-21 Thread Mark Janes
This series hits an assertion on ILK and G45: src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Assertion `dest.nr < (devinfo->gen == 6 ? 24 : 16)' failed. It triggers about 8k piglit assertions on those platforms. I'm turning off testing for G45 and ILK until it is resolved. https://b

[Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

2015-09-18 Thread Iago Toral Quiroga
There are some bug reports about shaders failing to compile in gen6 because MRF 14 is used when we need to spill. For example: https://bugs.freedesktop.org/show_bug.cgi?id=86469 https://bugs.freedesktop.org/show_bug.cgi?id=90631 Discussion in bugzilla pointed to the fact that gen6 might actually h