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 >
