Re: [PATCH] add mes type in mes ring dump.

["StDenis, Tom" <Tom.StDenis@xxxxxxx>]

[AMD Official Use Only - AMD Internal Distribution Only]

Thanks this was pushed out to main just now.

Cheers,
Tom

________________________________________
From: Zhang, Yifan <Yifan1.Zhang@xxxxxxx>
Sent: Wednesday, March 12, 2025 21:40
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: StDenis, Tom; Zhang, Yifan
Subject: [PATCH] add mes type in mes ring dump.

According to MES API spec, type 0 means invald. Pass mes type in ring
dump function to avoid confusion like below:

[0x0@0x00000000 + 0x7200]       [        0x000400e1]    Opcode 0xe [MES_SCH_API_MISC] (64 words, type: 0, hdr: 0x400e1)

Signed-off-by: Yifan Zhang <yifan1.zhang@xxxxxxx>
---
 src/lib/read_mes_stream.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/read_mes_stream.c b/src/lib/read_mes_stream.c
index 6146fd3b..b92ad9fc 100644
--- a/src/lib/read_mes_stream.c
+++ b/src/lib/read_mes_stream.c
@@ -295,7 +295,7 @@ struct umr_mes_stream *umr_mes_decode_stream_opcodes(struct umr_asic *asic, stru
        ui->start_ib(ui, ib_addr, ib_vmid, 0, 0, 0, 0);
        while (stream && opcodes-- && stream->nwords) {
                opcode_name = STR_LOOKUP(mes_v10_opcodes, stream->opcode, "MES_UNK");
-               ui->start_opcode(ui, ib_addr, ib_vmid, 0, stream->opcode, 0, stream->nwords, opcode_name, stream->header, stream->words);
+               ui->start_opcode(ui, ib_addr, ib_vmid, stream->type, stream->opcode, 0, stream->nwords, opcode_name, stream->header, stream->words);

                i = 0;
                ib_addr += 4; // skip over header
--
2.43.0