[i-g-t,5/8] tests/gem_sync: Tune down for BAT to ~1s for basic-each and ~2s for basic-all.

Commit Message

Marius Vlad May 27, 2016, 11:50 a.m. UTC

Signed-off-by: Marius Vlad <[e-mail]>
---
 tests/gem_sync.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Chris Wilson May 27, 2016, 11:59 a.m. UTC | #1

On Fri, May 27, 2016 at 02:50:35PM +0300, Marius Vlad wrote:
> Signed-off-by: Marius Vlad <[e-mail]>

NAK. gem_sync is not reliable enough, please suggest how to improve
detection rates for *existing* issues.
-Chris

Patch

diff --git a/tests/gem_sync.c b/tests/gem_sync.c
index 320bce3..f824d1e 100644
--- a/tests/gem_sync.c
+++ b/tests/gem_sync.c
@@ -126,7 +126,7 @@  sync_ring(int fd, unsigned ring, int num_children)
 				gem_execbuf(fd, &execbuf);
 				gem_sync(fd, object.handle);
 			} while (++cycles & 1023);
-		} while ((elapsed = gettime() - start) < SLOW_QUICK(10, 1));
+		} while ((elapsed = gettime() - start) < 1);
 		igt_info("%s%sompleted %ld cycles: %.3f us\n",
 			 names[child % num_engines] ?: "",
 			 names[child % num_engines] ? " c" : "C",
@@ -191,7 +191,7 @@  sync_all(int fd, int num_children)
 				}
 				gem_sync(fd, object.handle);
 			} while (++cycles & 1023);
-		} while ((elapsed = gettime() - start) < SLOW_QUICK(10, 1));
+		} while ((elapsed = gettime() - start) < SLOW_QUICK(2, 1));
 		igt_info("Completed %ld cycles: %.3f us\n",
 			 cycles, elapsed*1e6/cycles);