Re: [PATCH] tests/intel/xe_configfs: Skip ctx-restore subtests on SR-IOV VF

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

On 2/24/2026 2:20 PM, Kamil Konieczny wrote:
> Hi Nitin,
> On 2026-02-24 at 16:50:42 +0530, Nitin Gote wrote:
>> The ctx-restore subtests use intel_register_read() via direct MMIO
>> (PCI BAR) access to verify that the batch buffer wrote the expected
>> register values. On SR-IOV Virtual Functions (VF), only a very limited
>> subset of registers is accessible. Registers like 0x4F100 are not
>> exposed to VFs, so the read always returns 0, causing a false test
>> failure.
>>
>> Skip all ctx-restore subtests (post-bb, mid-bb, and their invalid
>> variants) when the device is running in SR-IOV VF mode.
Does it mean the feature is not applicable to VFs or just the 
verification is a bit harder ?
> +cc Lukasz and Marcin
>
>> Signed-off-by: Nitin Gote <nitin.r.gote@intel.com>
>> ---
>>   tests/intel/xe_configfs.c | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/tests/intel/xe_configfs.c b/tests/intel/xe_configfs.c
>> index 0db4af201..ab55e333b 100644
>> --- a/tests/intel/xe_configfs.c
>> +++ b/tests/intel/xe_configfs.c
>> @@ -409,6 +409,7 @@ int igt_main()
>>   
>>   	igt_describe("Validate ctx_restore_post_bb with invalid options");
>>   	igt_subtest("ctx-restore-post-bb-invalid") {
>> +		igt_require_f(!is_vf_device, "ctx-restore not supported in VF\n");
>>   		configfs_device_fd = create_device_configfs_group(configfs_fd);
>>   		test_ctx_restore_invalid(configfs_device_fd, "post");
>>   		close_configfs_group(configfs_fd, configfs_device_fd);
>> @@ -416,6 +417,7 @@ int igt_main()
>>   
>>   	igt_describe("Validate ctx_restore_post_bb");
>>   	igt_subtest("ctx-restore-post-bb") {
>> +		igt_require_f(!is_vf_device, "ctx-restore not supported in VF\n");
>>   		configfs_device_fd = create_device_configfs_group(configfs_fd);
>>   		test_ctx_restore(configfs_device_fd, "post");
>>   		close_configfs_group(configfs_fd, configfs_device_fd);
>> @@ -423,6 +425,7 @@ int igt_main()
>>   
>>   	igt_describe("Validate ctx_restore_mid_bb with invalid options");
>>   	igt_subtest("ctx-restore-mid-bb-invalid") {
>> +		igt_require_f(!is_vf_device, "ctx-restore not supported in VF\n");
>>   		configfs_device_fd = create_device_configfs_group(configfs_fd);
>>   		test_ctx_restore_invalid(configfs_device_fd, "mid");
>>   		close_configfs_group(configfs_fd, configfs_device_fd);
>> @@ -430,6 +433,7 @@ int igt_main()
>>   
>>   	igt_describe("Validate ctx_restore_mid_bb");
>>   	igt_subtest("ctx-restore-mid-bb") {
>> +		igt_require_f(!is_vf_device, "ctx-restore not supported in VF\n");
>>   		configfs_device_fd = create_device_configfs_group(configfs_fd);
>>   		test_ctx_restore(configfs_device_fd, "mid");
>>   		close_configfs_group(configfs_fd, configfs_device_fd);
>> -- 
>> 2.50.1
>>