diff options
| author | Zhu Lingshan <lingshan.zhu@amd.com> | 2025-01-26 17:21:10 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-02-21 14:01:20 +0100 |
| commit | ae5ab1c1ae504f622cc1ff48830a9ed48428146d (patch) | |
| tree | 48b76d605b475f3f6a5a50ac414fabc3d110a397 /scripts/objdiff | |
| parent | 98d6ee3e305e8317827232acb98367e8f0814267 (diff) | |
| download | linux-ae5ab1c1ae504f622cc1ff48830a9ed48428146d.tar.gz linux-ae5ab1c1ae504f622cc1ff48830a9ed48428146d.tar.bz2 linux-ae5ab1c1ae504f622cc1ff48830a9ed48428146d.zip | |
amdkfd: properly free gang_ctx_bo when failed to init user queue
[ Upstream commit a33f7f9660705fb2ecf3467b2c48965564f392ce ]
The destructor of a gtt bo is declared as
void amdgpu_amdkfd_free_gtt_mem(struct amdgpu_device *adev, void **mem_obj);
Which takes void** as the second parameter.
GCC allows passing void* to the function because void* can be implicitly
casted to any other types, so it can pass compiling.
However, passing this void* parameter into the function's
execution process(which expects void** and dereferencing void**)
will result in errors.
Signed-off-by: Zhu Lingshan <lingshan.zhu@amd.com>
Reviewed-by: Felix Kuehling <felix.kuehling@amd.com>
Fixes: fb91065851cd ("drm/amdkfd: Refactor queue wptr_bo GART mapping")
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'scripts/objdiff')
0 files changed, 0 insertions, 0 deletions