From 551ffb73c7c8dd67681a6ef2cd824936c728e4cb Mon Sep 17 00:00:00 2001 From: Rob Clark Date: Sat, 14 Jun 2025 08:31:23 -0700 Subject: [PATCH] freedreno/cffdec: Skip printing null descriptors If the descriptor is all 0's, it isn't interesting to print. This reduces the verbosity of --bindless arg quite a bit. Signed-off-by: Rob Clark Part-of: --- src/freedreno/decode/cffdec.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/freedreno/decode/cffdec.c b/src/freedreno/decode/cffdec.c index 0dad7412579..712f6c4d36c 100644 --- a/src/freedreno/decode/cffdec.c +++ b/src/freedreno/decode/cffdec.c @@ -1539,18 +1539,21 @@ dump_bindless_descriptors(bool is_compute, int level) if (!contents) continue; + uint32_t empty_contents[16] = {}; + unsigned length = hostlen(ext_src_addr); unsigned desc_count = length / (16 * sizeof(uint32_t)); for (unsigned desc_idx = 0; desc_idx < desc_count; desc_idx++) { - printl(2, "%sUBO[%u]:\n", levels[level + 1], desc_idx); - dump_domain(contents, 2, level + 2, "A6XX_UBO"); + if (memcmp(contents, empty_contents, sizeof(empty_contents))) { + printl(2, "%sUBO[%u]:\n", levels[level + 1], desc_idx); + dump_domain(contents, 2, level + 2, "A6XX_UBO"); - printl(2, "%sSTORAGE/TEXEL/IMAGE[%u]:\n", levels[level + 1], desc_idx); - dump_tex_const(contents, 1, level); - - printl(2, "%sSAMPLER[%u]:\n", levels[level + 1], desc_idx); - dump_tex_samp(contents, STATE_SRC_BINDLESS, 1, level); + printl(2, "%sSTORAGE/TEXEL/IMAGE[%u]:\n", levels[level + 1], desc_idx); + dump_tex_const(contents, 1, level); + printl(2, "%sSAMPLER[%u]:\n", levels[level + 1], desc_idx); + dump_tex_samp(contents, STATE_SRC_BINDLESS, 1, level); + } contents += 16; } }