Bug 219763
Summary: | The system didn't shut off properly with 6.14 kernels | ||
---|---|---|---|
Product: | Drivers | Reporter: | Matt Fagnani (matt.fagnani) |
Component: | Video(DRI - non Intel) | Assignee: | acpi_power-off |
Status: | RESOLVED MOVED | ||
Severity: | normal | ||
Priority: | P3 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | 6.14.0-0.rc0.20250130git72deda0abee6.11.fc42 to 6.14.0-0.rc1.20250207gitbb066fe812d6.19.fc43 | Subsystem: | |
Regression: | Yes | Bisected commit-id: | |
Attachments: | 6.14.0-0.rc1.20250207gitbb066fe812d6.19.fc43 kernel log when the system didn't shut off properly |
Description
Matt Fagnani
2025-02-08 22:28:21 UTC
Please bisect. https://docs.kernel.org/admin-guide/bug-bisect.html I bisected. The first bad commit involved amdgpu pm. ff69bba05f085cd6d4277c27ac7600160167b384 is the first bad commit commit ff69bba05f085cd6d4277c27ac7600160167b384 (HEAD) Author: Boyuan Zhang <boyuan.zhang@amd.com> Date: Wed Oct 2 23:52:01 2024 -0400 drm/amd/pm: add inst to dpm_set_powergating_by_smu Add an instance parameter to amdgpu_dpm_set_powergating_by_smu() function, and use the instance to call set_powergating_by_smu(). v2: remove duplicated functions. remove for-loop in amdgpu_dpm_set_powergating_by_smu(), and temporarily move it to amdgpu_dpm_enable_vcn(), in order to keep the exact same logic as before, until further separation in next patch. v3: drop SI logic in amdgpu_dpm_enable_vcn(). Signed-off-by: Boyuan Zhang <boyuan.zhang@amd.com> Acked-by: Christian König <christian.koenig@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 14 +++++++------- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 6 +++--- drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v4_0_5.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/vcn_v5_0_0.c | 4 ++-- drivers/gpu/drm/amd/pm/amdgpu_dpm.c | 37 ++++++++++++++++++++++++++----------- drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h | 3 ++- 16 files changed, 59 insertions(+), 43 deletions(-) My GPU is a gfx_v8, and there's a gfx_v8_0.c changed in that patch. When I booted 6.14.0-0.rc1.20250207gitbb066fe812d6.19.fc43 with nomodeset on the kernel command line and the simpledrm kernel driver was used, the system shut down normally. I reported this problem at https://gitlab.freedesktop.org/drm/amd/-/issues/3959 Alex Deucher wrote a patch which fixed the problem https://gitlab.freedesktop.org/drm/amd/-/issues/3959#note_2773981 |