From: Matthew Brost <matthew.brost@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: intel-xe@lists.freedesktop.org
Subject: [PATCH v2 1/2] xe_exec_reset: Add job cancel tests
Date: Fri, 7 Jun 2024 17:12:10 -0700 [thread overview]
Message-ID: <20240608001211.2365347-2-matthew.brost@intel.com> (raw)
In-Reply-To: <20240608001211.2365347-1-matthew.brost@intel.com>
This test has pulled out when the ability to set the job timeout via
IOCTL was removed from the uAPI while upstreaming Xe. Add this coverage
back in but only run on a single hardware engine given by default jobs
take 5 seconds to timeout.
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
---
tests/intel/xe_exec_reset.c | 21 ++++++++++++++++++++-
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/tests/intel/xe_exec_reset.c b/tests/intel/xe_exec_reset.c
index 8d731e6893..5e9e48adda 100644
--- a/tests/intel/xe_exec_reset.c
+++ b/tests/intel/xe_exec_reset.c
@@ -99,6 +99,7 @@ static void test_spin(int fd, struct drm_xe_engine_class_instance *eci)
#define VIRTUAL (0x1 << 3)
#define PARALLEL (0x1 << 4)
#define CAT_ERROR (0x1 << 5)
+#define PREEMPT (0x1 << 6)
/**
* SUBTEST: %s-cat-error
@@ -280,6 +281,12 @@ test_balancer(int fd, int gt, int class, int n_exec_queues, int n_execs,
* SUBTEST: cat-error
* Description: Test cat error
*
+ * SUBTEST: cancel
+ * Description: Test job cancel
+ *
+ * SUBTEST: cancel-preempt
+ * Description: Test job cancel with a preemptable job
+ *
* SUBTEST: gt-reset
* Description: Test GT reset
*
@@ -318,7 +325,7 @@ test_legacy_mode(int fd, struct drm_xe_engine_class_instance *eci,
uint64_t pad;
uint32_t data;
} *data;
- struct xe_spin_opts spin_opts = { .preempt = false };
+ struct xe_spin_opts spin_opts = { .preempt = flags & PREEMPT };
int i, b;
igt_assert(n_exec_queues <= MAX_N_EXECQUEUES);
@@ -727,6 +734,18 @@ igt_main
xe_for_each_engine(fd, hwe)
test_legacy_mode(fd, hwe, 2, 2, CAT_ERROR);
+ igt_subtest("cancel")
+ xe_for_each_engine(fd, hwe) {
+ test_legacy_mode(fd, hwe, 1, 1, 0);
+ break;
+ }
+
+ igt_subtest("cancel-preempt")
+ xe_for_each_engine(fd, hwe) {
+ test_legacy_mode(fd, hwe, 1, 1, PREEMPT);
+ break;
+ }
+
igt_subtest("gt-reset")
xe_for_each_engine(fd, hwe)
test_legacy_mode(fd, hwe, 2, 2, GT_RESET);
--
2.34.1
next prev parent reply other threads:[~2024-06-08 0:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-08 0:12 [PATCH v2 0/2] Job cancel tests Matthew Brost
2024-06-08 0:12 ` Matthew Brost [this message]
2024-06-08 0:12 ` [PATCH v2 2/2] xe_exec_reset: Add timeslice preempt test Matthew Brost
2024-06-14 13:53 ` Kamil Konieczny
2024-06-08 0:16 ` ✗ CI.Patch_applied: failure for Job cancel tests Patchwork
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=20240608001211.2365347-2-matthew.brost@intel.com \
--to=matthew.brost@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox