From: nishit.sharma@intel.com
To: igt-dev@lists.freedesktop.org, kamil.konieczny@intel.com
Subject: [PATCH] tests/intel/xe_exec_reset: add GT reset fault injection stress coverage
Date: Tue, 25 Aug 2026 03:25:20 +0000 [thread overview]
Message-ID: <20260825032520.507695-1-nishit.sharma@intel.com> (raw)
From: Nishit Sharma <nishit.sharma@intel.com>
Inject a GT reset failure via the KMD fail_gt_reset debugfs hook, verify
the device wedges as expected, tolerate the resulting -ECANCELED errors in
the submitting threads, then recover by rebind and confirm the driver is
usable again.
Signed-off-by: Nishit Sharma <nishit.sharma@intel.com>
---
tests/intel/xe_exec_reset.c | 107 ++++++++++++++++++++++++++++++++++--
1 file changed, 101 insertions(+), 6 deletions(-)
diff --git a/tests/intel/xe_exec_reset.c b/tests/intel/xe_exec_reset.c
index 6eda71c32..1796e4ec1 100644
--- a/tests/intel/xe_exec_reset.c
+++ b/tests/intel/xe_exec_reset.c
@@ -16,6 +16,8 @@
#include "igt.h"
#include "igt_sysfs.h"
+#include "igt_device.h"
+#include "igt_kmod.h"
#include "lib/igt_syncobj.h"
#include "lib/intel_reg.h"
#include "xe_drm.h"
@@ -140,6 +142,7 @@ static void test_spin(int fd, struct drm_xe_engine_class_instance *eci,
#define DESTROY_VM_CTX_STRESS (0x1 << 20)
#define MIXED_ENGINE_STRESS (0x1 << 21)
#define PM_TRANSITION_STRESS (0x1 << 22)
+#define FAULT_INJECT_STRESS (0x1 << 23)
/**
* SUBTEST: %s-cat-error
@@ -760,7 +763,7 @@ static void submit_jobs(struct gt_thread_data *t)
uint32_t pressure_bos[PRESSURE_COUNT];
uint32_t *data;
int pressure_count;
- int i = 0;
+ int i = 0, exec_ret;
bo = xe_bo_create(fd, vm, bo_size, vram_if_possible(fd, t->gt),
DRM_XE_GEM_CREATE_FLAG_NEEDS_VISIBLE_VRAM);
@@ -794,8 +797,25 @@ static void submit_jobs(struct gt_thread_data *t)
continue;
}
- xe_exec(fd, &exec);
- xe_exec_queue_destroy(fd, exec.exec_queue_id);
+ /*
+ * Once an injected GT reset failure wedges the device, exec and
+ * queue teardown return -ECANCELED. That is the expected outcome
+ * for the fault-injection stress
+ */
+ if (t->flags & FAULT_INJECT_STRESS) {
+ struct drm_xe_exec_queue_destroy destroy = {
+ .exec_queue_id = exec.exec_queue_id,
+ };
+
+ exec_ret = __xe_exec(fd, &exec);
+ igt_assert_f(exec_ret == 0 || exec_ret == -ECANCELED,
+ "exec returned unexpected error %d (expected 0 or -ECANCELED)\n",
+ exec_ret);
+ igt_ioctl(fd, DRM_IOCTL_XE_EXEC_QUEUE_DESTROY, &destroy);
+ } else {
+ xe_exec(fd, &exec);
+ xe_exec_queue_destroy(fd, exec.exec_queue_id);
+ }
(*t->num_submit)++;
if ((t->flags & MEM_PRESSURE_STRESS) && !(i % 128)) {
@@ -829,8 +849,20 @@ static void submit_jobs(struct gt_thread_data *t)
pressure_bo_destroy(fd, pressure_bos, PRESSURE_COUNT);
munmap(data, bo_size);
- gem_close(fd, bo);
- xe_vm_destroy(fd, vm);
+ /*
+ * On a device wedged by injected GT reset failures, BO close and VM
+ * destroy also return -ECANCELED.
+ */
+ if (t->flags & FAULT_INJECT_STRESS) {
+ struct drm_gem_close close_bo = { .handle = bo };
+ struct drm_xe_vm_destroy vm_destroy = { .vm_id = vm };
+
+ igt_ioctl(fd, DRM_IOCTL_GEM_CLOSE, &close_bo);
+ igt_ioctl(fd, DRM_IOCTL_XE_VM_DESTROY, &vm_destroy);
+ } else {
+ gem_close(fd, bo);
+ xe_vm_destroy(fd, vm);
+ }
}
static void *gt_reset_thread(void *data)
@@ -850,6 +882,33 @@ static void *gt_reset_thread(void *data)
return NULL;
}
+static void gt_reset_fault_injection(int fd, bool enable)
+{
+ igt_debugfs_write(fd, "fail_gt_reset/probability", enable ? "100" : "0");
+ igt_debugfs_write(fd, "fail_gt_reset/times", enable ? "2" : "1");
+}
+
+static int try_vm_create(int fd)
+{
+ struct drm_xe_vm_create create = { 0 };
+ int err = 0;
+
+ if (igt_ioctl(fd, DRM_IOCTL_XE_VM_CREATE, &create))
+ err = -errno;
+ else
+ xe_vm_destroy(fd, create.vm_id);
+
+ return err;
+}
+
+static void ignore_gt_reset_fault_dmesg(void)
+{
+ igt_emit_ignore_dmesg_regex("reset failed \\(-ECANCELED\\)"
+ "|declared device .* as wedged"
+ "|GPU HANG"
+ "|Failed to reset");
+}
+
/**
* SUBTEST: gt-reset-stress
* Description: Stress GT reset
@@ -891,6 +950,9 @@ static void *gt_reset_thread(void *data)
* Description: Test GT reset while long spinner workload is active
* Test category: stress test
*
+ * SUBTEST: gt-reset-fault-injection
+ * Description: Stress concurrent GT resets and job submissions with GT reset failures injected via debugfs
+ * Test category: fault injection
*/
static void
gt_reset(int fd, int gt, int n_threads, int n_sec, unsigned int flags)
@@ -907,6 +969,9 @@ gt_reset(int fd, int gt, int n_threads, int n_sec, unsigned int flags)
pthread_mutex_init(&mutex, 0);
pthread_cond_init(&cond, 0);
+ if (flags & FAULT_INJECT_STRESS)
+ gt_reset_fault_injection(fd, true);
+
for (i = 0; i < n_threads; ++i) {
threads[i].mutex = &mutex;
threads[i].cond = &cond;
@@ -940,6 +1005,9 @@ gt_reset(int fd, int gt, int n_threads, int n_sec, unsigned int flags)
igt_info("number of resets %d, submissions %d, submit fails %d vm_recreate %d\n",
num_reset, num_submit, num_submit_fail, num_vm_recreate);
+ if (flags & FAULT_INJECT_STRESS)
+ gt_reset_fault_injection(fd, true);
+
igt_assert_neq(num_reset, 0);
igt_assert_neq(num_submit, 0);
free(threads);
@@ -1122,6 +1190,7 @@ int igt_main()
int gt;
int class;
int fd;
+ char pci_slot[NAME_MAX];
igt_fixture()
fd = drm_open_driver(DRIVER_XE);
@@ -1326,6 +1395,29 @@ int igt_main()
break;
}
+ igt_subtest("gt-reset-fault-injection") {
+ igt_require_f(igt_debugfs_exists(fd, "fail_gt_reset/probability",
+ O_RDWR),
+ "GT reset fault injection not available; "
+ "CONFIG_DRM_XE_KUNIT_TEST/fault-injection must be "
+ "enabled in the KMD\n");
+
+ igt_device_get_pci_slot_name(fd, pci_slot);
+ ignore_gt_reset_fault_dmesg();
+
+ gt_reset(fd, 0, 8, 2, FAULT_INJECT_STRESS);
+
+ igt_assert_f(try_vm_create(fd) != 0,
+ "Device did not wedge after injected GT reset failure\n");
+
+ drm_close_driver(fd);
+ igt_kmod_rebind("xe", pci_slot);
+ fd = drm_open_driver(DRIVER_XE);
+
+ igt_assert_f(try_vm_create(fd) == 0,
+ "Device not functional after rebind recovery\n");
+ }
+
igt_subtest("gt-mocs-reset")
xe_for_each_gt(fd, gt)
gt_mocs_reset(fd, gt);
@@ -1455,6 +1547,9 @@ int igt_main()
}
}
- igt_fixture()
+ igt_fixture() {
+ if (igt_debugfs_exists(fd, "fail_gt_reset/probability", O_RDWR))
+ gt_reset_fault_injection(fd, false);
drm_close_driver(fd);
+ }
}
--
2.34.1
next reply other threads:[~2026-08-25 3:26 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-25 3:25 nishit.sharma [this message]
2026-08-25 4:03 ` ✓ Xe.CI.BAT: success for tests/intel/xe_exec_reset: add GT reset fault injection stress coverage (rev2) Patchwork
2026-08-25 4:10 ` ✓ i915.CI.BAT: " Patchwork
2026-08-25 7:23 ` ✓ Xe.CI.FULL: " Patchwork
2026-08-25 7:30 ` ✗ i915.CI.Full: failure " Patchwork
2026-08-31 17:20 ` [PATCH] tests/intel/xe_exec_reset: add GT reset fault injection stress coverage Kamil Konieczny
-- strict thread matches above, loose matches on Subject: below --
2026-09-02 3:33 nishit.sharma
2026-09-02 10:12 ` Kamil Konieczny
2026-09-01 14:01 nishit.sharma
2026-09-01 18:13 ` Kamil Konieczny
2026-09-01 3:44 nishit.sharma
2026-09-01 12:48 ` Kamil Konieczny
2026-08-19 7:36 nishit.sharma
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260825032520.507695-1-nishit.sharma@intel.com \
--to=nishit.sharma@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=kamil.konieczny@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.