[PATCH 3/3] drm/amdgpu: move gtt usage statistic to gtt mgr
Christian König
christian.koenig at amd.com
Wed Apr 19 06:45:01 UTC 2017
Am 19.04.2017 um 05:50 schrieb Chunming Zhou:
> Change-Id: Ifea42c8ae2206143d7e22b35eea537ba9e928fe8
> Signed-off-by: Chunming Zhou <David1.Zhou at amd.com>
Nice idea for a cleanup, but please do even a few more things:
1. Rebase on not moving the statistics from mgr_new to mgr_alloc.
2. Joing mgr->available and adev->gtt_usage, keeping track of both is
probably not a good idea.
3. Do the same thing for the VRAM stats as well.
Regards,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c | 13 ++++++++++---
> drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 6 ------
> 2 files changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> index 85de145..db7bbde 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> @@ -97,6 +97,7 @@ int amdgpu_gtt_mgr_alloc(struct ttm_mem_type_manager *man,
> {
> struct amdgpu_gtt_mgr *mgr = man->priv;
> struct drm_mm_node *node = mem->mm_node;
> + struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
> enum drm_mm_search_flags sflags = DRM_MM_SEARCH_BEST;
> enum drm_mm_allocator_flags aflags = DRM_MM_CREATE_DEFAULT;
> unsigned long fpfn, lpfn;
> @@ -124,8 +125,10 @@ int amdgpu_gtt_mgr_alloc(struct ttm_mem_type_manager *man,
> r = drm_mm_insert_node_in_range_generic(&mgr->mm, node, mem->num_pages,
> mem->page_alignment, 0,
> fpfn, lpfn, sflags, aflags);
> - if (!r)
> + if (!r) {
> mgr->available -= mem->num_pages;
> + atomic64_add(mem->size, &adev->gtt_usage);
> + }
> spin_unlock(&mgr->lock);
>
> if (!r) {
> @@ -140,10 +143,12 @@ int amdgpu_gtt_mgr_alloc(struct ttm_mem_type_manager *man,
>
> void amdgpu_gtt_mgr_print(struct seq_file *m, struct ttm_mem_type_manager *man)
> {
> + struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
> struct amdgpu_gtt_mgr *mgr = man->priv;
>
> - seq_printf(m, "man size:%llu pages, gtt available:%llu pages\n",
> - man->size, mgr->available);
> + seq_printf(m, "man size:%llu pages, gtt available:%llu pages, usage:%lluMB\n",
> + man->size, mgr->available,
> + (u64)atomic64_read(&adev->gtt_usage) >> 20);
> }
> /**
> * amdgpu_gtt_mgr_new - allocate a new node
> @@ -214,6 +219,7 @@ static void amdgpu_gtt_mgr_del(struct ttm_mem_type_manager *man,
> {
> struct amdgpu_gtt_mgr *mgr = man->priv;
> struct drm_mm_node *node = mem->mm_node;
> + struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
>
> if (!node)
> return;
> @@ -222,6 +228,7 @@ static void amdgpu_gtt_mgr_del(struct ttm_mem_type_manager *man,
> if (node->start != AMDGPU_BO_INVALID_OFFSET) {
> drm_mm_remove_node(node);
> mgr->available += mem->num_pages;
> + atomic64_sub(mem->size, &adev->gtt_usage);
> }
> spin_unlock(&mgr->lock);
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> index 3cde1c9..2249eb6 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> @@ -61,9 +61,6 @@ static void amdgpu_update_memory_usage(struct amdgpu_device *adev,
>
> if (new_mem) {
> switch (new_mem->mem_type) {
> - case TTM_PL_TT:
> - atomic64_add(new_mem->size, &adev->gtt_usage);
> - break;
> case TTM_PL_VRAM:
> atomic64_add(new_mem->size, &adev->vram_usage);
> vis_size = amdgpu_get_vis_part_size(adev, new_mem);
> @@ -80,9 +77,6 @@ static void amdgpu_update_memory_usage(struct amdgpu_device *adev,
>
> if (old_mem) {
> switch (old_mem->mem_type) {
> - case TTM_PL_TT:
> - atomic64_sub(old_mem->size, &adev->gtt_usage);
> - break;
> case TTM_PL_VRAM:
> atomic64_sub(old_mem->size, &adev->vram_usage);
> vis_size = amdgpu_get_vis_part_size(adev, old_mem);
More information about the amd-gfx
mailing list