From 6e3b220ad3c6b790ccba63e0ed742f3314c158ef Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Thu, 25 Jul 2019 13:09:51 -0700 Subject: [PATCH] freedreno: Fix data races with allocating/freeing struct ir3. There is a single ir3_compiler in the screen, and each context may be compiling ir3 shaders, which call ir3_create. ralloc doesn't do any locking on its own, so eventually you can end up racing to break ralloc's linked lists. We really don't want struct ir3 to live as long as the compiler (maybe struct ir3_shader's lifetime, if anything), so you'd better be freeing it anyway. Fixes: 8fe20762433d ("freedreno/ir3: convert over to ralloc") Reviewed-by: Rob Clark --- src/freedreno/ir3/ir3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/freedreno/ir3/ir3.c b/src/freedreno/ir3/ir3.c index 97f4ae9..5c049a5 100644 --- a/src/freedreno/ir3/ir3.c +++ b/src/freedreno/ir3/ir3.c @@ -48,7 +48,7 @@ void * ir3_alloc(struct ir3 *shader, int sz) struct ir3 * ir3_create(struct ir3_compiler *compiler, gl_shader_stage type, unsigned nin, unsigned nout) { - struct ir3 *shader = rzalloc(compiler, struct ir3); + struct ir3 *shader = rzalloc(NULL, struct ir3); shader->compiler = compiler; shader->type = type; -- 2.7.4