Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-25 Thread Mario Limonciello
On 8/25/24 15:40, Luna Nova wrote: Raised this as an issue a while back on the bug tracker and it got closed as WONTFIX. https://gitlab.freedesktop.org/drm/amd/-/issues/2229 Been running a patched kernel with a similar patch locally ever since because even figuring out everything on the syst

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-25 Thread Luna Nova
Raised this as an issue a while back on the bug tracker and it got closed as WONTFIX. https://gitlab.freedesktop.org/drm/amd/-/issues/2229 Been running a patched kernel with a similar patch locally ever since because even figuring out everything on the system that's accidentally waking the GPU w

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-24 Thread Luke Jones
On Sat, 24 Aug 2024, at 2:39 AM, Mario Limonciello wrote: > On 8/23/2024 09:31, Alex Deucher wrote: > > On Fri, Aug 23, 2024 at 10:13 AM Mario Limonciello > > wrote: > >> > >> On 8/23/2024 09:09, Alex Deucher wrote: > >>> On Mon, Aug 19, 2024 at 10:30 PM Mario Limonciello > >>> wrote: > > >

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Mario Limonciello
On 8/23/2024 09:31, Alex Deucher wrote: On Fri, Aug 23, 2024 at 10:13 AM Mario Limonciello wrote: On 8/23/2024 09:09, Alex Deucher wrote: On Mon, Aug 19, 2024 at 10:30 PM Mario Limonciello wrote: From: Mario Limonciello If the dGPU is off, then reading the sysfs files with a sensor monit

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Alex Deucher
On Fri, Aug 23, 2024 at 10:13 AM Mario Limonciello wrote: > > On 8/23/2024 09:09, Alex Deucher wrote: > > On Mon, Aug 19, 2024 at 10:30 PM Mario Limonciello > > wrote: > >> > >> From: Mario Limonciello > >> > >> If the dGPU is off, then reading the sysfs files with a sensor monitoring > >> appl

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Mario Limonciello
On 8/23/2024 09:09, Alex Deucher wrote: On Mon, Aug 19, 2024 at 10:30 PM Mario Limonciello wrote: From: Mario Limonciello If the dGPU is off, then reading the sysfs files with a sensor monitoring application will wake it. Change the behavior to return an error when the dGPU is in D3cold. I

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Alex Deucher
On Mon, Aug 19, 2024 at 10:30 PM Mario Limonciello wrote: > > From: Mario Limonciello > > If the dGPU is off, then reading the sysfs files with a sensor monitoring > application will wake it. Change the behavior to return an error when the > dGPU is in D3cold. I'm a little concerned that this wi

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Mario Limonciello
On 8/23/2024 08:44, Hamza Mahfooz wrote: On 8/19/24 22:04, Mario Limonciello wrote: From: Mario Limonciello If the dGPU is off, then reading the sysfs files with a sensor monitoring application will wake it. Change the behavior to return an error when the dGPU is in D3cold. Signed-off-by: Mar

Re: [PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-23 Thread Hamza Mahfooz
On 8/19/24 22:04, Mario Limonciello wrote: From: Mario Limonciello If the dGPU is off, then reading the sysfs files with a sensor monitoring application will wake it. Change the behavior to return an error when the dGPU is in D3cold. Signed-off-by: Mario Limonciello --- drivers/gpu/drm/amd/

[PATCH] drm/amd: Don't wake dGPUs while reading sensors

2024-08-19 Thread Mario Limonciello
From: Mario Limonciello If the dGPU is off, then reading the sysfs files with a sensor monitoring application will wake it. Change the behavior to return an error when the dGPU is in D3cold. Signed-off-by: Mario Limonciello --- drivers/gpu/drm/amd/pm/amdgpu_pm.c | 90 +++---