[igt-dev] [PATCH i-g-t 1/2] tests/perf: bump max number of block/poll interations

[Lionel Landwerlin]

Adding Matt in Cc ;)

On 11/12/2018 18:06, Lionel Landwerlin wrote:
> Those 2 tests fail regularly on HSW, probably because the OA period
> aligns slightly differently there because of the differnce in the
> timestamp frequency between HSW and other generation. Just bump the
> max number by 1 to fix the issue.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102252
> ---
>   tests/perf.c | 16 ++--------------
>   1 file changed, 2 insertions(+), 14 deletions(-)
>
> diff --git a/tests/perf.c b/tests/perf.c
> index 48b323e5..220c52ef 100644
> --- a/tests/perf.c
> +++ b/tests/perf.c
> @@ -2025,7 +2025,7 @@ test_blocking(void)
>   	int64_t tick_ns = 1000000000 / sysconf(_SC_CLK_TCK);
>   	int64_t test_duration_ns = tick_ns * 1000;
>   
> -	int max_iterations = (test_duration_ns / oa_period) + 1;
> +	int max_iterations = (test_duration_ns / oa_period) + 2;
>   	int n_extra_iterations = 0;
>   
>   	/* It's a bit tricky to put a lower limit here, but we expect a
> @@ -2116,12 +2116,6 @@ test_blocking(void)
>   		n++;
>   	}
>   
> -	/* Updated the maximum of iterations based on the time spent
> -	 * in the loop.
> -	 */
> -	max_iterations = (end - start) / oa_period + 1;
> -	igt_debug("adjusted max iter. = %d\n", max_iterations);
> -
>   	times(&end_times);
>   
>   	/* Using nanosecond units is fairly silly here, given the tick in-
> @@ -2190,7 +2184,7 @@ test_polling(void)
>   	int64_t tick_ns = 1000000000 / sysconf(_SC_CLK_TCK);
>   	int64_t test_duration_ns = tick_ns * 1000;
>   
> -	int max_iterations = (test_duration_ns / oa_period) + 1;
> +	int max_iterations = (test_duration_ns / oa_period) + 2;
>   	int n_extra_iterations = 0;
>   
>   	/* It's a bit tricky to put a lower limit here, but we expect a
> @@ -2309,12 +2303,6 @@ test_polling(void)
>   		n++;
>   	}
>   
> -	/* Updated the maximum of iterations based on the time spent
> -	 * in the loop.
> -	 */
> -	max_iterations = (end - start) / oa_period + 1;
> -	igt_debug("adjusted max iter. = %d\n", max_iterations);
> -
>   	times(&end_times);
>   
>   	/* Using nanosecond units is fairly silly here, given the tick in-