Re: [Intel-gfx] [igt-dev] [PATCH i-g-t] tests/i915/gem_ctx_persistence: Skip some subtests

["Belgaumkar, Vinay" <vinay.belgaumkar@intel.com>]

On 6/1/2023 12:55 PM, Andrzej Hajda wrote:
>
>
> On 24.05.2023 21:19, Vinay Belgaumkar wrote:
>> Hang and heartbeat subtests are not supported with GuC submission
>> enabled.
>>
>> Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>> ---
>>   tests/i915/gem_ctx_persistence.c | 32 +++++++++++++++++++-------------
>>   1 file changed, 19 insertions(+), 13 deletions(-)
>>
>> diff --git a/tests/i915/gem_ctx_persistence.c 
>> b/tests/i915/gem_ctx_persistence.c
>> index 42cf96329..1e122535e 100644
>> --- a/tests/i915/gem_ctx_persistence.c
>> +++ b/tests/i915/gem_ctx_persistence.c
>> @@ -1366,19 +1366,25 @@ igt_main
>>         igt_subtest("hostile")
>>           test_nohangcheck_hostile(i915, &empty_cfg);
>> -    igt_subtest("hang")
>> -        test_nohangcheck_hang(i915, &empty_cfg);
>> -
>> -    igt_subtest("heartbeat-stop")
>> -        test_noheartbeat_many(i915, 1, 0);
>> -    igt_subtest("heartbeat-hang")
>> -        test_noheartbeat_many(i915, 1, IGT_SPIN_NO_PREEMPTION);
>> -    igt_subtest("heartbeat-many")
>> -        test_noheartbeat_many(i915, 16, 0);
>> -    igt_subtest("heartbeat-close")
>> -        test_noheartbeat_close(i915, 0);
>> -    igt_subtest("heartbeat-hostile")
>> -        test_noheartbeat_close(i915, IGT_SPIN_NO_PREEMPTION);
>> +
>> +    igt_subtest_group {
>> +        igt_fixture
>> +            igt_skip_on(gem_using_guc_submission(i915));
>
> As Kamil said this should be put into test function.
> Otherwise you will have misleading errors in other tests - fixture 
> will be called always regardless of running test.
>
>> +
>> +        igt_subtest("hang")
>> +            test_nohangcheck_hang(i915, &empty_cfg);
>
> What is 'missing' in GuC in case of this test? CI is happy :)

For now. I have seen this  fail before, so better to skip. I have sent 
out a patch with a skip for just this one since all others have been 
taken care of.

https://patchwork.freedesktop.org/patch/541407/

Thanks,

Vinay.

>
>
>> +
>> +        igt_subtest("heartbeat-stop")
>> +            test_noheartbeat_many(i915, 1, 0);
>> +        igt_subtest("heartbeat-hang")
>> +            test_noheartbeat_many(i915, 1, IGT_SPIN_NO_PREEMPTION);
>> +        igt_subtest("heartbeat-many")
>> +            test_noheartbeat_many(i915, 16, 0);
>> +        igt_subtest("heartbeat-close")
>> +            test_noheartbeat_close(i915, 0);
>> +        igt_subtest("heartbeat-hostile")
>> +            test_noheartbeat_close(i915, IGT_SPIN_NO_PREEMPTION);
>
> These tests are handled already by recently merged:
> https://patchwork.freedesktop.org/patch/539647/?series=118423&rev=3
>
> Regards
> Andrzej
>
>
>> +    }
>>         igt_subtest_group {
>>           igt_fixture
>