[PATCH] drm/amdgpu: fix amdgpu_bo_gpu_accessible()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Seems to fix the regression I noticed on Carrizo.  Can add my Acked-By 
if you haven't pushed it already.

Cheers,
Tom

On 13/07/17 08:19 AM, He, Roger wrote:
> Reviewed-and-Tested-by: Roger He <Hongbo.He at amd.com>
> 
> Thanks
> Roger(Hongbo.He)
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of Christian K?nig
> Sent: Thursday, July 13, 2017 6:23 PM
> To: amd-gfx at lists.freedesktop.org
> Cc: Huang, Ray <Ray.Huang at amd.com>
> Subject: [PATCH] drm/amdgpu: fix amdgpu_bo_gpu_accessible()
> 
> From: Christian König <christian.koenig at amd.com>
> 
> The test was relaxed a bit to much.
> 
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_object.h | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
> index 3824851..833b172 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
> @@ -120,7 +120,11 @@ static inline u64 amdgpu_bo_mmap_offset(struct amdgpu_bo *bo)
>    */
>   static inline bool amdgpu_bo_gpu_accessible(struct amdgpu_bo *bo)  {
> -	return bo->tbo.mem.mem_type != TTM_PL_SYSTEM;
> +	switch (bo->tbo.mem.mem_type) {
> +	case TTM_PL_TT: return amdgpu_ttm_is_bound(bo->tbo.ttm);
> +	case TTM_PL_VRAM: return true;
> +	default: return false;
> +	}
>   }
>   
>   int amdgpu_bo_create(struct amdgpu_device *adev,
> --
> 2.7.4
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 



[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux