diff options
| author | xinhui pan <xinhui.pan@amd.com> | 2021-01-30 08:41:04 +0800 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2021-02-02 12:06:21 -0500 | 
| commit | e1a4b67aac9ee870b4d95a008d47910c47db3092 (patch) | |
| tree | 0cd08367c2b04935648e47ed92409f763d864f8f /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
| parent | b131c363c8a37815e5c7d748883f7d3e1d8e40c2 (diff) | |
drm/amdgpu: Fix a false positive when pin non-VRAM memory
Flag TTM_PL_FLAG_CONTIGUOUS is only valid for VRAM domain. So fix the
false positive by checking memory type too.
Suggested-by: Felix Kuehling <Felix.Kuehling@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: xinhui pan <xinhui.pan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 3 | 
1 files changed, 2 insertions, 1 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c index 52f98e370a87..996147111c3e 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c @@ -916,7 +916,8 @@ int amdgpu_bo_pin_restricted(struct amdgpu_bo *bo, u32 domain,  		if (!(domain & amdgpu_mem_type_to_domain(mem_type)))  			return -EINVAL; -		if ((bo->flags & AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS) && +		if ((mem_type == TTM_PL_VRAM) && +		    (bo->flags & AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS) &&  		    !(mem_flags & TTM_PL_FLAG_CONTIGUOUS))  			return -EINVAL; | 
