diff options
author | Kenneth Graunke <kenneth@whitecape.org> | 2019-04-15 21:59:50 -0700 |
---|---|---|
committer | Kenneth Graunke <kenneth@whitecape.org> | 2019-04-16 09:01:15 -0700 |
commit | fad7801afd194c504bd945c497741ecaf04e2cdd (patch) | |
tree | 552a1266c996048a4f900f333a54fb0b8bffb473 /src/mesa/drivers/dri/i965/brw_gs.c | |
parent | 4f715868a94b2c43656b3574b876bd254757521b (diff) |
i965: Move program key debugging to the compiler.
The i965 driver has a bunch of code to compare two sets of program keys
and print out the differences. This can be useful for debugging why a
shader needed to be recompiled on the fly due to non-orthogonal state
dependencies. anv doesn't do recompiles, so we didn't need to share
this in the past - but I'd like to use it in iris.
This moves the bulk of the code to the compiler where it can be reused.
To make that possible, we need to decouple it from i965 - we can't get
at the brw program cache directly, nor use brw_context to print things.
Instead, we use compiler->shader_perf_log(), and simply pass in keys.
We put all of this debugging code in brw_debug_recompile.c, and only
export a single function, for simplicity. I also tidied the code a
bit while moving it, now that it all lives in one file.
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_gs.c')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_gs.c | 27 |
1 files changed, 2 insertions, 25 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_gs.c b/src/mesa/drivers/dri/i965/brw_gs.c index a35ce97db3e..eeff4784e72 100644 --- a/src/mesa/drivers/dri/i965/brw_gs.c +++ b/src/mesa/drivers/dri/i965/brw_gs.c @@ -36,30 +36,6 @@ #include "compiler/glsl/ir_uniform.h" static void -brw_gs_debug_recompile(struct brw_context *brw, struct gl_program *prog, - const struct brw_gs_prog_key *key) -{ - perf_debug("Recompiling geometry shader for program %d\n", prog->Id); - - bool found = false; - const struct brw_gs_prog_key *old_key = - brw_find_previous_compile(&brw->cache, BRW_CACHE_GS_PROG, - key->program_string_id); - - if (!old_key) { - perf_debug(" Didn't find previous compile in the shader cache for " - "debug\n"); - return; - } - - found |= brw_debug_recompile_sampler_key(brw, &old_key->tex, &key->tex); - - if (!found) { - perf_debug(" Something else\n"); - } -} - -static void assign_gs_binding_table_offsets(const struct gen_device_info *devinfo, const struct gl_program *prog, struct brw_gs_prog_data *prog_data) @@ -128,7 +104,8 @@ brw_codegen_gs_prog(struct brw_context *brw, if (unlikely(brw->perf_debug)) { if (gp->compiled_once) { - brw_gs_debug_recompile(brw, &gp->program, key); + brw_debug_recompile(brw, MESA_SHADER_GEOMETRY, gp->program.Id, + key->program_string_id, key); } if (start_busy && !brw_bo_busy(brw->batch.last_bo)) { perf_debug("GS compile took %.03f ms and stalled the GPU\n", |