From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 27161C56208 for ; Thu, 6 Aug 2026 10:04:49 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id CBC0910F15D; Thu, 6 Aug 2026 10:04:48 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="mplUAjLz"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0089910F191 for ; Thu, 6 Aug 2026 10:03:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1786010628; x=1817546628; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=Ue+mCjCeE0bwflK7pkhWpwqULhtLwv6uUjcuZP1Kne8=; b=mplUAjLzq0zHs34AaS1bH0gS08ZJWJtwQzau9p4GbwZQ+EgHcTrgAfkd oVMkwrLZO2ePNjuyKDw2tUJiFWyAX/OHHwMNiqxTn2yPev/qt0FEhX2KL V3VU6guNeNsDcmyxe5KH/Cj9ixUib4qHpudAQl5QgFY3Tt43PYDmQvwK9 hfvoUli6hK21ohKy8cICR6fAGHwj4oO4qwVnjsFfsQ14VbrLxBmsljN+b fBRPUzLIaVxM6K58mGdjLIA7ANkm22Z9vnJlv7rgbJaJfiOeqHp5I9GtY Z5c4b+V81Tf5MqKzGLkaObkjwS24lzwTETr+8hj9TyZB/jziZ/2fYz6tA g==; X-CSE-ConnectionGUID: oRVht1DcSZOFSpIi09CrfA== X-CSE-MsgGUID: yc4eFEAnSwC7HjSYq2R6IQ== X-IronPort-AV: E=McAfee;i="6800,10657,11866"; a="104136411" X-IronPort-AV: E=Sophos;i="6.25,208,1779174000"; d="scan'208";a="104136411" Received: from orviesa010.jf.intel.com ([10.64.159.150]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Aug 2026 03:03:48 -0700 X-CSE-ConnectionGUID: QGCcUO6+ROyRr1UAEwSxQQ== X-CSE-MsgGUID: I1FexxNOSke/CyppYCvnkw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.25,208,1779174000"; d="scan'208";a="260806161" Received: from dev-150.igk.intel.com (HELO localhost) ([10.91.214.40]) by orviesa010-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Aug 2026 03:03:47 -0700 From: Lukasz Laguna To: igt-dev@lists.freedesktop.org Cc: marcin.bernatowicz@intel.com Subject: [PATCH v4 2/2] tests/sriov_basic: Validate PF unbind with VFs enabled Date: Thu, 6 Aug 2026 12:03:32 +0200 Message-ID: <20260806100332.1228103-3-lukasz.laguna@intel.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260806100332.1228103-1-lukasz.laguna@intel.com> References: <20260806100332.1228103-1-lukasz.laguna@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" Add new subtests covering PF driver unbind with VFs enabled in two scenarios: - with all VFs enabled, - with one VF enabled and probed. The tests verify that PF ends up unbound and VFs are disabled. Signed-off-by: Lukasz Laguna --- v4: - Fix unspecified values of local variables after longjmp (Marcin) --- tests/sriov_basic.c | 98 ++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 96 insertions(+), 2 deletions(-) diff --git a/tests/sriov_basic.c b/tests/sriov_basic.c index 1e563cee9..050d209b1 100644 --- a/tests/sriov_basic.c +++ b/tests/sriov_basic.c @@ -3,8 +3,13 @@ * Copyright(c) 2023 Intel Corporation. All rights reserved. */ +#include + #include "drmtest.h" #include "igt_core.h" +#include "igt_device.h" +#include "igt_device_scan.h" +#include "igt_pci.h" #include "igt_sriov_device.h" IGT_TEST_DESCRIPTION("Basic tests for enabling SR-IOV Virtual Functions"); @@ -118,10 +123,71 @@ static void bind_unbind_vf(int pf_fd, unsigned int vf_num) igt_sriov_disable_vfs(pf_fd); } +static unsigned int count_pci_virtfn_entries(const char *pci_slot) +{ + char path[PATH_MAX]; + unsigned int count = 0; + struct dirent *de; + DIR *dir; + + snprintf(path, sizeof(path), "/sys/bus/pci/devices/%s", pci_slot); + dir = opendir(path); + igt_assert_f(dir, "Failed to open %s\n", path); + + while ((de = readdir(dir))) { + if (!strncmp(de->d_name, "virtfn", strlen("virtfn"))) + count++; + } + + closedir(dir); + + return count; +} + +/** + * SUBTEST: pf-unbind-with-vfs-enabled-numvfs-all + * Description: + * Verify the PF driver unbind when all VFs are enabled. + * + * SUBTEST: pf-unbind-with-vf-probed + * Description: + * Verify the PF driver unbind when one VF is enabled and probed. + */ +static void pf_unbind_with_vfs_enabled(int *pf_fd, unsigned int num_vfs, bool vf_probe) +{ + char pci_slot[NAME_MAX]; + + vf_probe ? igt_sriov_enable_driver_autoprobe(*pf_fd) : + igt_sriov_disable_driver_autoprobe(*pf_fd); + igt_sriov_enable_vfs(*pf_fd, num_vfs); + + igt_device_get_pci_slot_name(*pf_fd, pci_slot); + igt_assert_eq(count_pci_virtfn_entries(pci_slot), num_vfs); + igt_assert(!drm_close_driver(*pf_fd)); + *pf_fd = -1; + igt_assert(!igt_pci_device_unbind(pci_slot)); + igt_assert(!igt_pci_get_bound_driver_name(pci_slot, NULL, 0)); + igt_assert_eq(count_pci_virtfn_entries(pci_slot), 0); +} + +static void restore_pf_after_unbind(int *pf_fd, const char *pci_slot, const char *driver) +{ + int ret; + + ret = igt_pci_get_bound_driver_name(pci_slot, NULL, 0); + if (!ret) + igt_assert(!igt_pci_driver_bind(driver, pci_slot)); + else + igt_assert_eq(ret, 1); + + if (*pf_fd < 0) + *pf_fd = drm_open_driver(DRIVER_ANY); +} + int igt_main() { - int pf_fd; - bool autoprobe; + static int pf_fd = -1; + static bool autoprobe; igt_fixture() { pf_fd = drm_open_driver(DRIVER_ANY); @@ -210,7 +276,35 @@ int igt_main() } } + igt_subtest_group() { + static char pci_slot[NAME_MAX]; + static char driver[NAME_MAX]; + + igt_fixture() { + igt_device_set_filter_from_fd(pf_fd); + igt_device_get_pci_slot_name(pf_fd, pci_slot); + igt_assert(igt_pci_get_bound_driver_name(pci_slot, driver, sizeof(driver))); + } + + igt_describe("Test unbinds the PF driver when all VFs are enabled"); + igt_subtest("pf-unbind-with-vfs-enabled-numvfs-all") { + for_max_sriov_num_vfs(pf_fd, num_vfs) { + pf_unbind_with_vfs_enabled(&pf_fd, num_vfs, false); + } + } + + igt_describe("Test unbinds the PF driver when one VF is enabled and probed"); + igt_subtest("pf-unbind-with-vf-probed") { + pf_unbind_with_vfs_enabled(&pf_fd, 1, true); + } + + igt_fixture() { + restore_pf_after_unbind(&pf_fd, pci_slot, driver); + } + } + igt_fixture() { + igt_abort_on_f(pf_fd < 0, "Device is not accessible\n"); 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)"); -- 2.43.0