drm/amdgpu: allow GTT overcommit during bind
authorChristian König <christian.koenig@amd.com>
Mon, 16 Oct 2017 09:18:54 +0000 (11:18 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 19 Oct 2017 19:27:21 +0000 (15:27 -0400)
While binding BOs to GART we need to allow a bit overcommit in the GTT
domain. Otherwise we can never use the full GART space when GART size=GTT size.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c

index 0d15eb7d31d7d0471b9253155c211bf5d3a641bc..33535d3477343bab044dba5c1cb171ed6befa475 100644 (file)
@@ -169,7 +169,8 @@ static int amdgpu_gtt_mgr_new(struct ttm_mem_type_manager *man,
        int r;
 
        spin_lock(&mgr->lock);
-       if (atomic64_read(&mgr->available) < mem->num_pages) {
+       if ((&tbo->mem == mem || tbo->mem.mem_type != TTM_PL_TT) &&
+           atomic64_read(&mgr->available) < mem->num_pages) {
                spin_unlock(&mgr->lock);
                return 0;
        }
@@ -244,8 +245,9 @@ static void amdgpu_gtt_mgr_del(struct ttm_mem_type_manager *man,
 uint64_t amdgpu_gtt_mgr_usage(struct ttm_mem_type_manager *man)
 {
        struct amdgpu_gtt_mgr *mgr = man->priv;
+       s64 result = man->size - atomic64_read(&mgr->available);
 
-       return (u64)(man->size - atomic64_read(&mgr->available)) * PAGE_SIZE;
+       return (result > 0 ? result : 0) * PAGE_SIZE;
 }
 
 /**
@@ -265,7 +267,7 @@ static void amdgpu_gtt_mgr_debug(struct ttm_mem_type_manager *man,
        drm_mm_print(&mgr->mm, printer);
        spin_unlock(&mgr->lock);
 
-       drm_printf(printer, "man size:%llu pages, gtt available:%llu pages, usage:%lluMB\n",
+       drm_printf(printer, "man size:%llu pages, gtt available:%lld pages, usage:%lluMB\n",
                   man->size, (u64)atomic64_read(&mgr->available),
                   amdgpu_gtt_mgr_usage(man) >> 20);
 }