[PATCH i-g-t v2 2/2] tests/amdgpu: Add SDMA with UMQ test case to amd_basic

[vitaly prosyak]

Both patches look solid—nice work.
Reviewed-by: Vitaly Prosyak vitaly.prosyak at amd.com>

On 2025-06-24 04:05, Jesse.Zhang wrote:
> Add a new subtest "cs-sdma-with-IP-DMA-UMQ" to verify DMA command
> submission works correctly with UMQ (User Mode Queue) for write,
> constant fill, and copy operations.
>
> The test checks SDMA functionality when using UMQ by calling
> amdgpu_command_submission_sdma() with the UMQ flag set to true.
> The test will only run if the user queue capability is available
> for the DMA IP block.
>
> Signed-off-by: Jesse Zhang <Jesse.Zhang at amd.com>
> ---
>  tests/amdgpu/amd_basic.c | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/tests/amdgpu/amd_basic.c b/tests/amdgpu/amd_basic.c
> index 97a08a9a3..10af3271a 100644
> --- a/tests/amdgpu/amd_basic.c
> +++ b/tests/amdgpu/amd_basic.c
> @@ -116,7 +116,7 @@ static void amdgpu_command_submission_compute(amdgpu_device_handle device, bool
>  						     get_ip_block(device, AMDGPU_HW_IP_COMPUTE),
>  						     user_queue);
>  	/* nop test */
> -	amdgpu_command_submission_compute_nop(device, user_queue);
> +	amdgpu_command_submission_nop(device, AMDGPU_HW_IP_DMA, user_queue);
>  }
>  
>  /**
> @@ -136,6 +136,8 @@ static void amdgpu_command_submission_sdma(amdgpu_device_handle device, bool use
>  	amdgpu_command_submission_copy_linear_helper(device,
>  						     get_ip_block(device, AMDGPU_HW_IP_DMA),
>  						     user_queue);
> +	/* nop test */
> +	amdgpu_command_submission_nop(device, AMDGPU_HW_IP_DMA, user_queue);
>  }
>  
>  /**
> @@ -815,6 +817,14 @@ igt_main
>  			amdgpu_sync_dependency_test(device, true);
>  		}
>  	}
> +
> +	igt_describe("Check-DMA-CS-for-every-available-ring-works-for-write-const-fill-copy-operation");
> +	igt_subtest_with_dynamic("cs-sdma-with-IP-DMA-UMQ") {
> +		if (userq_arr_cap[AMD_IP_DMA]) {
> +			igt_dynamic_f("cs-sdma-with-umq")
> +			amdgpu_command_submission_sdma(device, true);
> +		}
> +	}
>  #endif
>  
>  	igt_fixture {