drm: amdgpu: Create amdgpu_vm_print_task_info()

To avoid repetitive code in amdgpu, create a function that prints the
content of struct amdgpu_task_info.

Reviewed-by: Christian König <christian.koenig@amd.com>
Link: https://lore.kernel.org/r/20250617124949.2151549-3-andrealmeid@igalia.com
Signed-off-by: André Almeida <andrealmeid@igalia.com>

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
index 75262ce..3d88742 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
@@ -124,9 +124,7 @@ static enum drm_gpu_sched_stat amdgpu_job_timedout(struct drm_sched_job *s_job)
 
        ti = amdgpu_vm_get_task_info_pasid(ring->adev, job->pasid);
        if (ti) {
-               dev_err(adev->dev,
-                       "Process information: process %s pid %d thread %s pid %d\n",
-                       ti->process_name, ti->tgid, ti->task_name, ti->pid);
+               amdgpu_vm_print_task_info(adev, ti);
                amdgpu_vm_put_task_info(ti);
        }
 

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index de914a3..3bf63ee 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -3157,3 +3157,12 @@ bool amdgpu_vm_is_bo_always_valid(struct amdgpu_vm *vm, struct amdgpu_bo *bo)
 {
        return bo && bo->tbo.base.resv == vm->root.bo->tbo.base.resv;
 }
+
+void amdgpu_vm_print_task_info(struct amdgpu_device *adev,
+                              struct amdgpu_task_info *task_info)
+{
+       dev_err(adev->dev,
+               " Process %s pid %d thread %s pid %d\n",
+               task_info->process_name, task_info->tgid,
+               task_info->task_name, task_info->pid);
+}

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
index f3ad687..9ec5d94 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
@@ -668,4 +668,7 @@ void amdgpu_vm_tlb_fence_create(struct amdgpu_device *adev,
                                 struct amdgpu_vm *vm,
                                 struct dma_fence **fence);
 
+void amdgpu_vm_print_task_info(struct amdgpu_device *adev,
+                              struct amdgpu_task_info *task_info);
+
 #endif

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
index a3e2787..7923f49 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
@@ -164,10 +164,7 @@ static int gmc_v10_0_process_interrupt(struct amdgpu_device *adev,
                entry->src_id, entry->ring_id, entry->vmid, entry->pasid);
        task_info = amdgpu_vm_get_task_info_pasid(adev, entry->pasid);
        if (task_info) {
-               dev_err(adev->dev,
-                       " in process %s pid %d thread %s pid %d\n",
-                       task_info->process_name, task_info->tgid,
-                       task_info->task_name, task_info->pid);
+               amdgpu_vm_print_task_info(adev, task_info);
                amdgpu_vm_put_task_info(task_info);
        }
 

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c
index 7221140..f15d691 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c
@@ -134,10 +134,7 @@ static int gmc_v11_0_process_interrupt(struct amdgpu_device *adev,
                        entry->src_id, entry->ring_id, entry->vmid, entry->pasid);
                task_info = amdgpu_vm_get_task_info_pasid(adev, entry->pasid);
                if (task_info) {
-                       dev_err(adev->dev,
-                               " in process %s pid %d thread %s pid %d)\n",
-                               task_info->process_name, task_info->tgid,
-                               task_info->task_name, task_info->pid);
+                       amdgpu_vm_print_task_info(adev, task_info);
                        amdgpu_vm_put_task_info(task_info);
                }
 

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c
index b645d3e..de76310 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c
@@ -127,10 +127,7 @@ static int gmc_v12_0_process_interrupt(struct amdgpu_device *adev,
                        entry->src_id, entry->ring_id, entry->vmid, entry->pasid);
                task_info = amdgpu_vm_get_task_info_pasid(adev, entry->pasid);
                if (task_info) {
-                       dev_err(adev->dev,
-                               " in process %s pid %d thread %s pid %d)\n",
-                               task_info->process_name, task_info->tgid,
-                               task_info->task_name, task_info->pid);
+                       amdgpu_vm_print_task_info(adev, task_info);
                        amdgpu_vm_put_task_info(task_info);
                }
 

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
index 99ca08e..b45fa0c 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
@@ -1458,9 +1458,7 @@ static int gmc_v8_0_process_interrupt(struct amdgpu_device *adev,
 
                task_info = amdgpu_vm_get_task_info_pasid(adev, entry->pasid);
                if (task_info) {
-                       dev_err(adev->dev, " for process %s pid %d thread %s pid %d\n",
-                               task_info->process_name, task_info->tgid,
-                               task_info->task_name, task_info->pid);
+                       amdgpu_vm_print_task_info(adev, task_info);
                        amdgpu_vm_put_task_info(task_info);
                }
 

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index 282197f..78f65ae 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -641,10 +641,7 @@ static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev,
 
        task_info = amdgpu_vm_get_task_info_pasid(adev, entry->pasid);
        if (task_info) {
-               dev_err(adev->dev,
-                       " for process %s pid %d thread %s pid %d)\n",
-                       task_info->process_name, task_info->tgid,
-                       task_info->task_name, task_info->pid);
+               amdgpu_vm_print_task_info(adev, task_info);
                amdgpu_vm_put_task_info(task_info);
        }