<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Agree, the placement of amdgpu_amdkfd_pre/post _reset in amdgpu_device_lock/unlock_adev is a bit wierd.</p>
<p>Andrey<br>
</p>
<div class="moz-cite-prefix">On 8/29/19 10:06 AM, Koenig, Christian wrote:<br>
</div>
<blockquote type="cite" cite="mid:8c2b0bf3-47ff-ea3b-a682-7ec76240be4d@amd.com">
<blockquote type="cite" style="color: #000000;">
<pre class="moz-quote-pre" wrap="">Felix advised that the way to stop all KFD activity is simply to NOT
call amdgpu_amdkfd_post_reset so that why I added this. Do you mean you
prefer amdgpu_amdkfd_post_reset to be outside of amdgpu_device_unlock_adev ?
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">Yes, exactly. It doesn't seems to be related to the unlock operation in
the first place, but rather only signals the KFD that the reset is
completed.
Christian.
</pre>
</blockquote>
</body>
</html>