Module: Mesa
Branch: main
Commit: d5740d85c4b8c33f05c8326e680c8e997ab07aa4
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d5740d85c4b8c33f05c8326e680c8e997ab07aa4

Author: Matt Coster <[email protected]>
Date:   Fri Oct 28 17:09:56 2022 +0100

pvr: debug: Print hexdump at the end of all sub buffers

Signed-off-by: Matt Coster <[email protected]>
Reviewed-by: Karmjit Mahil <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20040>

---

 src/imagination/vulkan/pvr_dump_csb.c | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/src/imagination/vulkan/pvr_dump_csb.c 
b/src/imagination/vulkan/pvr_dump_csb.c
index 895bd1e50c8..b723fa914d0 100644
--- a/src/imagination/vulkan/pvr_dump_csb.c
+++ b/src/imagination/vulkan/pvr_dump_csb.c
@@ -2306,8 +2306,7 @@ static bool print_sub_buffer(struct pvr_dump_ctx *const 
ctx,
    switch (type) {
    case BUFFER_TYPE_NONE:
       pvr_dump_field(base_ctx, "<content>", "<not decoded>");
-      pvr_dump_indent(&sub_ctx.base.base);
-      ret = pvr_dump_buffer_hex(&sub_ctx.base, 0);
+      ret = true;
       break;
 
    case BUFFER_TYPE_PPP:
@@ -2317,9 +2316,19 @@ static bool print_sub_buffer(struct pvr_dump_ctx *const 
ctx,
 
    default:
       pvr_dump_field(base_ctx, "<content>", "<unsupported format>");
-      goto end_pop_ctx;
+      ret = false;
    }
 
+   pvr_dump_field_u32_units(&sub_ctx.base.base,
+                            "<raw>",
+                            sub_ctx.base.capacity,
+                            "bytes");
+
+   pvr_dump_indent(&sub_ctx.base.base);
+   pvr_dump_buffer_restart(&sub_ctx.base);
+   pvr_dump_buffer_hex(&sub_ctx.base, 0);
+   pvr_dump_dedent(&sub_ctx.base.base);
+
 end_pop_ctx:
    pvr_dump_bo_ctx_pop(&sub_ctx);
 

Reply via email to