diff options
| author | Nathan Chancellor <nathan@kernel.org> | 2022-10-17 09:28:38 -0700 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2022-10-17 17:41:21 -0400 |
| commit | e299b00adf3d4505132e624894f549422ad05eeb (patch) | |
| tree | 8497578315fbd93e351a82b0a9fbd00422a7ada1 /drivers/gpu/drm/amd/display/dc/irq/irq_service.c | |
| parent | a2c554262d39f81be7422fd8bee2f2fe3779f7f5 (diff) | |
| download | linux-e299b00adf3d4505132e624894f549422ad05eeb.tar.gz linux-e299b00adf3d4505132e624894f549422ad05eeb.tar.bz2 linux-e299b00adf3d4505132e624894f549422ad05eeb.zip | |
drm/amdkfd: Fix type of reset_type parameter in hqd_destroy() callback
When booting a kernel compiled with CONFIG_CFI_CLANG on a machine with
an RX 6700 XT, there is a CFI failure in kfd_destroy_mqd_cp():
[ 12.894543] CFI failure at kfd_destroy_mqd_cp+0x2a/0x40 [amdgpu] (target: hqd_destroy_v10_3+0x0/0x260 [amdgpu]; expected type: 0x8594d794)
Clang's kernel Control Flow Integrity (kCFI) makes sure that all
indirect call targets have a type that exactly matches the function
pointer prototype. In this case, hqd_destroy()'s third parameter,
reset_type, should have a type of 'uint32_t' but every implementation of
this callback has a third parameter type of 'enum kfd_preempt_type'.
Update the function pointer prototype to match reality so that there is
no more CFI violation.
Link: https://github.com/ClangBuiltLinux/linux/issues/1738
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/dc/irq/irq_service.c')
0 files changed, 0 insertions, 0 deletions