Re: [PATCH v3 i-g-t 6/7] tests/intel/xe_sriov_flr: Add --no-xe-vfio-pci option

["Laguna, Lukasz" <lukasz.laguna@intel.com>]

On 2/4/2026 17:32, Marcin Bernatowicz wrote:
> Add --no-xe-vfio-pci to skip loading/binding VFs to xe-vfio-pci.
>
> Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
> Cc: Jakub Kolakowski <jakub1.kolakowski@intel.com>
> Cc: Lukasz Laguna <lukasz.laguna@intel.com>

Reviewed-by: Lukasz Laguna <lukasz.laguna@intel.com>

> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> ---
>   tests/intel/xe_sriov_flr.c | 13 +++++++++++--
>   1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/tests/intel/xe_sriov_flr.c b/tests/intel/xe_sriov_flr.c
> index 951628159..b73727787 100644
> --- a/tests/intel/xe_sriov_flr.c
> +++ b/tests/intel/xe_sriov_flr.c
> @@ -63,6 +63,7 @@ static const char STOP_REASON_SKIP[]  = "SKIP";
>   #define DRIVER_OVERRIDE_TIMEOUT_MS 200
>   
>   static int g_wait_flr_ms = 200;
> +static bool g_use_xe_vfio_pci = true;
>   
>   static struct g_mmio {
>   	struct xe_mmio *mmio;
> @@ -385,7 +386,9 @@ static void verify_flr(int pf_fd, int num_vfs, struct subcheck *checks,
>   	if (igt_warn_on(igt_pci_system_reinit()))
>   		goto disable_vfs;
>   
> -	xe_vfio_loaded = igt_kmod_load("xe_vfio_pci", NULL) >= 0;
> +	xe_vfio_loaded = false;
> +	if (g_use_xe_vfio_pci)
> +		xe_vfio_loaded = igt_kmod_load("xe_vfio_pci", NULL) >= 0;
>   	if (xe_vfio_loaded) {
>   		vf_bound = calloc(num_vfs + 1, sizeof(*vf_bound));
>   		igt_assert(vf_bound);
> @@ -1097,6 +1100,10 @@ static int opt_handler(int opt, int opt_index, void *data)
>   	long val;
>   
>   	switch (opt) {
> +	case 'v':
> +		g_use_xe_vfio_pci = false;
> +		igt_info("xe-vfio-pci binding: disabled\n");
> +		break;
>   	case 'w':
>   		errno = 0;
>   		val = strtol(optarg, &end, 0);
> @@ -1113,14 +1120,16 @@ static int opt_handler(int opt, int opt_index, void *data)
>   }
>   
>   static const struct option long_options[] = {
> +	{ .name = "no-xe-vfio-pci", .has_arg = false, .val = 'v', },
>   	{ .name = "wait-flr-ms", .has_arg = true, .val = 'w', },
>   	{},
>   };
>   
>   static const char help_str[] =
> +	"  --no-xe-vfio-pci\tDo not load/bind xe-vfio-pci for VFs\n"
>   	"  --wait-flr-ms=MS\tSleep MS milliseconds after VF reset sysfs write (default: 200)\n";
>   
> -int igt_main_args("w:", long_options, help_str, opt_handler, NULL)
> +int igt_main_args("vw:", long_options, help_str, opt_handler, NULL)
>   {
>   	int pf_fd;
>   	bool autoprobe;