[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230828122614.3815122-6-Arvind.Yadav@amd.com>
Date: Mon, 28 Aug 2023 17:56:12 +0530
From: Arvind Yadav <Arvind.Yadav@....com>
To: <Christian.Koenig@....com>, <alexander.deucher@....com>,
<shashank.sharma@....com>, <Xinhui.Pan@....com>,
<airlied@...il.com>, <daniel@...ll.ch>, <Felix.Kuehling@....com>,
<amd-gfx@...ts.freedesktop.org>
CC: <dri-devel@...ts.freedesktop.org>, <linux-kernel@...r.kernel.org>,
"Arvind Yadav" <Arvind.Yadav@....com>,
Christian Koenig <christian.koenig@....com>
Subject: [PATCH v3 5/7] drm/amdgpu: Set/Reset GPU workload profile
This patch is to switch the GPU workload profile based
on the submitted job. The workload profile is reset to
default when the job is done.
v3:
- Addressed the review comment about changing the function
name from *_set() to *_get().
Cc: Christian Koenig <christian.koenig@....com>
Cc: Alex Deucher <alexander.deucher@....com>
Reviewed-by: Shashank Sharma <shashank.sharma@....com>
Signed-off-by: Arvind Yadav <Arvind.Yadav@....com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
index c3d9d75143f4..c5032762d497 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
@@ -176,6 +176,9 @@ void amdgpu_job_free_resources(struct amdgpu_job *job)
static void amdgpu_job_free_cb(struct drm_sched_job *s_job)
{
struct amdgpu_job *job = to_amdgpu_job(s_job);
+ struct amdgpu_ring *ring = to_amdgpu_ring(s_job->sched);
+
+ amdgpu_workload_profile_put(ring->adev, ring->funcs->type);
drm_sched_job_cleanup(s_job);
@@ -295,6 +298,8 @@ static struct dma_fence *amdgpu_job_run(struct drm_sched_job *sched_job)
DRM_ERROR("Error scheduling IBs (%d)\n", r);
}
+ amdgpu_workload_profile_get(adev, ring->funcs->type);
+
job->job_run_counter++;
amdgpu_job_free_resources(job);
--
2.34.1
Powered by blists - more mailing lists