From d7aa51a50c65d06108a7979265226d1746d89da1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cl=C3=A9ment=20Foucault?= Date: Tue, 10 Apr 2018 19:30:13 +0200 Subject: DRW: Deferred Shader Compilation: Don't recreate ogl context. Only recreate ogl context if we cannot reuse the one of the previous thread. Adding lots of shaders were recreating as many ogl context which was very slow. --- source/blender/draw/intern/draw_manager_shader.c | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) (limited to 'source') diff --git a/source/blender/draw/intern/draw_manager_shader.c b/source/blender/draw/intern/draw_manager_shader.c index 760d3a359fe..c7fbe1dd42b 100644 --- a/source/blender/draw/intern/draw_manager_shader.c +++ b/source/blender/draw/intern/draw_manager_shader.c @@ -98,8 +98,9 @@ static void drw_deferred_shader_queue_free(ListBase *queue) static void drw_deferred_shader_compilation_exec(void *custom_data, short *stop, short *do_update, float *progress) { DRWShaderCompiler *comp = (DRWShaderCompiler *)custom_data; + void *ogl_context = comp->ogl_context; - WM_opengl_context_activate(comp->ogl_context); + WM_opengl_context_activate(ogl_context); while (true) { BLI_spin_lock(&comp->list_lock); @@ -143,7 +144,7 @@ static void drw_deferred_shader_compilation_exec(void *custom_data, short *stop, drw_deferred_shader_free(comp->mat_compiling); } - WM_opengl_context_release(comp->ogl_context); + WM_opengl_context_release(ogl_context); } static void drw_deferred_shader_compilation_free(void *custom_data) @@ -155,7 +156,10 @@ static void drw_deferred_shader_compilation_free(void *custom_data) BLI_spin_end(&comp->list_lock); BLI_mutex_end(&comp->compilation_lock); - WM_opengl_context_dispose(comp->ogl_context); + if (comp->ogl_context) { + /* Only destroy if the job owns the context. */ + WM_opengl_context_dispose(comp->ogl_context); + } MEM_freeN(comp); } @@ -199,13 +203,18 @@ static void drw_deferred_shader_add( BLI_spin_lock(&old_comp->list_lock); BLI_movelisttolist(&comp->queue, &old_comp->queue); BLI_spin_unlock(&old_comp->list_lock); + /* Do not recreate context, just pass ownership. */ + comp->ogl_context = old_comp->ogl_context; + old_comp->ogl_context = NULL; } BLI_addtail(&comp->queue, dsh); - /* Create one context per task. */ - comp->ogl_context = WM_opengl_context_create(); - WM_opengl_context_activate(DST.ogl_context); + /* Create only one context. */ + if (comp->ogl_context == NULL) { + comp->ogl_context = WM_opengl_context_create(); + WM_opengl_context_activate(DST.ogl_context); + } WM_jobs_customdata_set(wm_job, comp, drw_deferred_shader_compilation_free); WM_jobs_timer(wm_job, 0.1, NC_MATERIAL | ND_SHADING_DRAW, 0); -- cgit v1.2.3