From fc02ce57131c3edd462d093e7e4f842bf93315db Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Mon, 18 Jul 2022 16:48:39 -0700 Subject: [PATCH] intel/eu: Mark header present in URB memory fences on XeHP Fixes the following EU validation error: ERROR: Header must be present for all URB messages. The message header is ignored for URB fence messages, so I doubt that this actually matters in practice. But we should probably mark it as present, because you have to send something, and according to the documentation, there is a message header, it's just ignored. Fixes: e6a9501aa27 ("intel/fs: Add the URB fence message") Reviewed-by: Francisco Jerez Part-of: --- src/intel/compiler/brw_eu_emit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/compiler/brw_eu_emit.c b/src/intel/compiler/brw_eu_emit.c index 28360abed70..4507ade3a1a 100644 --- a/src/intel/compiler/brw_eu_emit.c +++ b/src/intel/compiler/brw_eu_emit.c @@ -3268,7 +3268,7 @@ gfx12_set_memory_fence_message(struct brw_codegen *p, if (sfid == BRW_SFID_URB) { brw_set_desc(p, insn, brw_urb_fence_desc(p->devinfo) | - brw_message_desc(p->devinfo, mlen, rlen, false)); + brw_message_desc(p->devinfo, mlen, rlen, true)); } else { enum lsc_fence_scope scope = lsc_fence_msg_desc_scope(p->devinfo, desc); enum lsc_flush_type flush_type = lsc_fence_msg_desc_flush_type(p->devinfo, desc);