All of lore.kernel.org
 help / color / mirror / Atom feed
* [Intel-gfx] [PATCH i-g-t] i915/gem_exec_create: Trim loop counters
@ 2020-03-03 14:18 ` Chris Wilson
  0 siblings, 0 replies; 6+ messages in thread
From: Chris Wilson @ 2020-03-03 14:18 UTC (permalink / raw)
  To: intel-gfx; +Cc: igt-dev

gem_create + gem_execbuf should be long enough that the accuracy of
gettime is insignificant; and this is a test not a benchmark! Remove the
1024 loop so that we check for timeout more frequently.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/i915/gem_exec_create.c | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/tests/i915/gem_exec_create.c b/tests/i915/gem_exec_create.c
index 102a161d5..e3ceb3f5c 100644
--- a/tests/i915/gem_exec_create.c
+++ b/tests/i915/gem_exec_create.c
@@ -93,23 +93,21 @@ static void all(int fd, unsigned flags, int timeout, int ncpus)
 		count = 0;
 		clock_gettime(CLOCK_MONOTONIC, &start);
 		do {
-			for (int loop = 0; loop < 1024; loop++) {
-				for (int n = 0; n < nengine; n++) {
-					obj.handle =  gem_create(fd, 4096);
-					gem_write(fd, obj.handle, 0, &bbe, sizeof(bbe));
-					execbuf.flags &= ~ENGINE_FLAGS;
-					execbuf.flags |= engines[n];
-					gem_execbuf(fd, &execbuf);
-					if (flags & LEAK)
-						gem_madvise(fd, obj.handle, I915_MADV_DONTNEED);
-					else
-						gem_close(fd, obj.handle);
-				}
+			for (int n = 0; n < nengine; n++) {
+				obj.handle = gem_create(fd, 4096);
+				gem_write(fd, obj.handle, 0, &bbe, sizeof(bbe));
+				execbuf.flags &= ~ENGINE_FLAGS;
+				execbuf.flags |= engines[n];
+				gem_execbuf(fd, &execbuf);
+				if (flags & LEAK)
+					gem_madvise(fd, obj.handle, I915_MADV_DONTNEED);
+				else
+					gem_close(fd, obj.handle);
 			}
-			count += nengine * 1024;
+			count += nengine;
 			clock_gettime(CLOCK_MONOTONIC, &now);
 		} while (elapsed(&start, &now) < timeout); /* Hang detection ~120s */
-		obj.handle =  gem_create(fd, 4096);
+		obj.handle = gem_create(fd, 4096);
 		gem_write(fd, obj.handle, 0, &bbe, sizeof(bbe));
 		for (int n = 0; n < nengine; n++) {
 			execbuf.flags &= ~ENGINE_FLAGS;
-- 
2.25.1

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

^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-03-04  3:39 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-03 14:18 [Intel-gfx] [PATCH i-g-t] i915/gem_exec_create: Trim loop counters Chris Wilson
2020-03-03 14:18 ` [igt-dev] " Chris Wilson
2020-03-03 14:34 ` [Intel-gfx] " Mika Kuoppala
2020-03-03 14:34   ` [igt-dev] " Mika Kuoppala
2020-03-03 15:43 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2020-03-04  3:39 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.