[Mesa-dev] [PATCH 3/6] ac/debug: use an explicit getter for fetching words from the IB
Nicolai Hähnle
nhaehnle at gmail.com
Tue Aug 22 15:45:03 UTC 2017
From: Nicolai Hähnle <nicolai.haehnle at amd.com>
Guard against out-of-bounds accesses, and prepare for upcoming changes.
---
src/amd/common/ac_debug.c | 368 +++++++++++++++++++++++++++-------------------
1 file changed, 215 insertions(+), 153 deletions(-)
diff --git a/src/amd/common/ac_debug.c b/src/amd/common/ac_debug.c
index 42a72c086b1..518893ff481 100644
--- a/src/amd/common/ac_debug.c
+++ b/src/amd/common/ac_debug.c
@@ -43,6 +43,19 @@
#define INDENT_PKT 8
+struct ac_ib_parser {
+ uint32_t *ib;
+ unsigned num_dw;
+ int trace_id;
+ enum chip_class chip_class;
+ ac_debug_addr_callback addr_callback;
+ void *addr_callback_data;
+
+ unsigned cur_dw;
+};
+
+static void ac_do_parse_ib(FILE *f, struct ac_ib_parser *ib);
+
static void print_spaces(FILE *f, unsigned num)
{
fprintf(f, "%*s", num, "");
@@ -129,11 +142,23 @@ void ac_dump_reg(FILE *file, unsigned offset, uint32_t value,
fprintf(file, COLOR_YELLOW "0x%05x" COLOR_RESET " <- 0x%08x\n", offset, value);
}
-static void ac_parse_set_reg_packet(FILE *f, uint32_t *ib, unsigned count,
- unsigned reg_offset)
+static uint32_t ac_ib_get(struct ac_ib_parser *ib)
+{
+ uint32_t v = 0xdeadbeef;
+
+ if (ib->cur_dw < ib->num_dw)
+ v = ib->ib[ib->cur_dw];
+
+ ib->cur_dw++;
+ return v;
+}
+
+static void ac_parse_set_reg_packet(FILE *f, unsigned count, unsigned reg_offset,
+ struct ac_ib_parser *ib)
{
- unsigned reg = ((ib[1] & 0xFFFF) << 2) + reg_offset;
- unsigned index = ib[1] >> 28;
+ unsigned reg_dw = ac_ib_get(ib);
+ unsigned reg = ((reg_dw & 0xFFFF) << 2) + reg_offset;
+ unsigned index = reg_dw >> 28;
int i;
if (index != 0) {
@@ -142,17 +167,15 @@ static void ac_parse_set_reg_packet(FILE *f, uint32_t *ib, unsigned count,
}
for (i = 0; i < count; i++)
- ac_dump_reg(f, reg + i*4, ib[2+i], ~0);
+ ac_dump_reg(f, reg + i*4, ac_ib_get(ib), ~0);
}
-static uint32_t *ac_parse_packet3(FILE *f, uint32_t *ib, int *num_dw,
- int trace_id, enum chip_class chip_class,
- ac_debug_addr_callback addr_callback,
- void *addr_callback_data)
+static void ac_parse_packet3(FILE *f, uint32_t header, struct ac_ib_parser *ib)
{
- unsigned count = PKT_COUNT_G(ib[0]);
- unsigned op = PKT3_IT_OPCODE_G(ib[0]);
- const char *predicate = PKT3_PREDICATE(ib[0]) ? "(predicate)" : "";
+ unsigned first_dw = ib->cur_dw;
+ int count = PKT_COUNT_G(header);
+ unsigned op = PKT3_IT_OPCODE_G(header);
+ const char *predicate = PKT3_PREDICATE(header) ? "(predicate)" : "";
int i;
/* Print the name first. */
@@ -179,180 +202,206 @@ static uint32_t *ac_parse_packet3(FILE *f, uint32_t *ib, int *num_dw,
/* Print the contents. */
switch (op) {
case PKT3_SET_CONTEXT_REG:
- ac_parse_set_reg_packet(f, ib, count, SI_CONTEXT_REG_OFFSET);
+ ac_parse_set_reg_packet(f, count, SI_CONTEXT_REG_OFFSET, ib);
break;
case PKT3_SET_CONFIG_REG:
- ac_parse_set_reg_packet(f, ib, count, SI_CONFIG_REG_OFFSET);
+ ac_parse_set_reg_packet(f, count, SI_CONFIG_REG_OFFSET, ib);
break;
case PKT3_SET_UCONFIG_REG:
- ac_parse_set_reg_packet(f, ib, count, CIK_UCONFIG_REG_OFFSET);
+ ac_parse_set_reg_packet(f, count, CIK_UCONFIG_REG_OFFSET, ib);
break;
case PKT3_SET_SH_REG:
- ac_parse_set_reg_packet(f, ib, count, SI_SH_REG_OFFSET);
+ ac_parse_set_reg_packet(f, count, SI_SH_REG_OFFSET, ib);
break;
case PKT3_ACQUIRE_MEM:
- ac_dump_reg(f, R_0301F0_CP_COHER_CNTL, ib[1], ~0);
- ac_dump_reg(f, R_0301F4_CP_COHER_SIZE, ib[2], ~0);
- ac_dump_reg(f, R_030230_CP_COHER_SIZE_HI, ib[3], ~0);
- ac_dump_reg(f, R_0301F8_CP_COHER_BASE, ib[4], ~0);
- ac_dump_reg(f, R_0301E4_CP_COHER_BASE_HI, ib[5], ~0);
- print_named_value(f, "POLL_INTERVAL", ib[6], 16);
+ ac_dump_reg(f, R_0301F0_CP_COHER_CNTL, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0301F4_CP_COHER_SIZE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_030230_CP_COHER_SIZE_HI, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0301F8_CP_COHER_BASE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0301E4_CP_COHER_BASE_HI, ac_ib_get(ib), ~0);
+ print_named_value(f, "POLL_INTERVAL", ac_ib_get(ib), 16);
break;
case PKT3_SURFACE_SYNC:
- if (chip_class >= CIK) {
- ac_dump_reg(f, R_0301F0_CP_COHER_CNTL, ib[1], ~0);
- ac_dump_reg(f, R_0301F4_CP_COHER_SIZE, ib[2], ~0);
- ac_dump_reg(f, R_0301F8_CP_COHER_BASE, ib[3], ~0);
+ if (ib->chip_class >= CIK) {
+ ac_dump_reg(f, R_0301F0_CP_COHER_CNTL, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0301F4_CP_COHER_SIZE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0301F8_CP_COHER_BASE, ac_ib_get(ib), ~0);
} else {
- ac_dump_reg(f, R_0085F0_CP_COHER_CNTL, ib[1], ~0);
- ac_dump_reg(f, R_0085F4_CP_COHER_SIZE, ib[2], ~0);
- ac_dump_reg(f, R_0085F8_CP_COHER_BASE, ib[3], ~0);
+ ac_dump_reg(f, R_0085F0_CP_COHER_CNTL, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0085F4_CP_COHER_SIZE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0085F8_CP_COHER_BASE, ac_ib_get(ib), ~0);
}
- print_named_value(f, "POLL_INTERVAL", ib[4], 16);
+ print_named_value(f, "POLL_INTERVAL", ac_ib_get(ib), 16);
break;
- case PKT3_EVENT_WRITE:
- ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, ib[1],
+ case PKT3_EVENT_WRITE: {
+ uint32_t event_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, event_dw,
S_028A90_EVENT_TYPE(~0));
- print_named_value(f, "EVENT_INDEX", (ib[1] >> 8) & 0xf, 4);
- print_named_value(f, "INV_L2", (ib[1] >> 20) & 0x1, 1);
+ print_named_value(f, "EVENT_INDEX", (event_dw >> 8) & 0xf, 4);
+ print_named_value(f, "INV_L2", (event_dw >> 20) & 0x1, 1);
if (count > 0) {
- print_named_value(f, "ADDRESS_LO", ib[2], 32);
- print_named_value(f, "ADDRESS_HI", ib[3], 16);
+ print_named_value(f, "ADDRESS_LO", ac_ib_get(ib), 32);
+ print_named_value(f, "ADDRESS_HI", ac_ib_get(ib), 16);
}
break;
- case PKT3_EVENT_WRITE_EOP:
- ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, ib[1],
+ }
+ case PKT3_EVENT_WRITE_EOP: {
+ uint32_t event_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, event_dw,
S_028A90_EVENT_TYPE(~0));
- print_named_value(f, "EVENT_INDEX", (ib[1] >> 8) & 0xf, 4);
- print_named_value(f, "TCL1_VOL_ACTION_ENA", (ib[1] >> 12) & 0x1, 1);
- print_named_value(f, "TC_VOL_ACTION_ENA", (ib[1] >> 13) & 0x1, 1);
- print_named_value(f, "TC_WB_ACTION_ENA", (ib[1] >> 15) & 0x1, 1);
- print_named_value(f, "TCL1_ACTION_ENA", (ib[1] >> 16) & 0x1, 1);
- print_named_value(f, "TC_ACTION_ENA", (ib[1] >> 17) & 0x1, 1);
- print_named_value(f, "ADDRESS_LO", ib[2], 32);
- print_named_value(f, "ADDRESS_HI", ib[3], 16);
- print_named_value(f, "DST_SEL", (ib[3] >> 16) & 0x3, 2);
- print_named_value(f, "INT_SEL", (ib[3] >> 24) & 0x7, 3);
- print_named_value(f, "DATA_SEL", ib[3] >> 29, 3);
- print_named_value(f, "DATA_LO", ib[4], 32);
- print_named_value(f, "DATA_HI", ib[5], 32);
+ print_named_value(f, "EVENT_INDEX", (event_dw >> 8) & 0xf, 4);
+ print_named_value(f, "TCL1_VOL_ACTION_ENA", (event_dw >> 12) & 0x1, 1);
+ print_named_value(f, "TC_VOL_ACTION_ENA", (event_dw >> 13) & 0x1, 1);
+ print_named_value(f, "TC_WB_ACTION_ENA", (event_dw >> 15) & 0x1, 1);
+ print_named_value(f, "TCL1_ACTION_ENA", (event_dw >> 16) & 0x1, 1);
+ print_named_value(f, "TC_ACTION_ENA", (event_dw >> 17) & 0x1, 1);
+ print_named_value(f, "ADDRESS_LO", ac_ib_get(ib), 32);
+ uint32_t addr_hi_dw = ac_ib_get(ib);
+ print_named_value(f, "ADDRESS_HI", addr_hi_dw, 16);
+ print_named_value(f, "DST_SEL", (addr_hi_dw >> 16) & 0x3, 2);
+ print_named_value(f, "INT_SEL", (addr_hi_dw >> 24) & 0x7, 3);
+ print_named_value(f, "DATA_SEL", addr_hi_dw >> 29, 3);
+ print_named_value(f, "DATA_LO", ac_ib_get(ib), 32);
+ print_named_value(f, "DATA_HI", ac_ib_get(ib), 32);
break;
- case PKT3_RELEASE_MEM:
- ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, ib[1],
+ }
+ case PKT3_RELEASE_MEM: {
+ uint32_t event_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_028A90_VGT_EVENT_INITIATOR, event_dw,
S_028A90_EVENT_TYPE(~0));
- print_named_value(f, "EVENT_INDEX", (ib[1] >> 8) & 0xf, 4);
- print_named_value(f, "TCL1_VOL_ACTION_ENA", (ib[1] >> 12) & 0x1, 1);
- print_named_value(f, "TC_VOL_ACTION_ENA", (ib[1] >> 13) & 0x1, 1);
- print_named_value(f, "TC_WB_ACTION_ENA", (ib[1] >> 15) & 0x1, 1);
- print_named_value(f, "TCL1_ACTION_ENA", (ib[1] >> 16) & 0x1, 1);
- print_named_value(f, "TC_ACTION_ENA", (ib[1] >> 17) & 0x1, 1);
- print_named_value(f, "TC_NC_ACTION_ENA", (ib[1] >> 19) & 0x1, 1);
- print_named_value(f, "TC_WC_ACTION_ENA", (ib[1] >> 20) & 0x1, 1);
- print_named_value(f, "TC_MD_ACTION_ENA", (ib[1] >> 21) & 0x1, 1);
- print_named_value(f, "DST_SEL", (ib[2] >> 16) & 0x3, 2);
- print_named_value(f, "INT_SEL", (ib[2] >> 24) & 0x7, 3);
- print_named_value(f, "DATA_SEL", ib[2] >> 29, 3);
- print_named_value(f, "ADDRESS_LO", ib[3], 32);
- print_named_value(f, "ADDRESS_HI", ib[4], 32);
- print_named_value(f, "DATA_LO", ib[5], 32);
- print_named_value(f, "DATA_HI", ib[6], 32);
- print_named_value(f, "CTXID", ib[7], 32);
+ print_named_value(f, "EVENT_INDEX", (event_dw >> 8) & 0xf, 4);
+ print_named_value(f, "TCL1_VOL_ACTION_ENA", (event_dw >> 12) & 0x1, 1);
+ print_named_value(f, "TC_VOL_ACTION_ENA", (event_dw >> 13) & 0x1, 1);
+ print_named_value(f, "TC_WB_ACTION_ENA", (event_dw >> 15) & 0x1, 1);
+ print_named_value(f, "TCL1_ACTION_ENA", (event_dw >> 16) & 0x1, 1);
+ print_named_value(f, "TC_ACTION_ENA", (event_dw >> 17) & 0x1, 1);
+ print_named_value(f, "TC_NC_ACTION_ENA", (event_dw >> 19) & 0x1, 1);
+ print_named_value(f, "TC_WC_ACTION_ENA", (event_dw >> 20) & 0x1, 1);
+ print_named_value(f, "TC_MD_ACTION_ENA", (event_dw >> 21) & 0x1, 1);
+ uint32_t sel_dw = ac_ib_get(ib);
+ print_named_value(f, "DST_SEL", (sel_dw >> 16) & 0x3, 2);
+ print_named_value(f, "INT_SEL", (sel_dw >> 24) & 0x7, 3);
+ print_named_value(f, "DATA_SEL", sel_dw >> 29, 3);
+ print_named_value(f, "ADDRESS_LO", ac_ib_get(ib), 32);
+ print_named_value(f, "ADDRESS_HI", ac_ib_get(ib), 32);
+ print_named_value(f, "DATA_LO", ac_ib_get(ib), 32);
+ print_named_value(f, "DATA_HI", ac_ib_get(ib), 32);
+ print_named_value(f, "CTXID", ac_ib_get(ib), 32);
break;
+ }
case PKT3_WAIT_REG_MEM:
- print_named_value(f, "OP", ib[1], 32);
- print_named_value(f, "ADDRESS_LO", ib[2], 32);
- print_named_value(f, "ADDRESS_HI", ib[3], 32);
- print_named_value(f, "REF", ib[4], 32);
- print_named_value(f, "MASK", ib[5], 32);
- print_named_value(f, "POLL_INTERVAL", ib[6], 16);
+ print_named_value(f, "OP", ac_ib_get(ib), 32);
+ print_named_value(f, "ADDRESS_LO", ac_ib_get(ib), 32);
+ print_named_value(f, "ADDRESS_HI", ac_ib_get(ib), 32);
+ print_named_value(f, "REF", ac_ib_get(ib), 32);
+ print_named_value(f, "MASK", ac_ib_get(ib), 32);
+ print_named_value(f, "POLL_INTERVAL", ac_ib_get(ib), 16);
break;
case PKT3_DRAW_INDEX_AUTO:
- ac_dump_reg(f, R_030930_VGT_NUM_INDICES, ib[1], ~0);
- ac_dump_reg(f, R_0287F0_VGT_DRAW_INITIATOR, ib[2], ~0);
+ ac_dump_reg(f, R_030930_VGT_NUM_INDICES, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0287F0_VGT_DRAW_INITIATOR, ac_ib_get(ib), ~0);
break;
case PKT3_DRAW_INDEX_2:
- ac_dump_reg(f, R_028A78_VGT_DMA_MAX_SIZE, ib[1], ~0);
- ac_dump_reg(f, R_0287E8_VGT_DMA_BASE, ib[2], ~0);
- ac_dump_reg(f, R_0287E4_VGT_DMA_BASE_HI, ib[3], ~0);
- ac_dump_reg(f, R_030930_VGT_NUM_INDICES, ib[4], ~0);
- ac_dump_reg(f, R_0287F0_VGT_DRAW_INITIATOR, ib[5], ~0);
+ ac_dump_reg(f, R_028A78_VGT_DMA_MAX_SIZE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0287E8_VGT_DMA_BASE, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0287E4_VGT_DMA_BASE_HI, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_030930_VGT_NUM_INDICES, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_0287F0_VGT_DRAW_INITIATOR, ac_ib_get(ib), ~0);
break;
case PKT3_INDEX_TYPE:
- ac_dump_reg(f, R_028A7C_VGT_DMA_INDEX_TYPE, ib[1], ~0);
+ ac_dump_reg(f, R_028A7C_VGT_DMA_INDEX_TYPE, ac_ib_get(ib), ~0);
break;
case PKT3_NUM_INSTANCES:
- ac_dump_reg(f, R_030934_VGT_NUM_INSTANCES, ib[1], ~0);
+ ac_dump_reg(f, R_030934_VGT_NUM_INSTANCES, ac_ib_get(ib), ~0);
break;
case PKT3_WRITE_DATA:
- ac_dump_reg(f, R_370_CONTROL, ib[1], ~0);
- ac_dump_reg(f, R_371_DST_ADDR_LO, ib[2], ~0);
- ac_dump_reg(f, R_372_DST_ADDR_HI, ib[3], ~0);
+ ac_dump_reg(f, R_370_CONTROL, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_371_DST_ADDR_LO, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_372_DST_ADDR_HI, ac_ib_get(ib), ~0);
for (i = 2; i < count; i++) {
print_spaces(f, INDENT_PKT);
- fprintf(f, "0x%08x\n", ib[2+i]);
+ fprintf(f, "0x%08x\n", ac_ib_get(ib));
}
break;
case PKT3_CP_DMA:
- ac_dump_reg(f, R_410_CP_DMA_WORD0, ib[1], ~0);
- ac_dump_reg(f, R_411_CP_DMA_WORD1, ib[2], ~0);
- ac_dump_reg(f, R_412_CP_DMA_WORD2, ib[3], ~0);
- ac_dump_reg(f, R_413_CP_DMA_WORD3, ib[4], ~0);
- ac_dump_reg(f, R_414_COMMAND, ib[5], ~0);
+ ac_dump_reg(f, R_410_CP_DMA_WORD0, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_411_CP_DMA_WORD1, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_412_CP_DMA_WORD2, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_413_CP_DMA_WORD3, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_414_COMMAND, ac_ib_get(ib), ~0);
break;
case PKT3_DMA_DATA:
- ac_dump_reg(f, R_500_DMA_DATA_WORD0, ib[1], ~0);
- ac_dump_reg(f, R_501_SRC_ADDR_LO, ib[2], ~0);
- ac_dump_reg(f, R_502_SRC_ADDR_HI, ib[3], ~0);
- ac_dump_reg(f, R_503_DST_ADDR_LO, ib[4], ~0);
- ac_dump_reg(f, R_504_DST_ADDR_HI, ib[5], ~0);
- ac_dump_reg(f, R_414_COMMAND, ib[6], ~0);
+ ac_dump_reg(f, R_500_DMA_DATA_WORD0, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_501_SRC_ADDR_LO, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_502_SRC_ADDR_HI, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_503_DST_ADDR_LO, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_504_DST_ADDR_HI, ac_ib_get(ib), ~0);
+ ac_dump_reg(f, R_414_COMMAND, ac_ib_get(ib), ~0);
break;
case PKT3_INDIRECT_BUFFER_SI:
case PKT3_INDIRECT_BUFFER_CONST:
- case PKT3_INDIRECT_BUFFER_CIK:
- ac_dump_reg(f, R_3F0_IB_BASE_LO, ib[1], ~0);
- ac_dump_reg(f, R_3F1_IB_BASE_HI, ib[2], ~0);
- ac_dump_reg(f, R_3F2_CONTROL, ib[3], ~0);
-
- if (addr_callback) {
- uint64_t addr = ((uint64_t)ib[2] << 32) | ib[1];
- void *data = addr_callback(addr_callback_data, addr);
- const char *name = G_3F2_CHAIN(ib[3]) ? "chained" : "nested";
-
- if (data)
- ac_parse_ib(f, data, G_3F2_IB_SIZE(ib[3]),
- trace_id, name, chip_class,
- addr_callback, addr_callback_data);
+ case PKT3_INDIRECT_BUFFER_CIK: {
+ uint32_t base_lo_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_3F0_IB_BASE_LO, base_lo_dw, ~0);
+ uint32_t base_hi_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_3F1_IB_BASE_HI, base_hi_dw, ~0);
+ uint32_t control_dw = ac_ib_get(ib);
+ ac_dump_reg(f, R_3F2_CONTROL, control_dw, ~0);
+
+ if (!ib->addr_callback)
+ break;
+
+ uint64_t addr = ((uint64_t)base_hi_dw << 32) | base_lo_dw;
+ void *data = ib->addr_callback(ib->addr_callback_data, addr);
+ if (!data)
+ break;
+
+ if (G_3F2_CHAIN(control_dw)) {
+ ib->ib = data;
+ ib->num_dw = G_3F2_IB_SIZE(control_dw);
+ ib->cur_dw = 0;
+ return;
}
+
+ struct ac_ib_parser ib_recurse;
+ memcpy(&ib_recurse, ib, sizeof(ib_recurse));
+ ib_recurse.ib = data;
+ ib_recurse.num_dw = G_3F2_IB_SIZE(control_dw);
+ ib_recurse.cur_dw = 0;
+
+ fprintf(f, "------------------ nested begin ------------------\n");
+ ac_do_parse_ib(f, &ib_recurse);
+ fprintf(f, "------------------- nested end -------------------\n");
break;
+ }
case PKT3_CLEAR_STATE:
case PKT3_INCREMENT_DE_COUNTER:
case PKT3_PFP_SYNC_ME:
break;
case PKT3_NOP:
- if (ib[0] == 0xffff1000) {
+ if (header == 0xffff1000) {
count = -1; /* One dword NOP. */
- break;
- } else if (count == 0 && AC_IS_TRACE_POINT(ib[1])) {
- unsigned packet_id = AC_GET_TRACE_POINT_ID(ib[1]);
+ } else if (count == 0 && ib->cur_dw < ib->num_dw &&
+ AC_IS_TRACE_POINT(ib->ib[ib->cur_dw])) {
+ unsigned packet_id = AC_GET_TRACE_POINT_ID(ib->ib[ib->cur_dw]);
print_spaces(f, INDENT_PKT);
fprintf(f, COLOR_RED "Trace point ID: %u\n", packet_id);
- if (trace_id == -1)
+ if (ib->trace_id == -1)
break; /* tracing was disabled */
print_spaces(f, INDENT_PKT);
- if (packet_id < trace_id)
+ if (packet_id < ib->trace_id)
fprintf(f, COLOR_RED
"This trace point was reached by the CP."
COLOR_RESET "\n");
- else if (packet_id == trace_id)
+ else if (packet_id == ib->trace_id)
fprintf(f, COLOR_RED
"!!!!! This is the last trace point that "
"was reached by the CP !!!!!"
COLOR_RESET "\n");
- else if (packet_id+1 == trace_id)
+ else if (packet_id+1 == ib->trace_id)
fprintf(f, COLOR_RED
"!!!!! This is the first trace point that "
"was NOT been reached by the CP !!!!!"
@@ -364,51 +413,37 @@ static uint32_t *ac_parse_packet3(FILE *f, uint32_t *ib, int *num_dw,
COLOR_RESET "\n");
break;
}
- /* fall through, print all dwords */
- default:
- for (i = 0; i < count+1; i++) {
- print_spaces(f, INDENT_PKT);
- fprintf(f, "0x%08x\n", ib[1+i]);
- }
+ break;
}
- ib += count + 2;
- *num_dw -= count + 2;
- return ib;
+ /* print additional dwords */
+ while (ib->cur_dw <= first_dw + count) {
+ print_spaces(f, INDENT_PKT);
+ fprintf(f, "0x%08x\n", ac_ib_get(ib));
+ }
+
+ if (ib->cur_dw > first_dw + count + 1)
+ fprintf(f, COLOR_RED "!!!!! count in header too low !!!!!"
+ COLOR_RESET "\n");
}
/**
* Parse and print an IB into a file.
- *
- * \param f file
- * \param ib IB
- * \param num_dw size of the IB
- * \param chip_class chip class
- * \param trace_id the last trace ID that is known to have been reached
- * and executed by the CP, typically read from a buffer
- * \param addr_callback Get a mapped pointer of the IB at a given address. Can
- * be NULL.
- * \param addr_callback_data user data for addr_callback
*/
-void ac_parse_ib_chunk(FILE *f, uint32_t *ib, int num_dw, int trace_id,
- enum chip_class chip_class,
- ac_debug_addr_callback addr_callback, void *addr_callback_data)
+static void ac_do_parse_ib(FILE *f, struct ac_ib_parser *ib)
{
- while (num_dw > 0) {
- unsigned type = PKT_TYPE_G(ib[0]);
+ while (ib->cur_dw < ib->num_dw) {
+ uint32_t header = ac_ib_get(ib);
+ unsigned type = PKT_TYPE_G(header);
switch (type) {
case 3:
- ib = ac_parse_packet3(f, ib, &num_dw, trace_id,
- chip_class, addr_callback,
- addr_callback_data);
+ ac_parse_packet3(f, header, ib);
break;
case 2:
/* type-2 nop */
- if (ib[0] == 0x80000000) {
+ if (header == 0x80000000) {
fprintf(f, COLOR_GREEN "NOP (type 2)" COLOR_RESET "\n");
- ib++;
- num_dw--;
break;
}
/* fall through */
@@ -418,7 +453,7 @@ void ac_parse_ib_chunk(FILE *f, uint32_t *ib, int num_dw, int trace_id,
}
}
- if (num_dw < 0) {
+ if (ib->cur_dw > ib->num_dw) {
printf("\nPacket ends after the end of IB.\n");
exit(0);
}
@@ -427,6 +462,33 @@ void ac_parse_ib_chunk(FILE *f, uint32_t *ib, int num_dw, int trace_id,
/**
* Parse and print an IB into a file.
*
+ * \param f file
+ * \param ib_ptr IB
+ * \param num_dw size of the IB
+ * \param chip_class chip class
+ * \param trace_id the last trace ID that is known to have been reached
+ * and executed by the CP, typically read from a buffer
+ * \param addr_callback Get a mapped pointer of the IB at a given address. Can
+ * be NULL.
+ * \param addr_callback_data user data for addr_callback
+ */
+void ac_parse_ib_chunk(FILE *f, uint32_t *ib_ptr, int num_dw, int trace_id,
+ enum chip_class chip_class,
+ ac_debug_addr_callback addr_callback, void *addr_callback_data)
+{
+ struct ac_ib_parser ib = {};
+ ib.ib = ib_ptr;
+ ib.num_dw = num_dw;
+ ib.trace_id = trace_id;
+ ib.chip_class = chip_class;
+ ib.addr_callback = addr_callback;
+ ib.addr_callback_data = addr_callback_data;
+ ac_do_parse_ib(f, &ib);
+}
+
+/**
+ * Parse and print an IB into a file.
+ *
* \param f file
* \param ib IB
* \param num_dw size of the IB
--
2.11.0
More information about the mesa-dev
mailing list