Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Martin Peres
On 14/06/16 18:47, Martin Peres wrote: On 14/06/16 17:58, Juha-Pekka Heikkila wrote: Here is fixed version of this ralloc set. Now I got to run this on many different machines thanks to Mark Janes. There didn't show up any regressions on different gen hw. On my IVB I've been running also many di

Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Eric Anholt
Rob Clark writes: > I (and I expect Eric too) would appreciate it if you went ahead and > replaced the current use of non-"z" versions in code that you can't > test w/ the "z" versions. That way we can switch over to non-zero'ing > on our own time, rather than getting a surprise next time we > p

Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Rob Clark
I (and I expect Eric too) would appreciate it if you went ahead and replaced the current use of non-"z" versions in code that you can't test w/ the "z" versions. That way we can switch over to non-zero'ing on our own time, rather than getting a surprise next time we pull/rebase I think it's only

Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Nicolai Hähnle
On 14.06.2016 17:07, Ilia Mirkin wrote: I assume you've only tested this with i965? ralloc is also used by st/mesa, freedreno, and vc4. Should probably try to coordinate with the responsible developers before making the big switch. In st_glsl_to_tgsi.c, there is one ralloc(mem_ctx, function_ent

Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Martin Peres
On 14/06/16 17:58, Juha-Pekka Heikkila wrote: Here is fixed version of this ralloc set. Now I got to run this on many different machines thanks to Mark Janes. There didn't show up any regressions on different gen hw. On my IVB I've been running also many different traces with Apitrace while havin

Re: [Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Ilia Mirkin
I assume you've only tested this with i965? ralloc is also used by st/mesa, freedreno, and vc4. Should probably try to coordinate with the responsible developers before making the big switch. -ilia On Tue, Jun 14, 2016 at 10:58 AM, Juha-Pekka Heikkila wrote: > Here is fixed version of this ral

[Mesa-dev] [PATCH 0/7] Fix ralloc/rzalloc usage v2

2016-06-14 Thread Juha-Pekka Heikkila
Here is fixed version of this ralloc set. Now I got to run this on many different machines thanks to Mark Janes. There didn't show up any regressions on different gen hw. On my IVB I've been running also many different traces with Apitrace while having Valgrind running on background but Valgrind di