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