Re: [igt-dev] [PATCH i-g-t] tests/psr: Add a few PSR tests to fastfeedback test list for some coverage

[Martin Peres <martin.peres@linux.intel.com>]

On 17/07/18 22:41, Rodrigo Vivi wrote:
> On Tue, Jul 17, 2018 at 11:58:51AM -0700, Dhinakaran Pandiyan wrote:
>> PSR tests are primarily executed on shards machines that do not have the
>> required panels, the only test in fastfeedback does not do much. So add
>> these tests to provide some meaningful coverage.
>>
>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> Cc: Martin Peres <martin.peres@linux.intel.com>
>> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>> ---
>>  tests/intel-ci/fast-feedback.testlist | 5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
>> index 4b405e36..2e1abf23 100644
>> --- a/tests/intel-ci/fast-feedback.testlist
>> +++ b/tests/intel-ci/fast-feedback.testlist
>> @@ -243,7 +243,10 @@ igt@kms_pipe_crc_basic@read-crc-pipe-c-frame-sequence
>>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a
>>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b
>>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c
>> -igt@kms_psr@basic
> 
> we should probably now delete this basic test...
> since it's only reason was this list...

So, adding ~13 seconds and removing 3 seconds. I think we can spend an
extra 10s on PSR testing :)

Thanks for this!

Acked-by: Martin Peres <martin.peres@linux.intel.com>

> 
>> +igt@kms_psr@primary_page_flip
>> +igt@kms_psr@cursor_plane_move
>> +igt@kms_psr@sprite_plane_onoff
>> +igt@kms_psr@primary_mmap_gtt
> 
> 
> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>


> 
> 
> 
>>  igt@kms_setmode@basic-clone-single-crtc
>>  igt@pm_backlight@basic-brightness
>>  igt@pm_rpm@basic-pci-d3-state
>> -- 
>> 2.17.1
>>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev