<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Am 25.07.2018 um 11:34 schrieb Paul
      Menzel:<br>
    </div>
    <blockquote type="cite"
      cite="mid:1525c8cd-75b8-9b1b-1e43-ca18c5a507bb@molgen.mpg.de">
      <pre wrap="">Dear Christian,


On 07/25/18 10:34, Christian König wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Am 25.07.2018 um 00:15 schrieb Paul Menzel:
</pre>
        <blockquote type="cite">
          <pre wrap="">From: Paul Menzel <a class="moz-txt-link-rfc2396E" href="mailto:pmenzel@molgen.mpg.de"><pmenzel@molgen.mpg.de></a>
Date: Wed, 18 Jul 2018 18:15:44 +0200

Improve commit d796d844 (drm/radeon/kms: make hibernate work on IGPs) to
only migrate VRAM objects if the Linux kernel is actually built with
support for hibernation (suspend to disk).

Link: <a class="moz-txt-link-freetext" href="https://bugs.freedesktop.org/show_bug.cgi?id=107277">https://bugs.freedesktop.org/show_bug.cgi?id=107277</a>
Cc: <a class="moz-txt-link-abbreviated" href="mailto:stable@vger.kernel.org">stable@vger.kernel.org</a>
Signed-off-by: Paul Menzel <a class="moz-txt-link-rfc2396E" href="mailto:pmenzel@molgen.mpg.de"><pmenzel@molgen.mpg.de></a>
</pre>
        </blockquote>
        <pre wrap="">
At least a step into the right direction.
</pre>
      </blockquote>
      <pre wrap="">
There is also the *nohibernate* keyword for the Linux command line.</pre>
    </blockquote>
    <br>
    Well in general it would be better to get that information from the
    PM subsystem during suspend.<br>
    <br>
    IIRC in amdgpu we already distinct between suspend and freez in
    amdgpu_device_suspend().<br>
    <br>
    So that information just needs to be passed on to
    amdgpu_evict_vram().<br>
    <br>
    On radeon it is probably a bit more work to implement this.<br>
    <br>
    Regards,<br>
    Christian.<br>
    <br>
    <blockquote type="cite"
      cite="mid:1525c8cd-75b8-9b1b-1e43-ca18c5a507bb@molgen.mpg.de">
      <pre wrap="">

</pre>
      <blockquote type="cite">
        <pre wrap="">nohibernate     [HIBERNATION] Disable hibernation and resume.
</pre>
      </blockquote>
      <pre wrap="">
I need to investigate, if that can be considered too.

</pre>
      <blockquote type="cite">
        <pre wrap="">Patch is Reviewed-by: Christian König <a class="moz-txt-link-rfc2396E" href="mailto:christian.koenig@amd.com"><christian.koenig@amd.com></a>, but we
are also going to need one for amdgpu.
</pre>
      </blockquote>
      <pre wrap="">
Indeed. I can prepare that too.


Kind regards,

Paul

</pre>
      <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>