[igt-dev] [PATCH i-g-t] tests/i915/query: more accurate available/fused topology printout

[Lionel Landwerlin]

On 08/11/2018 15:37, Tvrtko Ursulin wrote:
>
> On 08/11/2018 11:25, Lionel Landwerlin wrote:
>> Make availability of all elements more visible for debug purposes.
>>
>> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
>> ---
>>   tests/i915/query.c | 12 ++++++++++--
>>   1 file changed, 10 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/i915/query.c b/tests/i915/query.c
>> index 08aabf94..3e7fd140 100644
>> --- a/tests/i915/query.c
>> +++ b/tests/i915/query.c
>> @@ -373,12 +373,20 @@ test_query_topology_matches_eu_total(int fd)
>>       for (s = 0; s < topo_info->max_slices; s++) {
>>           int ss;
>>   -        igt_debug("slice%i:\n", s);
>> +        igt_debug("slice%i: (%s)\n", s,
>> +              slice_available(topo_info, s) ? "available" : "fused");
>> +
>> +        if (!slice_available(topo_info, s))
>> +            continue;
>>             for (ss = 0; ss < topo_info->max_subslices; ss++) {
>>               int eu, n_subslice_eus = 0;
>>   -            igt_debug("\tsubslice: %i\n", ss);
>> +            igt_debug("\tsubslice%i: (%s)\n", ss,
>> +                  subslice_available(topo_info, s, ss) ? "available" 
>> : "fused");
>> +
>> +            if (!subslice_available(topo_info, s, ss))
>> +                continue;
>>                 igt_debug("\t\teu_mask: 0b");
>>               for (eu = 0; eu < topo_info->max_eus_per_subslice; eu++) {
>>
>
> Looks like a no-brainer now.
>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>
> Regards,
>
> Tvrtko
>
Thanks, pushed.