Re: [PATCH i-g-t 2/3] tests/intel/xe_oa: Don't limit sync tests to render engine

["Pottumuttu, Sai Teja" <sai.teja.pottumuttu@intel.com>]

On 14-03-2025 05:50, Ashutosh Dixit wrote:
> OA sync tests need not be limited to render engine only. Extend them to use
> any engine using __for_one_hwe_in_oag macro instead.
> 
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

LGTM,

Reviewed-by: Sai Teja Pottumuttu <sai.teja.pottumuttu@intel.com>


> ---
>   tests/intel/xe_oa.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_oa.c b/tests/intel/xe_oa.c
> index 245481f8a3..2c48623a3b 100644
> --- a/tests/intel/xe_oa.c
> +++ b/tests/intel/xe_oa.c
> @@ -4905,7 +4905,7 @@ igt_main
>   
>   		for (const struct sync_section *s = sync_sections; s->name; s++) {
>   			igt_subtest_with_dynamic_f("syncs-%s", s->name) {
> -				__for_one_render_engine(hwe)
> +				__for_one_hwe_in_oag(hwe)
>   					test_syncs(hwe, s->sync_type, s->flags);
>   			}
>   		}