Re: [PATCH i-g-t] tests/intel/xe_exec_reset: Remove CANCEL logic for CM variants

[Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>]

On Mon, Apr 06, 2026 at 10:00:42PM +0000, Stuart Summers wrote:
>The compute mode tests are expected to be long running and will
>not trigger the TDR logic in the KMD. This is illustrated in
>the test in that the legacy cases implement the CANCEL case,
>which is meant to trigger the TDR based resets vs the CM
>cases which do not support this.
>
>Remove the CANCEL logic from the CM tests since it isn't actually
>used in any of the test cases here.
>
>Signed-off-by: Stuart Summers <stuart.summers@intel.com>
>---
> tests/intel/xe_exec_reset.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/tests/intel/xe_exec_reset.c b/tests/intel/xe_exec_reset.c
>index 7aaee31dd..4493a56fc 100644
>--- a/tests/intel/xe_exec_reset.c
>+++ b/tests/intel/xe_exec_reset.c
>@@ -421,7 +421,7 @@ test_compute_mode(int fd, struct drm_xe_engine_class_instance *eci,
> 		uint64_t exec_addr;
> 		int e = i % n_exec_queues;
>
>-		if (!i || flags & CANCEL) {
>+		if (!i) {
> 			spin_opts.addr = base_addr + spin_offset;
> 			xe_spin_init(&data[i].spin, &spin_opts);
> 			exec_addr = spin_opts.addr;
>@@ -478,7 +478,7 @@ test_compute_mode(int fd, struct drm_xe_engine_class_instance *eci,
> 	xe_vm_unbind_async(fd, vm, 0, 0, addr, bo_size, sync, 1);
> 	xe_wait_ufence(fd, &data[0].vm_sync, USER_FENCE_VALUE, 0, 3 * NSEC_PER_SEC);
>
>-	if (!(flags & (GT_RESET | CANCEL))) {
>+	if (!(flags & GT_RESET)) {
> 		for (i = 1; i < n_execs; i++)
> 			igt_assert_eq(data[i].data, 0xc0ffee);
> 	}
>-- 

LGTM
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

>2.43.0
>