From: Ken Wang Date: Thu, 9 Jul 2015 05:48:25 +0000 (+0800) Subject: amdgpu: add base_preferred parameter to amdgpu_vamgr_find_va X-Git-Tag: android-x86-6.0-r1~300 X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=5b01908d1c1f94700e68381afe948045d1ef54a1;p=android-x86%2Fexternal-libdrm.git amdgpu: add base_preferred parameter to amdgpu_vamgr_find_va base_preferred parameter is added to amdgpu_vamgr_find_va so UMD can specify preferred va address when allocating. Signed-off-by: Ken Wang Reviewed-by: Jammy Zhou --- diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c index 07df1c56..1ef15162 100644 --- a/amdgpu/amdgpu_bo.c +++ b/amdgpu/amdgpu_bo.c @@ -62,7 +62,7 @@ static int amdgpu_bo_map(amdgpu_bo_handle bo, uint32_t alignment) memset(&va, 0, sizeof(va)); bo->virtual_mc_base_address = amdgpu_vamgr_find_va(dev->vamgr, - bo->alloc_size, alignment); + bo->alloc_size, alignment, 0); if (bo->virtual_mc_base_address == AMDGPU_INVALID_VA_ADDRESS) return -ENOSPC; diff --git a/amdgpu/amdgpu_internal.h b/amdgpu/amdgpu_internal.h index ee1cb61c..8e6fbf40 100644 --- a/amdgpu/amdgpu_internal.h +++ b/amdgpu/amdgpu_internal.h @@ -123,8 +123,8 @@ struct amdgpu_bo_va_mgr* amdgpu_vamgr_get_global(struct amdgpu_device *dev); void amdgpu_vamgr_reference(struct amdgpu_bo_va_mgr **dst, struct amdgpu_bo_va_mgr *src); -uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, - uint64_t size, uint64_t alignment); +uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, uint64_t size, + uint64_t alignment, uint64_t base_preferred); void amdgpu_vamgr_free_va(struct amdgpu_bo_va_mgr *mgr, uint64_t va, uint64_t size); diff --git a/amdgpu/amdgpu_vamgr.c b/amdgpu/amdgpu_vamgr.c index b15729fa..2328e5d7 100644 --- a/amdgpu/amdgpu_vamgr.c +++ b/amdgpu/amdgpu_vamgr.c @@ -68,8 +68,8 @@ void amdgpu_vamgr_reference(struct amdgpu_bo_va_mgr **dst, *dst = src; } -uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, - uint64_t size, uint64_t alignment) +uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, uint64_t size, + uint64_t alignment, uint64_t base_required) { struct amdgpu_bo_va_hole *hole, *n; uint64_t offset = 0, waste = 0; @@ -77,16 +77,27 @@ uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, alignment = MAX2(alignment, mgr->va_alignment); size = ALIGN(size, mgr->va_alignment); + if (base_required % alignment) + return AMDGPU_INVALID_VA_ADDRESS; + pthread_mutex_lock(&mgr->bo_va_mutex); /* TODO: using more appropriate way to track the holes */ /* first look for a hole */ - LIST_FOR_EACH_ENTRY_SAFE(hole, n, &mgr->va_holes, list) { - offset = hole->offset; - waste = offset % alignment; - waste = waste ? alignment - waste : 0; - offset += waste; - if (offset >= (hole->offset + hole->size)) { - continue; + LIST_FOR_EACH_ENTRY_SAFE(hole, n, &vamgr.va_holes, list) { + if (base_required) { + if(hole->offset > base_required || + (hole->offset + hole->size) < (base_required + size)) + continue; + waste = base_required - hole->offset; + offset = base_required; + } else { + offset = hole->offset; + waste = offset % alignment; + waste = waste ? alignment - waste : 0; + offset += waste; + if (offset >= (hole->offset + hole->size)) { + continue; + } } if (!waste && hole->size == size) { offset = hole->offset; @@ -97,8 +108,7 @@ uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, } if ((hole->size - waste) > size) { if (waste) { - n = calloc(1, - sizeof(struct amdgpu_bo_va_hole)); + n = calloc(1, sizeof(struct amdgpu_bo_va_hole)); n->size = waste; n->offset = hole->offset; list_add(&n->list, &hole->list); @@ -115,9 +125,16 @@ uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, } } - offset = mgr->va_offset; - waste = offset % alignment; - waste = waste ? alignment - waste : 0; + if (base_required) { + if (base_required < mgr->va_offset) + return AMDGPU_INVALID_VA_ADDRESS; + offset = mgr->va_offset; + waste = base_required - mgr->va_offset; + } else { + offset = mgr->va_offset; + waste = offset % alignment; + waste = waste ? alignment - waste : 0; + } if (offset + waste + size > mgr->va_max) { pthread_mutex_unlock(&mgr->bo_va_mutex); @@ -130,6 +147,7 @@ uint64_t amdgpu_vamgr_find_va(struct amdgpu_bo_va_mgr *mgr, n->offset = offset; list_add(&n->list, &mgr->va_holes); } + offset += waste; mgr->va_offset += size + waste; pthread_mutex_unlock(&mgr->bo_va_mutex);