[PATCH i-g-t v2 05/10] tests/intel/xe_drm_fdinfo: Add single engine tests
Umesh Nerlige Ramappa
umesh.nerlige.ramappa at intel.com
Wed Jul 3 00:25:27 UTC 2024
Add simple tests that submit work to one engine and measure utilization
per class.
v2:
- Drop measured_usleep since return value is not used
- s/parallel engines/parallel submission/ in comment
- Use NSEC_PER_SEC for batch_duration_ns
- Percent should not be > 100
- Check utilization for both clients for isolation case
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
---
tests/intel/xe_drm_fdinfo.c | 97 +++++++++++++++++++++++++++++++++++++
1 file changed, 97 insertions(+)
diff --git a/tests/intel/xe_drm_fdinfo.c b/tests/intel/xe_drm_fdinfo.c
index 037f25e53..410c885e7 100644
--- a/tests/intel/xe_drm_fdinfo.c
+++ b/tests/intel/xe_drm_fdinfo.c
@@ -28,6 +28,15 @@
* SUBTEST: basic-engine-utilization
* Description: Check if basic fdinfo content is present for engine utilization
*
+ * SUBTEST: drm-idle
+ * Description: Check that engines show no load when idle
+ *
+ * SUBTEST: drm-busy-idle
+ * Description: Check that engines show load when idle after busy
+ *
+ * SUBTEST: drm-busy-idle-isolation
+ * Description: Check that engine load does not spill over to other drm clients
+ *
* SUBTEST: drm-total-resident
* Description: Create and compare total and resident memory consumption by client
*
@@ -42,11 +51,18 @@ IGT_TEST_DESCRIPTION("Read and verify drm client memory consumption and engine u
#define BO_SIZE (65536)
+/* flag masks */
+#define TEST_BUSY (1 << 0)
+#define TEST_TRAILING_IDLE (1 << 1)
+#define TEST_ISOLATION (1 << 2)
+
struct pceu_cycles {
uint64_t cycles;
uint64_t total_cycles;
};
+const unsigned long batch_duration_ns = (1 * NSEC_PER_SEC) / 2;
+
static const char *engine_map[] = {
"rcs",
"bcs",
@@ -464,8 +480,77 @@ spin_ctx_destroy(int fd, struct spin_ctx *ctx)
free(ctx);
}
+static void
+check_results(struct pceu_cycles *s1, struct pceu_cycles *s2,
+ int class, int width, unsigned int flags)
+{
+ double percent;
+
+ igt_debug("%s: sample 1: cycles %lu, total_cycles %lu\n",
+ engine_map[class], s1[class].cycles, s1[class].total_cycles);
+ igt_debug("%s: sample 2: cycles %lu, total_cycles %lu\n",
+ engine_map[class], s2[class].cycles, s2[class].total_cycles);
+
+ percent = ((s2[class].cycles - s1[class].cycles) * 100) /
+ ((s2[class].total_cycles + 1) - s1[class].total_cycles);
+
+ /* for parallel submission scale the busyness with width */
+ percent = percent / width;
+
+ igt_debug("%s: percent: %f\n", engine_map[class], percent);
+
+ if (flags & TEST_BUSY)
+ igt_assert(percent >= 95 && percent <= 100);
+ else
+ igt_assert(!percent);
+}
+
+static void
+single(int fd, struct drm_xe_engine_class_instance *hwe, int width, int count,
+ unsigned int flags)
+{
+ struct pceu_cycles pceu1[2][DRM_XE_ENGINE_CLASS_COMPUTE + 1];
+ struct pceu_cycles pceu2[2][DRM_XE_ENGINE_CLASS_COMPUTE + 1];
+ struct spin_ctx *ctx = NULL;
+ uint32_t vm;
+ int new_fd;
+
+ if (flags & TEST_ISOLATION)
+ new_fd = drm_reopen_driver(fd);
+
+ vm = xe_vm_create(fd, 0, 0);
+ if (flags & TEST_BUSY) {
+ ctx = spin_ctx_init(fd, hwe, vm, width, count);
+ spin_sync_start(fd, ctx);
+ }
+
+ read_engine_cycles(fd, pceu1[0]);
+ if (flags & TEST_ISOLATION)
+ read_engine_cycles(new_fd, pceu1[1]);
+
+ usleep(batch_duration_ns / 1000);
+ if (flags & TEST_TRAILING_IDLE)
+ spin_sync_end(fd, ctx);
+
+ read_engine_cycles(fd, pceu2[0]);
+ if (flags & TEST_ISOLATION)
+ read_engine_cycles(new_fd, pceu2[1]);
+
+ check_results(pceu1[0], pceu2[0], hwe->engine_class, width, flags);
+
+ if (flags & TEST_ISOLATION) {
+ check_results(pceu1[1], pceu2[1], hwe->engine_class, width, 0);
+ close(new_fd);
+ }
+
+ spin_sync_end(fd, ctx);
+ spin_ctx_destroy(fd, ctx);
+ xe_vm_destroy(fd, vm);
+}
+
igt_main
{
+ struct drm_xe_engine_class_instance *hwe;
int xe;
igt_fixture {
@@ -484,6 +569,18 @@ igt_main
igt_subtest("basic-engine-utilization")
basic_engine_utilization(xe);
+ igt_subtest("drm-idle")
+ xe_for_each_engine(xe, hwe)
+ single(xe, hwe, 1, 1, 0);
+
+ igt_subtest("drm-busy-idle")
+ xe_for_each_engine(xe, hwe)
+ single(xe, hwe, 1, 1, TEST_BUSY | TEST_TRAILING_IDLE);
+
+ igt_subtest("drm-busy-idle-isolation")
+ xe_for_each_engine(xe, hwe)
+ single(xe, hwe, 1, 1, TEST_BUSY | TEST_TRAILING_IDLE | TEST_ISOLATION);
+
igt_describe("Create and compare total and resident memory consumption by client");
igt_subtest("drm-total-resident")
test_total_resident(xe);
--
2.38.1
More information about the igt-dev
mailing list