<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 8, 2023 at 9:57 AM Christian König <<a href="mailto:christian.koenig@amd.com" target="_blank">christian.koenig@amd.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am 08.12.23 um 12:43 schrieb Friedrich Vock:<br>
> On 08.12.23 10:51, Christian König wrote:<br>
>> Well longer story short Alex and I have been digging up the<br>
>> documentation for this and as far as we can tell this isn't correct.<br>
> Huh. I initially talked to Marek about this, adding him in Cc.<br>
<br>
Yeah, from the userspace side all you need to do is to set the bit as <br>
far as I can tell.<br>
<br>
>><br>
>> You need to do quite a bit more before you can turn on this feature.<br>
>> What userspace side do you refer to?<br>
> I was referring to the Mesa merge request I made<br>
> (<a href="https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26462" rel="noreferrer" target="_blank">https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26462</a>).<br>
> If/When you have more details about what else needs to be done, feel<br>
> free to let me know.<br>
<br>
For example from the hardware specification explicitly states that the <br>
kernel driver should make sure that only one app/queue is using this at <br>
the same time. That might work for now since we should only have a <br>
single compute priority queue, but we are not 100% sure yet.<br></blockquote><div><br></div><div>This is incorrect. While the hw documentation says it's considered "unexpected programming", it also says that the hardware algorithm handles it correctly and it describes what happens in this case: Tunneled waves from different queues are treated as equal.<br></div><div><br></div><div>Marek</div></div></div>