linux/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c, branch v6.17Mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
https://git.shady.money/linux/atom?h=v6.172025-07-28T20:26:52Zdrm/amd/amdgpu: fix missing lock for cper.ring->rptr/wptr access2025-07-28T20:26:52ZYang Wangkevinyang.wang@amd.com2025-07-24T07:16:18Zurn:sha1:8e0d1edb5c16732b695eaf4bd7096b1569817cf0
Add lock protection for 'ring->wptr'/'ring->rptr' to ensure the correct execution.
Fixes: 8652920d2c00 ("drm/amdgpu: add mutex lock for cper ring")
Signed-off-by: Yang Wang <kevinyang.wang@amd.com>
Reviewed-by: Tao Zhou <tao.zhou1@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
drm/amdgpu: Use correct severity for BP threshold exceed event2025-06-30T16:08:00ZXiang Liuxiang.liu@amd.com2025-06-27T15:14:08Zurn:sha1:4a33ca3f6ee9a013a423a867426704e9c9d785bd
The severity of CPER for BP threshold exceed event should be set as
CPER_SEV_FATAL to match the OOB implementation.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Disable ACA on VFs2025-04-08T20:48:14ZVictor Skvortsovvictor.skvortsov@amd.com2025-04-02T21:35:56Zurn:sha1:3394069e7de984ea3a4c2c68bf69b65f1c0aa658
VFs query RAS error counts directly from host with
AMDGPU_RAS_VIRT_ERROR_COUNT_QUERY. When ACA is enabled,
an unusable aca_sysfs is created rather than amdgpu_ras_sysfs_create()
Likewise, VFs depend on host support to query CPERs, rather than ACA component.
Signed-off-by: Victor Skvortsov <victor.skvortsov@amd.com>
Reviewed-by: Zhigang Luo <Zhigang.luo@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Fix computation for remain size of CPER ring2025-03-18T18:03:46ZXiang Liuxiang.liu@amd.com2025-03-13T03:24:55Zurn:sha1:d6f9bbce18762116698c40cc7de836a9d0c9f832
The mistake of computation for remain size of CPER ring will cause
unbreakable while cycle when CPER ring overflow.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Use unique CPER record id across devices2025-03-07T17:54:08ZXiang Liuxiang.liu@amd.com2025-03-06T07:23:34Zurn:sha1:148084bbb1e5131b3f1200c72c2b60d85e73aa75
Encode socket id to CPER record id to be unique across devices.
v2: add pointer check for adev->smuio.funcs->get_socket_id
v2: set 0 if adev->smuio.funcs->get_socket_id is NULL
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Tao Zhou <tao.zhou1@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Free CPER entry after committing to ring2025-03-05T15:37:27ZXiang Liuxiang.liu@amd.com2025-02-28T03:11:08Zurn:sha1:677ae51f4901e22517f8291a0263f794c104e167
Free CPER entry when it's committed to CPER ring to avoid memory leak.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Fix spelling mistake "initiailize" -> "initialize" and grammar2025-02-27T21:50:03ZColin Ian Kingcolin.i.king@gmail.com2025-02-26T08:57:33Zurn:sha1:eaa3feb16d3b8fbb787cc6c0c16c45bf654d806a
There is a spelling mistake and a grammatical error in a dev_err
message. Fix it.
Reviewed-by: Tao Zhou <tao.zhou1@amd.com>
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Disable fru_id field in CPER section2025-02-27T21:50:03ZXiang Liuxiang.liu@amd.com2025-02-24T07:13:40Zurn:sha1:906d2859e1c69ad3bd680fc02a2de00105ee827b
The fru_id field is disabled cause of mis-matching defination
between CPER spec and driver.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Set CPER enabled flag after ring initiailized2025-02-25T16:45:12ZXiang Liuxiang.liu@amd.com2025-02-24T15:01:06Zurn:sha1:ff930483af93753cc3ba252e13f3cd92e505f0f0
Setting cper.enabled to be true only after cper ring is successfully
created.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Tao Zhou <tao.zhou1@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drm/amdgpu: Check aca enabled inside cper init/fini func2025-02-19T20:16:33ZXiang Liuxiang.liu@amd.com2025-02-19T04:21:59Zurn:sha1:663a87763b570d4e92d821b30508bed0025fa285
Move code about checking aca enabled to the cper init/fini function
to make code clean.
Signed-off-by: Xiang Liu <xiang.liu@amd.com>
Reviewed-by: Tao Zhou <tao.zhou1@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>