RE: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[Public]

 

> PM_SUSPEND_TO_IDLE should be under a compile guard

 

It is actually.  All of the amdgpu_acpi_* are.  It’s not obvious though looking at the patch, you need to apply it to notice it.

 

> It makes sense to rename to something like amdgpu_need_reset_on_suspend() as it decides on reset only for a suspend situation.

 

Remember it’s in amdgpu_acpi.c.  So it would be amdgpu_acpi_need_reset_on_suspend()?  It’s a bit lengthy.

 

From: Lazar, Lijo <Lijo.Lazar@xxxxxxx>
Sent: Tuesday, May 17, 2022 11:25
To: Limonciello, Mario <Mario.Limonciello@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: Limonciello, Mario <Mario.Limonciello@xxxxxxx>
Subject: Re: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle

 

[Public]

 

A couple of things -

 

PM_SUSPEND_TO_IDLE should be under a compile guard

 

It makes sense to rename to something like amdgpu_need_reset_on_suspend() as it decides on reset only for a suspend situation.

 

Thanks,
Lijo


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux