From e26dd92957df80fc31627eff5fc9c9453cf3021d Mon Sep 17 00:00:00 2001 From: Pierre-Eric Pelloux-Prayer Date: Mon, 8 Nov 2021 10:51:15 +0100 Subject: [PATCH] radeonsi/sqtt: fix FINISH_DONE / BUSY usage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit They're using more than a single bit so use the proper mask. Based on https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13694 Reviewed-by: Marek Olšák Part-of: --- src/gallium/drivers/radeonsi/si_sqtt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_sqtt.c b/src/gallium/drivers/radeonsi/si_sqtt.c index 44a77dc3a40..a5eb4f5851c 100644 --- a/src/gallium/drivers/radeonsi/si_sqtt.c +++ b/src/gallium/drivers/radeonsi/si_sqtt.c @@ -321,7 +321,7 @@ si_emit_thread_trace_stop(struct si_context *sctx, radeon_emit(R_008D20_SQ_THREAD_TRACE_STATUS >> 2); /* register */ radeon_emit(0); radeon_emit(0); /* reference value */ - radeon_emit(S_008D20_FINISH_DONE(1)); /* mask */ + radeon_emit(~C_008D20_FINISH_DONE); /* mask */ radeon_emit(4); /* poll interval */ /* Disable the thread trace mode. */ @@ -334,7 +334,7 @@ si_emit_thread_trace_stop(struct si_context *sctx, radeon_emit(R_008D20_SQ_THREAD_TRACE_STATUS >> 2); /* register */ radeon_emit(0); radeon_emit(0); /* reference value */ - radeon_emit(S_008D20_BUSY(1)); /* mask */ + radeon_emit(~C_008D20_BUSY); /* mask */ radeon_emit(4); /* poll interval */ } else { /* Disable the thread trace mode. */ @@ -347,7 +347,7 @@ si_emit_thread_trace_stop(struct si_context *sctx, radeon_emit(R_030CE8_SQ_THREAD_TRACE_STATUS >> 2); /* register */ radeon_emit(0); radeon_emit(0); /* reference value */ - radeon_emit(S_030CE8_BUSY(1)); /* mask */ + radeon_emit(~C_030CE8_BUSY); /* mask */ radeon_emit(4); /* poll interval */ } radeon_end();