From: "Dominik Karol Piątkowski" <dominik.karol.piatkowski@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: "Zbigniew Kempczyński" <zbigniew.kempczynski@intel.com>,
"Dominik Grzegorzek" <dominik.grzegorzek@intel.com>,
"Christoph Manszewski" <christoph.manszewski@intel.com>,
"Pawel Sikora" <pawel.sikora@intel.com>,
"Jonathan Cavitt" <jonathan.cavitt@intel.com>,
"Dominik Karol Piątkowski" <dominik.karol.piatkowski@intel.com>
Subject: [PATCH i-g-t 2/3] tests/intel/xe_eudebug: Add basic-vm-bind-drop-debugger-before-ufence-ack test
Date: Mon, 9 Dec 2024 15:13:58 +0100 [thread overview]
Message-ID: <20241209141359.5738-3-dominik.karol.piatkowski@intel.com> (raw)
In-Reply-To: <20241209141359.5738-1-dominik.karol.piatkowski@intel.com>
Add a test that gives user fence in application, holds it, drops the
debugger connection and checks if anything breaks. It is expected that
held acks are released when connection is dropped.
Signed-off-by: Dominik Karol Piątkowski <dominik.karol.piatkowski@intel.com>
---
tests/intel/xe_eudebug.c | 50 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 50 insertions(+)
diff --git a/tests/intel/xe_eudebug.c b/tests/intel/xe_eudebug.c
index 1853dee40..0a4bebf1e 100644
--- a/tests/intel/xe_eudebug.c
+++ b/tests/intel/xe_eudebug.c
@@ -2171,6 +2171,53 @@ static void test_basic_ufence(int fd, unsigned int flags)
ufence_priv_destroy(priv);
}
+/**
+ * SUBTEST: basic-vm-bind-drop-debugger-before-ufence-ack
+ * Description:
+ * Give user fence in application, hold it, drop the debugger connection and check if anything
+ * breaks. Expect that held acks are released when connection is dropped.
+ */
+static void test_ufence_drop_debugger_before_ack(int fd)
+{
+ struct xe_eudebug_debugger *d;
+ struct xe_eudebug_session *s;
+ struct xe_eudebug_client *c;
+ struct ufence_priv *priv;
+
+ priv = ufence_priv_create();
+ s = xe_eudebug_session_create(fd, basic_ufence_client, 0, priv);
+ c = s->client;
+ d = s->debugger;
+
+ xe_eudebug_debugger_add_trigger(d,
+ DRM_XE_EUDEBUG_EVENT_VM_BIND_UFENCE,
+ basic_ufence_trigger);
+
+ igt_assert_eq(xe_eudebug_debugger_attach(d, c), 0);
+ xe_eudebug_debugger_start_worker(d);
+ xe_eudebug_client_start(c);
+
+ xe_eudebug_debugger_wait_stage(s, STAGE_CLIENT_WAIT_ON_UFENCE_DONE);
+ xe_eudebug_assert_f(d, wait_for_ufence_events(priv, XE_EUDEBUG_DEFAULT_TIMEOUT_SEC * MSEC_PER_SEC) == 0,
+ "missing ufence events\n");
+
+ xe_eudebug_debugger_detach(d);
+ sleep(1);
+ igt_assert_eq(xe_eudebug_debugger_attach(d, c), 0);
+
+ xe_eudebug_client_wait_done(c);
+ xe_eudebug_debugger_stop_worker(d, 1);
+
+ xe_eudebug_event_log_print(d->log, true);
+ xe_eudebug_event_log_print(c->log, true);
+
+ xe_eudebug_session_check(s, true, XE_EUDEBUG_FILTER_EVENT_VM_BIND |
+ XE_EUDEBUG_FILTER_EVENT_VM | XE_EUDEBUG_FILTER_EVENT_OPEN);
+
+ xe_eudebug_session_destroy(s);
+ ufence_priv_destroy(priv);
+}
+
struct vm_bind_clear_thread_priv {
struct drm_xe_engine_class_instance *hwe;
struct xe_eudebug_client *c;
@@ -2825,6 +2872,9 @@ igt_main
igt_subtest("basic-vm-bind-ufence-delay-ack")
test_basic_ufence(fd, VM_BIND_DELAY_UFENCE_ACK);
+ igt_subtest("basic-vm-bind-drop-debugger-before-ufence-ack")
+ test_ufence_drop_debugger_before_ack(fd);
+
igt_subtest("vma-ufence")
test_vma_ufence(fd, 0);
--
2.34.1
next prev parent reply other threads:[~2024-12-09 14:14 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-09 14:13 [PATCH i-g-t 0/3] tests/intel/xe_eudebug: Introduce additional ufence testcases Dominik Karol Piątkowski
2024-12-09 14:13 ` [PATCH i-g-t 1/3] tests/intel/xe_eudebug: Add basic-vm-bind-ufence-delay-ack test Dominik Karol Piątkowski
2024-12-10 18:10 ` Manszewski, Christoph
2024-12-09 14:13 ` Dominik Karol Piątkowski [this message]
2024-12-10 18:22 ` [PATCH i-g-t 2/3] tests/intel/xe_eudebug: Add basic-vm-bind-drop-debugger-before-ufence-ack test Manszewski, Christoph
2024-12-09 14:13 ` [PATCH i-g-t 3/3] tests/intel/xe_eudebug: Add basic-vm-bind-ufence-sigint-client test Dominik Karol Piątkowski
2024-12-10 18:38 ` Manszewski, Christoph
2024-12-09 21:52 ` ✓ Xe.CI.BAT: success for tests/intel/xe_eudebug: Introduce additional ufence testcases Patchwork
2024-12-09 21:55 ` ✓ i915.CI.BAT: " Patchwork
2024-12-09 22:59 ` ✗ Xe.CI.Full: failure " Patchwork
2024-12-11 8:09 ` Manszewski, Christoph
2024-12-10 0:23 ` ✗ i915.CI.Full: " 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=20241209141359.5738-3-dominik.karol.piatkowski@intel.com \
--to=dominik.karol.piatkowski@intel.com \
--cc=christoph.manszewski@intel.com \
--cc=dominik.grzegorzek@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=jonathan.cavitt@intel.com \
--cc=pawel.sikora@intel.com \
--cc=zbigniew.kempczynski@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox