[PATCH i-g-t] tests/sriov_basic: Restore the igt_sriov_is_pf check

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu May 9 11:09:22 UTC 2024


Hi Marcin,
On 2024-05-09 at 10:59:56 +0000, Marcin Bernatowicz wrote:
> The requirement `igt_sriov_is_pf` is essential for accurately skipping
> tests in scenarios where the `sriov_totalvfs` attribute is absent. This
> update restores the previously removed check to ensure proper test
> behavior under such conditions.
> 
> Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz at linux.intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/sriov_basic.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/sriov_basic.c b/tests/sriov_basic.c
> index 1bc082d76..74f1ce41d 100644
> --- a/tests/sriov_basic.c
> +++ b/tests/sriov_basic.c
> @@ -120,6 +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);
> -- 
> 2.31.1
> 


More information about the igt-dev mailing list