[Intel-gfx] [PATCH i-g-t] tests/gem_exec_nop: Improved test run time

[Chris Wilson]

On Tue, Nov 03, 2015 at 04:29:45PM +0000, Derek Morton wrote:
> Reduced the Sleep period to 200mS and reduced the repetition count to 7
> to decrease the test run time significantly.
> 
> Also fixed a non ascii character that messed up the results table formatting.

Pardon? Are you working around someone else's bug?
 
> Signed-off-by: Derek Morton <derek.j.morton at intel.com>
> ---
>  tests/gem_exec_nop.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c
> index a287d08..5028145 100644
> --- a/tests/gem_exec_nop.c
> +++ b/tests/gem_exec_nop.c
> @@ -133,7 +133,7 @@ static void loop(int fd, uint32_t handle, unsigned ring_id, const char *ring_nam
>  	gem_sync(fd, handle);
>  
>  	for (count = 1; count <= SLOW_QUICK(1<<17, 1<<4); count <<= 1) {
> -		const int reps = 13;
> +		const int reps = 7;
>  		igt_stats_t stats;
>  		int n;
>  
> @@ -142,7 +142,7 @@ static void loop(int fd, uint32_t handle, unsigned ring_id, const char *ring_nam
>  		for (n = 0; n < reps; n++) {
>  			struct timespec start, end;
>  			int loops = count;
> -			sleep(1); /* wait for the hw to go back to sleep */
> +			usleep(200000); /* wait 200mS for the hw to go back to sleep */
>  			clock_gettime(CLOCK_MONOTONIC, &start);
>  			while (loops--)
>  				do_ioctl(fd, DRM_IOCTL_I915_GEM_EXECBUFFER2, &execbuf);
> @@ -151,7 +151,7 @@ static void loop(int fd, uint32_t handle, unsigned ring_id, const char *ring_nam
>  			igt_stats_push(&stats, elapsed(&start, &end, count));
>  		}
>  
> -		igt_info("Time to exec x %d:		%7.3fµs (ring=%s)\n",
> +		igt_info("Time to exec x %d:		%7.3fuS (ring=%s)\n",

Did SI change the standard unit of measurement for time?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre