[PATCH 5/8] drm/amdgpu: don't use ttm_bo_move_ttm in amdgpu_ttm_bind
Christian König
ckoenig.leichtzumerken at gmail.com
Thu Oct 26 18:39:23 UTC 2017
Am 26.10.2017 um 20:11 schrieb Deucher, Alexander:
>> -----Original Message-----
>> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
>> Of Christian König
>> Sent: Thursday, October 26, 2017 12:06 PM
>> To: amd-gfx at lists.freedesktop.org
>> Subject: [PATCH 5/8] drm/amdgpu: don't use ttm_bo_move_ttm in
>> amdgpu_ttm_bind
>>
>> From: Christian König <christian.koenig at amd.com>
>>
>> Just unbind and rebind to force updates of the GART space.
>>
>> This prevents forcing the BO to be idle.
> Is there a chance something could change like the caching on a rebind that we need to account for?
Not that I could think of.
The new placement should have identical attributes to the old one, just
with GART space allocated.
Christian.
>
> Alex
>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>> drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 24 ++++++++++++++++++-
>> -----
>> 1 file changed, 18 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> index 3045701..b40d2f3 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> @@ -917,14 +917,26 @@ int amdgpu_ttm_bind(struct ttm_buffer_object
>> *bo, struct ttm_mem_reg *bo_mem)
>> if (unlikely(r))
>> return r;
>>
>> - r = ttm_bo_move_ttm(bo, true, false, &tmp);
>> - if (unlikely(r))
>> + if (ttm->state == tt_bound) {
>> + r = ttm->func->unbind(ttm);
>> + if (unlikely(r))
>> + return r;
>> +
>> + ttm->state = tt_unbound;
>> + }
>> +
>> + r = ttm_tt_bind(ttm, &tmp);
>> + if (unlikely(r)) {
>> ttm_bo_mem_put(bo, &tmp);
>> - else
>> - bo->offset = (bo->mem.start << PAGE_SHIFT) +
>> - bo->bdev->man[bo->mem.mem_type].gpu_offset;
>> + return r;
>> + }
>>
>> - return r;
>> + ttm_bo_mem_put(bo, bo_mem);
>> + bo->mem = tmp;
>> + bo->offset = (bo->mem.start << PAGE_SHIFT) +
>> + bo->bdev->man[bo->mem.mem_type].gpu_offset;
>> +
>> + return 0;
>> }
>>
>> int amdgpu_ttm_recover_gart(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
More information about the amd-gfx
mailing list