| Message ID | 1447239063-13196-1-git-send-email-derek.j.morton@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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);
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);
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(-)