drm/amdgpu: NULL-check BO's backing store when determining GFX12 PTE flags
authorNatalie Vock <natalie.vock@gmx.de>
Mon, 10 Mar 2025 17:08:05 +0000 (18:08 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 11 Mar 2025 16:37:10 +0000 (12:37 -0400)
PRT BOs may not have any backing store, so bo->tbo.resource will be
NULL. Check for that before dereferencing.

Fixes: 0cce5f285d9a ("drm/amdkfd: Check correct memory types for is_system variable")
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Natalie Vock <natalie.vock@gmx.de>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c

index ea7c32d8380bac56ccbc766349a231e62711d4b3..bf8d01da88154d80c981af4f698f83ce40fb0da8 100644 (file)
@@ -528,8 +528,9 @@ static void gmc_v12_0_get_vm_pte(struct amdgpu_device *adev,
 
        bo_adev = amdgpu_ttm_adev(bo->tbo.bdev);
        coherent = bo->flags & AMDGPU_GEM_CREATE_COHERENT;
-       is_system = (bo->tbo.resource->mem_type == TTM_PL_TT) ||
-               (bo->tbo.resource->mem_type == AMDGPU_PL_PREEMPT);
+       is_system = bo->tbo.resource &&
+               (bo->tbo.resource->mem_type == TTM_PL_TT ||
+                bo->tbo.resource->mem_type == AMDGPU_PL_PREEMPT);
 
        if (bo && bo->flags & AMDGPU_GEM_CREATE_GFX12_DCC)
                *flags |= AMDGPU_PTE_DCC;