Re: [Mesa-dev] [PATCHv2 01/13] mesa: protect the debug state with a mutex

2014-08-19 Thread Chia-I Wu
Hi Ian, Thanks for the reviews. On Thu, Aug 14, 2014 at 8:45 AM, Ian Romanick i...@freedesktop.org wrote: Patches 7, 8, and 9 are Reviewed-by: Ian Romanick ian.d.roman...@intel.com I made a few minor comments on 7, but they should be trivial to resolve. Patches 10 through 13 are

Re: [Mesa-dev] [PATCHv2 01/13] mesa: protect the debug state with a mutex

2014-08-13 Thread Ian Romanick
This series fell off my radar. Sorry about the delays. :( Patches 1 through 5 are Reviewed-by: Ian Romanick ian.d.roman...@intel.com I'm still working my way through the rest. I'd like to see these land as soon as reasonably possible. On 07/09/2014 12:47 AM, Chia-I Wu wrote: We are about to

Re: [Mesa-dev] [PATCHv2 01/13] mesa: protect the debug state with a mutex

2014-08-13 Thread Ian Romanick
Patches 7, 8, and 9 are Reviewed-by: Ian Romanick ian.d.roman...@intel.com I made a few minor comments on 7, but they should be trivial to resolve. Patches 10 through 13 are Acked-by: Ian Romanick ian.d.roman...@intel.com I'd like to see some feed back on those last four from Ken and / or

[Mesa-dev] [PATCHv2 01/13] mesa: protect the debug state with a mutex

2014-07-09 Thread Chia-I Wu
We are about to change mesa to spawn threads for deferred glCompileShader and glLinkProgram, and we need to make sure those threads can send compiler warnings/errors to the debug output safely. Signed-off-by: Chia-I Wu o...@lunarg.com --- src/mesa/main/errors.c | 172