Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [igt-dev] [PATCH i-g-t 8/8] xe/xe_eudebug: Add TEST/SUBTEST documentation
Date: Tue, 16 May 2023 17:44:34 +0200	[thread overview]
Message-ID: <20230516154434.810356-9-dominik.grzegorzek@intel.com> (raw)
In-Reply-To: <20230516154434.810356-1-dominik.grzegorzek@intel.com>

Document xe_eudebug test, to reflect its internal logic.

Signed-off-by: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
---
 tests/xe/xe_eudebug.c | 49 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 49 insertions(+)

diff --git a/tests/xe/xe_eudebug.c b/tests/xe/xe_eudebug.c
index 92eaab7de..2aca01b10 100644
--- a/tests/xe/xe_eudebug.c
+++ b/tests/xe/xe_eudebug.c
@@ -2,6 +2,15 @@
 /*
  * Copyright © 2023 Intel Corporation
  */
+
+/**
+ * TEST: Test EU Debugger functionality
+ * Category: Software building block
+ * Sub-category: eu debugger
+ * Test category: functionality test
+ * Run type: BAT
+ */
+
 #include <poll.h>
 
 #include <xe_drm_tmp.h>
@@ -123,6 +132,12 @@ static int __debug_connect(int fd, int *debugfd, struct drm_xe_eudebug_connect_p
 	return ret;
 }
 
+/**
+ * SUBTEST: basic-connect
+ * Description:
+ * 	Exercise XE_EUDEBG_CONNECT ioctl with passing
+ *	valid and invalid params.
+ */
 static void test_connect(int fd)
 {
 	struct drm_xe_eudebug_connect_param param = {};
@@ -193,6 +208,11 @@ static void test_connect(int fd)
 	close(debugfd);
 }
 
+/**
+ * SUBTEST: basic-close
+ * Description:
+ * 	Test whether eudebug can be reattached after closure.
+ */
 static void test_close(int fd)
 {
 	struct drm_xe_eudebug_connect_param param = { 0,  };
@@ -216,6 +236,11 @@ static void test_close(int fd)
 	close(debug_fd1);
 }
 
+/**
+ * SUBTEST: basic-read-event
+ * Description:
+ * 	Synchronously exercise eu debugger event polling and reading.
+ */
 #define MAX_EVENT_SIZE (32 * 1024)
 static void test_read_event(int fd)
 {
@@ -265,6 +290,20 @@ static void test_read_event(int fd)
 	xe_eudebug_client_destroy(c);
 }
 
+/**
+ * SUBTEST: basic-client
+ * Description:
+ * 	Attach the debugger to proccess which opens and closes xe drm client.
+ *
+ * SUBTEST: multiple-sessions
+ * Description:
+ * 	Simultaneusly attach many debuggers to many proccesses.
+ * 	Each proccess opens and closes xe drm client.
+ *
+ * SUBTEST: basic-vms
+ * Description:
+ * 	Attach the debugger to procces which creates and destroys a few vms.
+ */
 static void test_basic_sessions(int fd, unsigned int flags, int count)
 {
 	struct session **s;
@@ -320,6 +359,16 @@ static void run_discovery_client(struct xe_eudebug_client *c)
 	}
 }
 
+/**
+ * SUBTEST: discovery-%s
+ * Description: Race discovery against %arg[1] and the debugger dettach.
+ *
+ * arg[1]:
+ *
+ * @race:		resources creation
+ * @empty:		resources destruction
+ * @empty-clients:	client closure
+ */
 static void *discovery_race_thread(void *data)
 {
 	struct {
-- 
2.34.1

  parent reply	other threads:[~2023-05-16 15:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-16 15:44 [igt-dev] [RFC i-g-t 0/8] Add initial eudebug coverage Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 1/8] xe: sync uapi headers Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 2/8] xe/xe_eudebug: test eudebug connection Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 3/8] xe/xe_eudebug: introduce eu debug testing framework Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 4/8] xe/xe_eudebug: test open close events Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 5/8] xe/xe_eudebug: exercise read_event ioctl Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 6/8] xe/xe_eudebug: add vm events sanity check Dominik Grzegorzek
2023-05-16 15:44 ` [igt-dev] [PATCH i-g-t 7/8] xe/xe_eudebug: Race discovery against eudebug attach Dominik Grzegorzek
2023-05-16 15:44 ` Dominik Grzegorzek [this message]
2023-05-16 17:17 ` [igt-dev] ✗ GitLab.Pipeline: warning for Add initial eudebug coverage Patchwork
2023-05-16 17:41 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2023-05-17  2:11 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2023-05-17 18:56 ` [igt-dev] ✓ Fi.CI.BAT: success for Add initial eudebug coverage (rev2) Patchwork
2023-05-18  8:16 ` [igt-dev] ✓ Fi.CI.IGT: " 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=20230516154434.810356-9-dominik.grzegorzek@intel.com \
    --to=dominik.grzegorzek@intel.com \
    --cc=igt-dev@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