<div dir="ltr"><div>I've already pushed the patch and marked it for stable.</div><div><br></div><div>I added a note into the commit message to discard the version bump for stable.</div><div><br></div><div>The GCR packet is always supported in the ring regardless of firmware.<br></div><div><br></div><div>Marek<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 27, 2020 at 9:01 AM Deucher, Alexander <<a href="mailto:Alexander.Deucher@amd.com">Alexander.Deucher@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">




<div dir="ltr">
<p style="font-family:Arial;font-size:10pt;color:rgb(0,120,215);margin:15pt" align="Left">
[AMD Official Use Only - Internal Distribution Only]<br>
</p>
<br>
<div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
Please split the patch into two, one to update the emit IB sequence, and one to bump the minor version.  That way we can make sure older kernels get the new sequence.  Also do we need an sdma fw version check for the new packet in the emi IB function?</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
Alex<br>
</div>
<div id="gmail-m_-8939609776142340197appendonsend"></div>
<hr style="display:inline-block;width:98%">
<div id="gmail-m_-8939609776142340197divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org" target="_blank">amd-gfx-bounces@lists.freedesktop.org</a>> on behalf of Christian König <<a href="mailto:christian.koenig@amd.com" target="_blank">christian.koenig@amd.com</a>><br>
<b>Sent:</b> Sunday, April 26, 2020 4:52 AM<br>
<b>To:</b> Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>><br>
<b>Cc:</b> amd-gfx mailing list <<a href="mailto:amd-gfx@lists.freedesktop.org" target="_blank">amd-gfx@lists.freedesktop.org</a>><br>
<b>Subject:</b> Re: drm/amdgpu: invalidate L2 before SDMA IBs (on gfx10)</font>
<div> </div>
</div>
<div style="background-color:rgb(255,255,255)">
<div>Thought so, we should try to get this get it committed ASAP. And maybe add a CC: stable tag as well.<br>
<br>
Patch is Reviewed-by: Christian König <a href="mailto:christian.koenig@amd.com" target="_blank">
<christian.koenig@amd.com></a>.<br>
<br>
Thanks,<br>
Christian.<br>
<br>
Am 26.04.20 um 02:28 schrieb Marek Olšák:<br>
</div>
<blockquote type="cite">
<div dir="auto">Not without a mandatory firmware update. The gcr packet support for IBs was added into the sdma firmware just two weeks ago.
<div dir="auto">
<div dir="auto"><br>
</div>
<div dir="auto">Marek</div>
</div>
</div>
<br>
<div>
<div dir="ltr">On Sat., Apr. 25, 2020, 11:04 Christian König, <<a href="mailto:ckoenig.leichtzumerken@gmail.com" target="_blank">ckoenig.leichtzumerken@gmail.com</a>> wrote:<br>
</div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div bgcolor="#FFFFFF">
<div>Could we do this in userspace as well?<br>
<br>
Am 25.04.20 um 10:52 schrieb Marek Olšák:<br>
</div>
<blockquote type="cite">
<div dir="ltr">
<div>This should fix SDMA hangs on gfx10.<br>
</div>
<div><br>
</div>
<div>Marek<br>
</div>
</div>
<br>
<fieldset></fieldset>
<pre>_______________________________________________
amd-gfx mailing list
<a href="mailto:amd-gfx@lists.freedesktop.org" rel="noreferrer" target="_blank">amd-gfx@lists.freedesktop.org</a>
<a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Calexander.deucher%40amd.com%7Cc88f73e068bb483bb21f08d7e9bf2cd4%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637234879980330283&sdata=LZ2GMq0a0n6cm74ulhIkMq%2Fyj6XlG1lj9AH0InU1%2BdQ%3D&reserved=0" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a>
</pre>
</blockquote>
<br>
</div>
</blockquote>
</div>
</blockquote>
<br>
</div>
</div>
</div>

</blockquote></div>