Mesa (master): intel/batch_decoder: Fix dynamic state printing

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Aug 25 12:51:06 UTC 2018


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

Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Fri Aug 24 16:04:03 2018 -0500

intel/batch_decoder: Fix dynamic state printing

Instead of printing addresses like everyone else, we were accidentally
printing the offset from state base address.  Also, state_map is a void
pointer so we were incrementing in bytes instead of dwords and every
state other than the first was wrong.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

---

 src/intel/common/gen_batch_decoder.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/intel/common/gen_batch_decoder.c b/src/intel/common/gen_batch_decoder.c
index f93f4df006..a57bd93e0f 100644
--- a/src/intel/common/gen_batch_decoder.c
+++ b/src/intel/common/gen_batch_decoder.c
@@ -665,10 +665,10 @@ decode_dynamic_state_pointers(struct gen_batch_decode_ctx *ctx,
 
    for (int i = 0; i < count; i++) {
       fprintf(ctx->fp, "%s %d\n", struct_type, i);
-      ctx_print_group(ctx, state, state_offset, state_map);
+      ctx_print_group(ctx, state, state_addr, state_map);
 
       state_addr += state->dw_length * 4;
-      state_map += state->dw_length;
+      state_map += state->dw_length * 4;
    }
 }
 




More information about the mesa-commit mailing list