diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-03-05 10:51:06 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2018-03-07 08:50:20 +0000 |
| commit | e3a0ed9855a2e9d77cfd0c770ceadac5b7030e4e (patch) | |
| tree | 33d3fcdf0dbdc687315a61935e5855aebc45d954 /tests | |
| parent | b4689dce36d0fbd9aec70d5a4b077c43a6b9c254 (diff) | |
Bump measure_ring_size() timer interval
It appears that waiting for a 100us period whereby we are unable to
submit another batch and proclaim the ring full, may have the false
positive where the scheduler intervenes and we are signalled twice
before having slept on ring space. Increasing the interval reduces the
likelihood of the scheduler stealing the cpu from us, but does not
eliminate it. Fortuitously it appears to be a rare false positive.
For the library routine, we can fork a RT process but that seems a bit
overkill!
References: https://bugs.freedesktop.org/show_bug.cgi?id=105343
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Reviewed-by: Antonio Argenziano <antonio.argenziano@intel.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/gem_exec_fence.c | 4 | ||||
| -rw-r--r-- | tests/gem_exec_schedule.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/tests/gem_exec_fence.c b/tests/gem_exec_fence.c index 93ed3b9b..26bde788 100644 --- a/tests/gem_exec_fence.c +++ b/tests/gem_exec_fence.c @@ -551,9 +551,9 @@ static void test_keep_in_fence(int fd, unsigned int engine, unsigned int flags) sigaction(SIGALRM, &sa, NULL); itv.it_interval.tv_sec = 0; - itv.it_interval.tv_usec = 100; + itv.it_interval.tv_usec = 1000; itv.it_value.tv_sec = 0; - itv.it_value.tv_usec = 1000; + itv.it_value.tv_usec = 10000; setitimer(ITIMER_REAL, &itv, NULL); execbuf.flags |= LOCAL_EXEC_FENCE_IN; diff --git a/tests/gem_exec_schedule.c b/tests/gem_exec_schedule.c index 8a69ab5c..89cf4285 100644 --- a/tests/gem_exec_schedule.c +++ b/tests/gem_exec_schedule.c @@ -833,9 +833,9 @@ static void test_pi_ringfull(int fd, unsigned int engine) sigaction(SIGALRM, &sa, NULL); itv.it_interval.tv_sec = 0; - itv.it_interval.tv_usec = 100; + itv.it_interval.tv_usec = 1000; itv.it_value.tv_sec = 0; - itv.it_value.tv_usec = 1000; + itv.it_value.tv_usec = 10000; setitimer(ITIMER_REAL, &itv, NULL); last = -1; |