[igt-dev] [PATCH i-g-t] intel-ci: Add i915 query tests

[Lionel Landwerlin]

On 15/03/18 13:32, Chris Wilson wrote:
> Quoting Lionel Landwerlin (2018-03-13 14:41:57)
>> These tests don't trigger any workload on the GPU. They verify that :
>>
>>      - the query uAPI works as expected
>>
>>      - the kernel reports appropriate values to userspaces for the RCS
>>        topology
>>
>> This group of tests completes in about 50ms.
>>
>> Overall it sounds like a sensible candidate for the fast feedback
>> list.
>>
>> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
>> ---
>>   tests/intel-ci/fast-feedback.testlist | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
>> index f71a16bc..286ed22a 100644
>> --- a/tests/intel-ci/fast-feedback.testlist
>> +++ b/tests/intel-ci/fast-feedback.testlist
>> @@ -158,6 +158,13 @@ igt at gem_wait@basic-busy-all
>>   igt at gem_wait@basic-wait-all
>>   igt at gem_wait@basic-await-all
>>   igt at gem_workarounds@basic-read
>> +igt at i915_query@query-garbage
>> +igt at i915_query@query-garbage-items
>> +igt at i915_query@query-topology-kernel-writes
>> +igt at i915_query@query-topology-unsupported
>> +igt at i915_query@query-topology-coherent-slice-mask
>> +igt at i915_query@query-topology-matches-eu-total
>> +igt at i915_query@query-topology-known-pci-ids
> I just feel this is a very niche set that doesn't provide any 
> significant coverage of the driver. It doesn't give any indication as 
> to the overall health, and so only need to be run later to catch any 
> inadvertent breakage to a corner of the API. Just my 2c. -Chris

Fair enough, I just wanted to get other people's feel.

-
Lionel