Re: [Mesa-dev] [PATCH 3/3] radeon/llvm: Initialize gallivm targets when initializing the AMDGPU target v2

2015-10-01 Thread Emil Velikov
On 30 September 2015 at 19:59, Tom Stellard wrote: > This fixes a race condition in the glx-multithreaded-shader-compile > test. > > v2: > - Replace gallivm_init_llvm_{begin,end}() with gallivm_init_llvm_targets(). > Thanks for sticking with my suggestion Tom. Fwiw the

Re: [Mesa-dev] [PATCH 3/3] radeon/llvm: Initialize gallivm targets when initializing the AMDGPU target v2

2015-10-01 Thread Tom Stellard
On Thu, Oct 01, 2015 at 03:40:45PM +0100, Emil Velikov wrote: > On 30 September 2015 at 19:59, Tom Stellard wrote: > > This fixes a race condition in the glx-multithreaded-shader-compile > > test. > > > > v2: > > - Replace gallivm_init_llvm_{begin,end}() with > >

Re: [Mesa-dev] [PATCH 3/3] radeon/llvm: Initialize gallivm targets when initializing the AMDGPU target v2

2015-10-01 Thread Mathias Fröhlich
Hi, The series is: Reviewed-by: Mathias Fröhlich Thanks for taking more care of multithread behavior! Mathias On Wednesday, September 30, 2015 18:59:44 Tom Stellard wrote: > This fixes a race condition in the glx-multithreaded-shader-compile > test. > > v2: > -

[Mesa-dev] [PATCH 3/3] radeon/llvm: Initialize gallivm targets when initializing the AMDGPU target v2

2015-09-30 Thread Tom Stellard
This fixes a race condition in the glx-multithreaded-shader-compile test. v2: - Replace gallivm_init_llvm_{begin,end}() with gallivm_init_llvm_targets(). CC: "10.6 11.0" --- src/gallium/drivers/radeon/radeon_llvm_emit.c | 2 ++ 1 file changed, 2

Re: [Mesa-dev] [PATCH 3/3] radeon/llvm: Initialize gallivm targets when initializing the AMDGPU target v2

2015-09-30 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Wed, Sep 30, 2015 at 8:59 PM, Tom Stellard wrote: > This fixes a race condition in the glx-multithreaded-shader-compile > test. > > v2: > - Replace gallivm_init_llvm_{begin,end}() with