[PATCH] tests/amdgpu: add missing user queue destroy for cs_nops

Zhang, Jesse(Jie) Jesse.Zhang at amd.com
Mon Apr 21 08:26:55 UTC 2025


[AMD Official Use Only - AMD Internal Distribution Only]

This patch is Reviewed-by: "Jesse.zhang at amd.com"

-----Original Message-----
From: Khatri, Sunil <Sunil.Khatri at amd.com>
Sent: Monday, April 21, 2025 4:11 PM
To: Khatri, Sunil <Sunil.Khatri at amd.com>; igt-dev at lists.freedesktop.org; Prosyak, Vitaly <Vitaly.Prosyak at amd.com>; Zhang, Jesse(Jie) <Jesse.Zhang at amd.com>
Subject: RE: [PATCH] tests/amdgpu: add missing user queue destroy for cs_nops

[AMD Official Use Only - AMD Internal Distribution Only]

Adding jesse

-----Original Message-----
From: Sunil Khatri <sunil.khatri at amd.com>
Sent: Monday, April 21, 2025 1:29 PM
To: igt-dev at lists.freedesktop.org; Prosyak, Vitaly <Vitaly.Prosyak at amd.com>
Cc: Khatri, Sunil <Sunil.Khatri at amd.com>
Subject: [PATCH] tests/amdgpu: add missing user queue destroy for cs_nops

user queues after completing the test case should destroy the user queues and do the necessary clean up. clean up was missing for cs_nops and that is added with the patch.

Signed-off-by: Sunil Khatri <sunil.khatri at amd.com>
---
 tests/amdgpu/amd_cs_nop.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/amdgpu/amd_cs_nop.c b/tests/amdgpu/amd_cs_nop.c index 167699328..67bb0419d 100644
--- a/tests/amdgpu/amd_cs_nop.c
+++ b/tests/amdgpu/amd_cs_nop.c
@@ -139,6 +139,9 @@ static void nop_cs(amdgpu_device_handle device,

        amdgpu_bo_unmap_and_free(ib_result_handle, va_handle,
                                 ib_result_mc_address, 4096);
+       if (user_queue)
+               amdgpu_user_queue_destroy(device, ring_context, ip_type);
+
        free(ring_context);
 }

--
2.43.0




More information about the igt-dev mailing list