From: Matt Coster <matt.coster@imgtec.com>
Date: Fri, 28 Oct 2022 16:09:56 +0000 (+0100)
Subject: pvr: debug: Print hexdump at the end of all sub buffers
X-Git-Tag: upstream/23.3.3~16120
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5740d85c4b8c33f05c8326e680c8e997ab07aa4;p=platform%2Fupstream%2Fmesa.git

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

Signed-off-by: Matt Coster <matt.coster@imgtec.com>
Reviewed-by: Karmjit Mahil <Karmjit.Mahil@imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20040>
---

diff --git a/src/imagination/vulkan/pvr_dump_csb.c b/src/imagination/vulkan/pvr_dump_csb.c
index 895bd1e..b723fa9 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);