[PATCH i-g-t v12 3/6] lib/igt_multigpu: Introduce library for multi-GPU scenarios
Kamil Konieczny
kamil.konieczny at linux.intel.com
Wed Mar 13 19:29:42 UTC 2024
From: Dominik Karol Piątkowski <dominik.karol.piatkowski at intel.com>
Implemented igt_require_multigpu in order to replace
igt_require(gpu_count > 1), as well as printing available
PCI devices if requirement fails.
Introduced igt_multigpu_count_class function that returns an actual
number of GPUs present in system, which allows for writing multi-GPU
test scenarios that does not require filter
--device pci:vendor=intel,device=discrete,card=all
to run as intended. Based on patch by Chris Wilson.
Introduced igt_multi_fork_foreach_gpu macro that helps with
writing multi-GPU test scenarios in idiomatic form:
igt_multi_fork_foreach_gpu(i915, DRIVER_INTEL)
test_function(i915);
igt_waitchildren();
v10: squashed two commits which introduce multigpu functions (Zbigniew)
renamed __id and id__ into __chipset/chipset__ (Zbigniew)
used __drm_close_driver() instead of close() in first macro (Kamil)
v11: added functions descriptions (Janusz)
v12: moved to igt_multigpu.c/h (Janusz)
changed names from gem_ to igt_, removed '\' from end of macro (Janusz)
Cc: "Zbigniew Kempczyński" <zbigniew.kempczynski at intel.com>
Cc: Janusz Krzysztofik <janusz.krzysztofik at linux.intel.com>
Signed-off-by: "Dominik Karol Piątkowski" <dominik.karol.piatkowski at intel.com>
[Kamil: fixed whitespace and tabs, moved to lib/igt_multigpu.*]
Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
---
lib/igt_multigpu.c | 86 ++++++++++++++++++++++++++++++++++++++++++++++
lib/igt_multigpu.h | 28 +++++++++++++++
lib/meson.build | 1 +
3 files changed, 115 insertions(+)
create mode 100644 lib/igt_multigpu.c
create mode 100644 lib/igt_multigpu.h
diff --git a/lib/igt_multigpu.c b/lib/igt_multigpu.c
new file mode 100644
index 000000000..9750ffda4
--- /dev/null
+++ b/lib/igt_multigpu.c
@@ -0,0 +1,86 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2023 Intel Corporation
+ */
+
+#include "drmtest.h"
+#include "i915/gem.h"
+#include "igt_core.h"
+#include "igt_device_scan.h"
+#include "igt_multigpu.h"
+
+/**
+ * igt_multigpu_count_class:
+ * @class: chipset, e.g. DRIVER_XE or DRIVER_INTEL
+ *
+ * Function counts number of GPU cards with the help of opening all of them.
+ *
+ * Returns: number of GPUs cards found
+ */
+int igt_multigpu_count_class(int class)
+{
+ int count = 0;
+
+ igt_foreach_gpu(fd, class)
+ count++;
+
+ return count;
+}
+
+static int print_gpus(int count, int gpu_num)
+{
+ struct igt_devices_print_format fmt = {
+ .type = IGT_PRINT_SIMPLE,
+ .option = IGT_PRINT_PCI,
+ };
+ int devices;
+
+ igt_info("PCI devices available in the system:\n");
+
+ igt_devices_scan(true);
+ devices = igt_device_filter_pci();
+ igt_devices_print(&fmt);
+
+ return devices;
+}
+
+/**
+ * igt_require_multigpu:
+ * @count: minimum number of GPUs required found with filters
+ *
+ * Function checks number of filtered GPU cards.
+ * On error prints available GPUs found on PCI bus and skips.
+ */
+void igt_require_multigpu(int gpus_wanted)
+{
+ int gpu_filters = igt_device_filter_count();
+ int num;
+
+ if (gpu_filters >= gpus_wanted)
+ return;
+
+ num = print_gpus(gpus_wanted, gpu_filters);
+ igt_skip_on_f(gpu_filters < gpus_wanted, "Test requires at least %d GPUs, got %d, available: %d\n", gpus_wanted, gpu_filters, num);
+}
+
+/**
+ * multigpu_open_another:
+ * @idx: index of GPU card, starting from 0
+ * @chipset: chipset, e.g. DRIVER_XE or DRIVER_INTEL
+ *
+ * Function opens GPU card with drm_open_driver_another(). For i915 it checks
+ * if opened card is functional. Skips on errors.
+ *
+ * Returns: opened fd for @idx card
+ */
+int multigpu_open_another(int idx, unsigned int chipset)
+{
+ int fd;
+
+ fd = drm_open_driver_another(idx, chipset);
+
+ if (chipset == DRIVER_INTEL)
+ igt_require_gem(fd);
+
+ return fd;
+}
diff --git a/lib/igt_multigpu.h b/lib/igt_multigpu.h
new file mode 100644
index 000000000..874b020c2
--- /dev/null
+++ b/lib/igt_multigpu.h
@@ -0,0 +1,28 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Copyright © 2023 Intel Corporation
+ */
+
+#ifndef __INTEL_MULTIGPU_H
+#define __INTEL_MULTIGPU_H
+
+#include "drmtest.h"
+#include "igt_core.h"
+
+void igt_require_multigpu(int count);
+int igt_multigpu_count_class(int chipset);
+
+int multigpu_open_another(int idx, unsigned int chipset);
+
+#define igt_foreach_gpu(fd__, chipset__) \
+ for (int igt_unique(i) = 0, fd__; \
+ (fd__ = __drm_open_driver_another(igt_unique(i)++, chipset__)) >= 0; \
+ __drm_close_driver(fd__))
+
+#define igt_multi_fork_foreach_gpu(__fd, __chipset) \
+ igt_multi_fork(igt_unique(__i), igt_multigpu_count_class(__chipset)) \
+ for (int __fd = multigpu_open_another(igt_unique(__i), __chipset); \
+ __fd >= 0; \
+ drm_close_driver(__fd), __fd = -1)
+
+#endif /* __INTEL_MULTIGPU_H */
diff --git a/lib/meson.build b/lib/meson.build
index 6122861d8..defc723ac 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -65,6 +65,7 @@ lib_sources = [
'intel_device_info.c',
'intel_mmio.c',
'intel_mocs.c',
+ 'igt_multigpu.c',
'intel_pat.c',
'ioctl_wrappers.c',
'media_spin.c',
--
2.42.0
More information about the igt-dev
mailing list