<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 14, 2018, 8:28 AM Samuel Pitoiset <<a href="mailto:samuel.pitoiset@gmail.com">samuel.pitoiset@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 8/1/18 5:12 AM, Marek Olšák wrote:<br>
> On Fri, Oct 20, 2017 at 4:34 PM, Andres Rodriguez <<a href="mailto:andresx7@gmail.com" target="_blank" rel="noreferrer">andresx7@gmail.com</a>> wrote:<br>
>> When WAVE_LIMIT is set, a submission will opt-in for SPI based resource<br>
>> scheduling. Because this mechanism is cooperative, we must ensure that<br>
>> all submissions have this field set, otherwise they will bypass resource<br>
>> arbitration.<br>
>><br>
>> We always hardcode the field to its maximum value, instead of<br>
>> attempting to calculate an approximate usage. In testing, there were no<br>
>> benefits to using anything other than the maximum.<br>
>><br>
>> Signed-off-by: Andres Rodriguez <<a href="mailto:andresx7@gmail.com" target="_blank" rel="noreferrer">andresx7@gmail.com</a>><br>
>> ---<br>
>>   src/amd/vulkan/si_cmd_buffer.c          | 27 ++++++++++++++++++---------<br>
>>   src/gallium/drivers/radeonsi/si_state.c | 21 ++++++++++++++-------<br>
>>   2 files changed, 32 insertions(+), 16 deletions(-)<br>
>><br>
>> diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c<br>
>> index de3b388..ac3dff8 100644<br>
>> --- a/src/amd/vulkan/si_cmd_buffer.c<br>
>> +++ b/src/amd/vulkan/si_cmd_buffer.c<br>
>> @@ -179,7 +179,8 @@ si_emit_compute(struct radv_physical_device *physical_device,<br>
>>          radeon_emit(cs, 0);<br>
>>          radeon_emit(cs, 0);<br>
>><br>
>> -       radeon_set_sh_reg_seq(cs, R_00B854_COMPUTE_RESOURCE_LIMITS, 3);<br>
>> +       radeon_set_sh_reg_seq(cs, R_00B854_COMPUTE_RESOURCE_LIMITS,<br>
>> +                             S_00B854_WAVES_PER_SH(0x3));<br>
> <br>
> This part doesn't set WAVES_PER_SH. Also, the number 3 would be wrong for it.<br>
<br>
This is just a default value for that register. COMPUTE_RESOURCE_LIMITS <br>
is computed like RadeonSI when we generate the PM4 stuff for compute <br>
pipelines, and then emitted at dispatch time.<br>
<br>
Do you think we should remove that default value?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Please read the patch again. It's obvious if you know what the seq function does.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> <br>
> Marek<br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank" rel="noreferrer">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
> <br>
</blockquote></div></div></div>