Re: [PATCH v2 i-g-t] tests/intel-ci/xe-sriov-vm.blocklist.txt: Blocklist for SR-IOV VM runs

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

On 3/27/2025 6:03 PM, Kamil Konieczny wrote:
> Hi Marcin,
> On 2025-03-26 at 13:42:03 +0100, Marcin Bernatowicz wrote:
>> This file contains tests that are not applicable for QEMU Virtual Machine
>> (VM) runs. It extends xe.blocklist.txt and should be kept as minimal as
>> possible.
> 
> Do this apply to all suspend related tests? I see only two added,
> but grep gives:
> 
> grep -A2 igt_system_suspend tests/intel/xe*c
> 
> tests/intel/xe_pm_residency.c
> tests/intel/xe_gt_freq.c

Those should be already SKIPs in VM due to missing sysfs attributes.

> 
>>
>> v2: Added igt@xe_css@suspend-resume.
>>      Removed inaccessible reference. (Lukasz)
>>
>> Cc: Adam Miszczak <adam.miszczak@linux.intel.com>
>> Cc: Jakub Kolakowski <jakub1.kolakowski@intel.com>
>> Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
>> Cc: Lukasz Laguna <lukasz.laguna@intel.com>
>> Cc: Michal Wajdeczko <Michal.Wajdeczko@intel.com>
>> Cc: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
>> Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
>> ---
>>   tests/intel-ci/xe-sriov-vm.blocklist.txt | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>   create mode 100644 tests/intel-ci/xe-sriov-vm.blocklist.txt
>>
>> diff --git a/tests/intel-ci/xe-sriov-vm.blocklist.txt b/tests/intel-ci/xe-sriov-vm.blocklist.txt
>> new file mode 100644
>> index 000000000..06ed60b52
>> --- /dev/null
>> +++ b/tests/intel-ci/xe-sriov-vm.blocklist.txt
>> @@ -0,0 +1,8 @@
>> +##################################################################
>> +# Block list for VM runs
>> +##################################################################
>> +#
>> +# rtcwake does not wake the VM from s2idle
>> +#
>> +igt@xe_pm@s2idle.*
>> +igt@xe_ccs@suspend-resume
> 
> This last could be:
> igt@xe_.*@suspend.*
> 

To be more precise, we are interested in SUSPEND_STATE_FREEZE.
Let's keep it as is for now.

Thanks for feedback
--
marcin

> Up to you.
>
> Regards,
> Kamil
> 
>> -- 
>> 2.31.1
>>