<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">It would probably be better to
      initialize job->vm_pd_addr with AMDGPU_BO_INVALID_OFFSET.<br>
      <br>
      And then just drop the vm flush alltogether when the vm_pd_addr
      isn't set to something sane.<br>
      <br>
      Thanks,<br>
      Christian.<br>
      <br>
      Am 11.09.2018 um 00:52 schrieb Andrey Grodzovsky:<br>
    </div>
    <blockquote type="cite"
      cite="mid:059118f3-2729-12a1-7c8d-e306f69369aa@amd.com">Attached
      patch fixes SDMA TO after GPU reset, it's a regression caused by
      cbd5285 drm/amdgpu: move setting the GART addr into TTM.
      <br>
      <br>
      But to me it looks safer just to revert the original patch all
      together since we never can predict for sure if VM flush will take
      place and so it's safer to just always assign job->vm_pd_addr.
      <br>
      <br>
      Andrey
      <br>
      <br>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
amd-gfx mailing list
<a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a>
<a class="moz-txt-link-freetext" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>