From: Lukasz Laguna <lukasz.laguna@intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [igt-dev] [PATCH i-g-t v6 5/8] tests/sriov_basic: add basic tests for enabling SR-IOV VFs
Date: Mon, 4 Dec 2023 18:10:30 +0100 [thread overview]
Message-ID: <20231204171033.2167-6-lukasz.laguna@intel.com> (raw)
In-Reply-To: <20231204171033.2167-1-lukasz.laguna@intel.com>
From: Katarzyna Dec <katarzyna.dec@intel.com>
Add subtests that validate SR-IOV VFs enabling in two variants: with
autoprobe disabled and enabled.
v2:
- rename function (Michal)
- remove checks that are outside the scope of the test (Michal)
v3:
- change run type of enable-vfs-autoprobe-on subtest (Michal)
- don't init random seed in test code (Michal)
Cc: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Signed-off-by: Lukasz Laguna <lukasz.laguna@intel.com>
---
tests/meson.build | 1 +
tests/sriov_basic.c | 123 ++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 124 insertions(+)
create mode 100644 tests/sriov_basic.c
diff --git a/tests/meson.build b/tests/meson.build
index facf60ccf..a6673c511 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -71,6 +71,7 @@ test_progs = [
'panfrost_submit',
'prime_udl',
'prime_vgem',
+ 'sriov_basic',
'syncobj_basic',
'syncobj_eventfd',
'syncobj_wait',
diff --git a/tests/sriov_basic.c b/tests/sriov_basic.c
new file mode 100644
index 000000000..582e644f2
--- /dev/null
+++ b/tests/sriov_basic.c
@@ -0,0 +1,123 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright(c) 2023 Intel Corporation. All rights reserved.
+ */
+
+#include "drmtest.h"
+#include "igt_core.h"
+#include "igt_sriov_device.h"
+
+IGT_TEST_DESCRIPTION("Basic tests for enabling SR-IOV Virtual Functions");
+
+/**
+ * TEST: sriov_basic
+ * Category: Software building block
+ * Mega feature: SR-IOV
+ * Sub-category: VFs enabling
+ * Description: Validate SR-IOV VFs enabling
+ */
+
+/**
+ * SUBTEST: enable-vfs-autoprobe-off
+ * Description:
+ * Verify VFs enabling without probing VF driver
+ * Run type: BAT
+ */
+static void enable_vfs_autoprobe_off(int pf_fd, unsigned int num_vfs)
+{
+ igt_debug("Testing %u VFs\n", num_vfs);
+
+ igt_require(igt_sriov_get_enabled_vfs(pf_fd) == 0);
+ igt_sriov_disable_driver_autoprobe(pf_fd);
+ igt_sriov_enable_vfs(pf_fd, num_vfs);
+ igt_assert_eq(num_vfs, igt_sriov_get_enabled_vfs(pf_fd));
+ igt_sriov_disable_vfs(pf_fd);
+}
+
+/**
+ * SUBTEST: enable-vfs-autoprobe-on
+ * Description:
+ * Verify VFs enabling and auto-probing VF driver
+ * Run type: FULL
+ */
+static void enable_vfs_autoprobe_on(int pf_fd, unsigned int num_vfs)
+{
+ bool err = false;
+
+ igt_debug("Testing %u VFs\n", num_vfs);
+
+ igt_require(igt_sriov_get_enabled_vfs(pf_fd) == 0);
+ igt_sriov_enable_driver_autoprobe(pf_fd);
+ igt_sriov_enable_vfs(pf_fd, num_vfs);
+ igt_assert_eq(num_vfs, igt_sriov_get_enabled_vfs(pf_fd));
+ for (int vf_num = 1; vf_num <= num_vfs; ++vf_num) {
+ if (!igt_sriov_is_vf_drm_driver_probed(pf_fd, vf_num)) {
+ igt_debug("VF%u probe failed\n", vf_num);
+ err = true;
+ }
+ }
+ igt_sriov_disable_vfs(pf_fd);
+ igt_assert(!err);
+}
+
+igt_main
+{
+ int pf_fd;
+ bool autoprobe;
+
+ igt_fixture {
+ pf_fd = drm_open_driver(DRIVER_ANY);
+ igt_require(igt_sriov_is_pf(pf_fd));
+ igt_require(igt_sriov_get_enabled_vfs(pf_fd) == 0);
+ autoprobe = igt_sriov_is_driver_autoprobe_enabled(pf_fd);
+ }
+
+ igt_describe("Verify VFs enabling without probing VF driver");
+ igt_subtest_with_dynamic("enable-vfs-autoprobe-off") {
+ for_each_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-%u", num_vfs) {
+ enable_vfs_autoprobe_off(pf_fd, num_vfs);
+ }
+ }
+ for_random_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-random") {
+ enable_vfs_autoprobe_off(pf_fd, num_vfs);
+ }
+ }
+ for_max_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-all") {
+ enable_vfs_autoprobe_off(pf_fd, num_vfs);
+ }
+ }
+ }
+
+ igt_describe("Verify VFs enabling and auto-probing VF driver");
+ igt_subtest_with_dynamic("enable-vfs-autoprobe-on") {
+ for_each_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-%u", num_vfs) {
+ enable_vfs_autoprobe_on(pf_fd, num_vfs);
+ }
+ }
+ for_random_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-random") {
+ enable_vfs_autoprobe_on(pf_fd, num_vfs);
+ }
+ }
+ for_max_sriov_num_vfs(pf_fd, num_vfs) {
+ igt_dynamic_f("numvfs-all") {
+ enable_vfs_autoprobe_on(pf_fd, num_vfs);
+ }
+ }
+ }
+
+ igt_fixture {
+ igt_sriov_disable_vfs(pf_fd);
+ /* abort to avoid execution of next tests with enabled VFs */
+ igt_abort_on_f(igt_sriov_get_enabled_vfs(pf_fd) > 0, "Failed to disable VF(s)");
+ autoprobe ? igt_sriov_enable_driver_autoprobe(pf_fd) :
+ igt_sriov_disable_driver_autoprobe(pf_fd);
+ igt_abort_on_f(autoprobe != igt_sriov_is_driver_autoprobe_enabled(pf_fd),
+ "Failed to restore sriov_drivers_autoprobe value\n");
+ close(pf_fd);
+ }
+}
--
2.40.0
next prev parent reply other threads:[~2023-12-04 17:16 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-04 17:10 [igt-dev] [PATCH i-g-t v6 0/8] Initial SR-IOV validation Lukasz Laguna
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 1/8] lib/igt_sriov_device: add core SR-IOV helpers Lukasz Laguna
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 2/8] lib/igt_sriov_device: add helper for opening VF device Lukasz Laguna
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 3/8] lib/igt_sriov_device: add helper for checking if VF DRM driver is probed Lukasz Laguna
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 4/8] lib/igt_sriov_device: add helpers for operations in different VFs scenarios Lukasz Laguna
2023-12-04 17:10 ` Lukasz Laguna [this message]
2023-12-04 20:50 ` [igt-dev] [PATCH i-g-t v6 5/8] tests/sriov_basic: add basic tests for enabling SR-IOV VFs Michal Wajdeczko
2023-12-05 8:04 ` Laguna, Lukasz
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 6/8] lib/igt_sriov_device: add helpers for VF DRM driver bind and unbind Lukasz Laguna
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 7/8] tests/sriov_basic: validate driver binding to VFs Lukasz Laguna
2023-12-04 20:47 ` Michal Wajdeczko
2023-12-04 17:10 ` [igt-dev] [PATCH i-g-t v6 8/8] tests/sriov_basic: add more tests for VF driver binding Lukasz Laguna
2023-12-04 18:15 ` [igt-dev] ✗ Fi.CI.BAT: failure for Initial SR-IOV validation (rev7) Patchwork
2023-12-04 19:00 ` [igt-dev] ✓ CI.xeBAT: success " 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=20231204171033.2167-6-lukasz.laguna@intel.com \
--to=lukasz.laguna@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