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

Kenneth Graunke kenneth at whitecape.org
Sat Feb 28 22:16:34 PST 2015


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?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150228/832601c4/attachment.sig>


More information about the Piglit mailing list