On Tue, Sep 20, 2016 at 06:19:33PM +0800, zhoucm1 wrote: > > > On 2016年09月20日 17:09, Flora Cui wrote: > >Change-Id: I89e9be3d5c96d46655f3a977fb557b20b4d87609 > >Signed-off-by: Flora Cui <flora....@amd.com> > >--- > > drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > >diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > >b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > >index 262e872..b266cf7 100644 > >--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > >+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > >@@ -128,8 +128,8 @@ int amdgpu_gtt_mgr_alloc(struct ttm_mem_type_manager > >*man, > > if (!r) { > > mem->start = node->start; > >- tbo->offset = (tbo->mem.start << PAGE_SHIFT) + > >- tbo->bdev->man[tbo->mem.mem_type].gpu_offset; > >+ tbo->offset = (mem->start << PAGE_SHIFT) + > >+ tbo->bdev->man[mem->mem_type].gpu_offset; > mem->mem_type seems not set yet as well if I am correct. > IIRC, I feel it's better that tbo->offset is set out of this function after > get_node successfully. > how about change to tbo->bdev->man[TTM_PL_TT].gpu_offset? since gtt_mgr is for GTT only.
> Regards, > David Zhou > > > } > > return r; > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx