[PATCH] gpu/drm/radeon: Do not evict VRAM with disabled HIBERNATE

[Christian König]

Am 25.07.2018 um 11:34 schrieb Paul Menzel:
> Dear Christian,
>
>
> On 07/25/18 10:34, Christian König wrote:
>> Am 25.07.2018 um 00:15 schrieb Paul Menzel:
>>> From: Paul Menzel <pmenzel at molgen.mpg.de>
>>> 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: https://bugs.freedesktop.org/show_bug.cgi?id=107277
>>> Cc: stable at vger.kernel.org
>>> Signed-off-by: Paul Menzel <pmenzel at molgen.mpg.de>
>> At least a step into the right direction.
> There is also the *nohibernate* keyword for the Linux command line.

Well in general it would be better to get that information from the PM 
subsystem during suspend.

IIRC in amdgpu we already distinct between suspend and freez in 
amdgpu_device_suspend().

So that information just needs to be passed on to amdgpu_evict_vram().

On radeon it is probably a bit more work to implement this.

Regards,
Christian.

>
>> nohibernate     [HIBERNATION] Disable hibernation and resume.
> I need to investigate, if that can be considered too.
>
>> Patch is Reviewed-by: Christian König <christian.koenig at amd.com>, but we
>> are also going to need one for amdgpu.
> Indeed. I can prepare that too.
>
>
> Kind regards,
>
> Paul
>
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180725/3933feea/attachment.html>