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

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

[Derek Morton]

Reduced the Sleep period to 200mS and reduced the repetition count to 7
to decrease the test run time significantly.

v2: Changed uS to us
v3: removed the output formatting change as the issue will be addressed
in a seperate patch from Thomas Wood.

Signed-off-by: Derek Morton <derek.j.morton@intel.com>
---
 tests/gem_exec_nop.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c
index a287d08..fcc9625 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);
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH i-g-t v3] tests/gem_exec_nop: Improved test run time

[Jani Nikula]

On Wed, 11 Nov 2015, Derek Morton <derek.j.morton@intel.com> wrote:
> Reduced the Sleep period to 200mS and reduced the repetition count to 7
> to decrease the test run time significantly.
>
> v2: Changed uS to us

The electrical engineer in me insists on v4 changing mS to ms, as we're
talking about time, not conductance. ;)

BR,
Jani.


> v3: removed the output formatting change as the issue will be addressed
> in a seperate patch from Thomas Wood.
>
> Signed-off-by: Derek Morton <derek.j.morton@intel.com>
> ---
>  tests/gem_exec_nop.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c
> index a287d08..fcc9625 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);

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx