[Intel-gfx] [PATCH igt] benchmarks/gem_exec_nop: fix engines selection

Dmitry Rogozhkin dmitry.v.rogozhkin at intel.com
Mon Nov 20 11:49:46 UTC 2017


Code on loop() function shadowed function parameter which led
to inability to try out different engines in the test: we always
loaded RCS0.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103804
Signed-off-by: Dmitry Rogozhkin <dmitry.v.rogozhkin at intel.com>
Cc: Chris Wilson <chris at chris-wilson.co.uk>
---
 benchmarks/gem_exec_nop.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/benchmarks/gem_exec_nop.c b/benchmarks/gem_exec_nop.c
index 03a0354..8eb88c1 100644
--- a/benchmarks/gem_exec_nop.c
+++ b/benchmarks/gem_exec_nop.c
@@ -106,11 +106,11 @@ static int loop(unsigned ring, int reps, int ncpus, unsigned flags)
 		return 77;
 
 	all_nengine = 0;
-	for (ring = 1; ring < 16; ring++) {
+	for (unsigned r = 1; r < 16; r++) {
 		execbuf.flags &= ~ENGINE_FLAGS;
-		execbuf.flags |= ring;
+		execbuf.flags |= r;
 		if (__gem_execbuf(fd, &execbuf) == 0)
-			all_engines[all_nengine++] = ring;
+			all_engines[all_nengine++] = r;
 	}
 
 	if (ring == -1) {
-- 
1.8.3.1



More information about the Intel-gfx mailing list