Re: [PATCH i-g-t 1/2] tests/intel-ci/: Extend VF and VM blocklists

["Bernatowicz, Marcin" <marcin.bernatowicz@linux.intel.com>]

On 8/25/2025 6:53 PM, Jakub Kolakowski wrote:
> Currently VF and VM blocklists are supposed to be short and only
> contain tests that are breaking/aborting the CI run. However with the
> huge amount of expected skips present in VM runs it is justified to
> change the approach of using those blocklists and include all tests that
> are expected to skip when driver is in VF mode.
> Extend VF and VM blocklists and include expected skips. This should
> speed up CI runs in configurations where driver is in VF mode (mainly VM
> runs where many kms tests are skipping).
> 
> Signed-off-by: Jakub Kolakowski <jakub1.kolakowski@intel.com>
> Cc: Adam Miszczak <adam.miszczak@linux.intel.com>
> Cc: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
> Cc: Lukasz Laguna <lukasz.laguna@intel.com>
> Cc: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
> ---
>   tests/intel-ci/xe-sriov-vf.blocklist.txt | 54 +++++++++++++++++++++++-
>   tests/intel-ci/xe-sriov-vm.blocklist.txt | 52 +++++++++++++++++++++++
>   2 files changed, 105 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel-ci/xe-sriov-vf.blocklist.txt b/tests/intel-ci/xe-sriov-vf.blocklist.txt
> index 03371d80c..69d647ef0 100644
> --- a/tests/intel-ci/xe-sriov-vf.blocklist.txt
> +++ b/tests/intel-ci/xe-sriov-vf.blocklist.txt
> @@ -4,4 +4,56 @@
>   #
>   # Prevent potential module reloads during tests
>   #
> -igt@xe_module_load
> +igt@xe_module_load@.*
> +#
> +# tests applicable for PF only
> +#
> +igt@sriov_basic.*
> +igt@xe_eudebug_sriov.*
> +igt@xe_pmu@.*fn-engine-activity.*
> +igt@xe_sriov.*
> +#
> +# display related tests
> +#
> +igt@fbdev.*
> +igt@kms.*
> +igt@testdisplay
> +igt@xe_pat@.*display.*
> +igt@xe_pxp@.*display.*
> +#
> +# power management related tests
> +#
> +igt@xe_gt_freq.*
> +igt@xe_pm_residency.*
> +#
> +# OA is disabled in VF mode
> +#
> +igt@xe_oa@.*
> +igt@xe_fault_injection@oa-add-config-fail-xe_oa_alloc_regs
> +igt@xe_query@.*query-oa-units
> +#
> +# engine utilization data not available in VF mode
> +#
> +igt@xe_drm_fdinfo@.*utilization.*
> +#
> +# EU stall feature not available in VF mode
> +#
> +igt@xe_eu_stall.*
> +#
> +# The "mocs" debugfs entry is expected to be missing in VF mode
> +#
> +igt@xe_exec_reset@gt-mocs-reset
> +igt@xe_live_ktest@xe_mocs
> +igt@xe_pm@.*-mocs
> +#
> +# PCI power management capability not present in VF mode
> +#
> +igt@xe_pm@.*d3
> +#
> +# freq0 node not available for VF
> +#
> +igt@xe_pmu@gt-frequency
> +#
> +# The feature query_cs_cycles is not supported on VF
> +#
> +igt@xe_query@.*cs-cycles
> \ No newline at end of file
> diff --git a/tests/intel-ci/xe-sriov-vm.blocklist.txt b/tests/intel-ci/xe-sriov-vm.blocklist.txt
> index 06ed60b52..6271b1a70 100644
> --- a/tests/intel-ci/xe-sriov-vm.blocklist.txt
> +++ b/tests/intel-ci/xe-sriov-vm.blocklist.txt
> @@ -6,3 +6,55 @@
>   #
>   igt@xe_pm@s2idle.*
>   igt@xe_ccs@suspend-resume
> +#
> +# tests applicable for PF only
> +#
> +igt@sriov_basic.*
> +igt@xe_eudebug_sriov.*
> +igt@xe_pmu@.*fn-engine-activity.*
> +igt@xe_sriov.*
> +#
> +# display related tests
> +#
> +igt@fbdev.*
> +igt@kms.*
> +igt@testdisplay
> +igt@xe_pat@.*display.*
> +igt@xe_pxp@.*display.*
> +#
> +# power management related tests
> +#
> +igt@xe_gt_freq.*
> +igt@xe_pm_residency.*
> +#
> +# OA is disabled in VF mode
> +#
> +igt@xe_oa@.*
> +igt@xe_fault_injection@oa-add-config-fail-xe_oa_alloc_regs
> +igt@xe_query@.*query-oa-units
> +#
> +# engine utilization data not available in VF mode
> +#
> +igt@xe_drm_fdinfo@.*utilization.*
> +#
> +# EU stall feature not available in VF mode
> +#
> +igt@xe_eu_stall.*
> +#
> +# The "mocs" debugfs entry is expected to be missing in VF mode
> +#
> +igt@xe_exec_reset@gt-mocs-reset
> +igt@xe_live_ktest@xe_mocs
> +igt@xe_pm@.*-mocs
> +#
> +# PCI power management capability not present in VF mode
> +#
> +igt@xe_pm@.*d3
> +#
> +# freq0 node not available for VF
> +#
> +igt@xe_pmu@gt-frequency
> +#
> +# The feature query_cs_cycles is not supported on VF
> +#
> +igt@xe_query@.*cs-cycles
> \ No newline at end of file

LGTM,
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>