Re: [Mesa-dev] [PATCH 2/3] mesa: implement ARB_parallel_shader_compile

2017-04-18 Thread Edward O'Callaghan
On 04/18/2017 03:39 PM, Jason Ekstrand wrote: > On Mon, Apr 17, 2017 at 4:49 PM, Edward O'Callaghan > > wrote: > > > > On 04/18/2017 01:13 AM, Jason Ekstrand wrote: > > I don't see anything in here that actually

Re: [Mesa-dev] [PATCH 2/3] mesa: implement ARB_parallel_shader_compile

2017-04-17 Thread Jason Ekstrand
On Mon, Apr 17, 2017 at 4:49 PM, Edward O'Callaghan < funfunc...@folklore1984.net> wrote: > > > On 04/18/2017 01:13 AM, Jason Ekstrand wrote: > > I don't see anything in here that actually implements parallel shader > > compilation. Does radeon already spawn back-end threads to compile > As far

Re: [Mesa-dev] [PATCH 2/3] mesa: implement ARB_parallel_shader_compile

2017-04-17 Thread Edward O'Callaghan
On 04/18/2017 01:13 AM, Jason Ekstrand wrote: > I don't see anything in here that actually implements parallel shader > compilation. Does radeon already spawn back-end threads to compile As far as I am aware that is exactly right. > shaders or are we just lying about it? Even if it does, the

Re: [Mesa-dev] [PATCH 2/3] mesa: implement ARB_parallel_shader_compile

2017-04-17 Thread Jason Ekstrand
I don't see anything in here that actually implements parallel shader compilation. Does radeon already spawn back-end threads to compile shaders or are we just lying about it? Even if it does, the front-end tends to take significant quantities of time so it'd be nice to parallelize that. On

[Mesa-dev] [PATCH 2/3] mesa: implement ARB_parallel_shader_compile

2017-04-17 Thread Edward O'Callaghan
Signed-off-by: Edward O'Callaghan --- src/mesa/main/get.c| 4 src/mesa/main/mtypes.h | 6 ++ src/mesa/main/shaderapi.c | 20 src/mesa/main/shaderapi.h | 3 +++