[Piglit] [PATCH] AMD_performance_monitor: add missing subtest result

Samuel Pitoiset samuel.pitoiset at gmail.com
Sun Mar 1 09:10:35 PST 2015


On 03/01/2015 07:16 AM, Kenneth Graunke wrote:
> On Saturday, February 28, 2015 06:20:15 PM Samuel Pitoiset wrote:
>> The "counters in range" subtest was not reported.
>>
>> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
>> ---
>>   tests/spec/amd_performance_monitor/measure.c | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/tests/spec/amd_performance_monitor/measure.c b/tests/spec/amd_performance_monitor/measure.c
>> index 5fe75d5..ace2879 100644
>> --- a/tests/spec/amd_performance_monitor/measure.c
>> +++ b/tests/spec/amd_performance_monitor/measure.c
>> @@ -244,6 +244,8 @@ test_basic_measurement(unsigned group)
>>   	}
>>   	verify(result_size == ((char *) p - (char *) data));
>>   
>> +	piglit_report_subtest_result(PIGLIT_PASS, "%s", test_name);
>> +
>>   	free(data);
>>   
>>   	/**
>>
> Oops!  Good catch.
>
> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
>
> Do you need someone to commit this for you?

Yes, I don't have commit access to piglit.
Thank you.


More information about the Piglit mailing list