<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<div class="moz-cite-prefix">On 5/8/2024 21:32, Marcin Bernatowicz
wrote:<br>
</div>
<blockquote type="cite" cite="mid:20240508193258.4069853-1-marcin.bernatowicz@linux.intel.com">
<pre class="moz-quote-pre" wrap="">There is no point to run the tests if no VFs can be enabled.
Additionally, the `for_random_sriov_num_vfs` macro will lead
to undefined behavior if `sriov_totalvfs` is zero.
Signed-off-by: Marcin Bernatowicz <a class="moz-txt-link-rfc2396E" href="mailto:marcin.bernatowicz@linux.intel.com"><marcin.bernatowicz@linux.intel.com></a>
---
tests/sriov_basic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/sriov_basic.c b/tests/sriov_basic.c
index f58b3b7ba..1bc082d76 100644
--- a/tests/sriov_basic.c
+++ b/tests/sriov_basic.c
@@ -120,7 +120,7 @@ igt_main
igt_fixture {
pf_fd = drm_open_driver(DRIVER_ANY);
- igt_require(igt_sriov_is_pf(pf_fd));
+ igt_require(igt_sriov_vfs_supported(pf_fd));
igt_require(igt_sriov_get_enabled_vfs(pf_fd) == 0);
autoprobe = igt_sriov_is_driver_autoprobe_enabled(pf_fd);
}</pre>
</blockquote>
<p><span style="white-space: pre-wrap">LGTM,
</span></p>
<p><span lang="en-US">Reviewed-by: </span><span lang="pl">L</span><span lang="en-US">ukasz </span><span lang="pl">L</span><span lang="en-US">aguna <</span><a href="mailto:lukasz.laguna@intel.com"><span lang="en-US">lukasz.laguna@intel.com</span></a><span lang="en-US">></span><span style="white-space: pre-wrap">
</span></p>
</body>
</html>