commit da553ff40e2555b9ef7989882e113ae33a7448e5
author Chris Wilson <chris@chris-wilson.co.uk> Fri Sep 22 10:32:44 2017 +0100
committer Chris Wilson <chris@chris-wilson.co.uk> Mon Sep 25 10:42:34 2017 +0100
tree bcf947df3f15356e4f02911b6acf251b6275c327
parent f16a799807487b6e2dcc8c7940a1a8b424f10802

igt/gem_exec_schedule: s/-MAX_PRIO/MIN_PRIO/

Replace instances of -MAX_PRIO with MIN_PRIO for readability.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

tests/gem_exec_schedule.c

diff --git a/tests/gem_exec_schedule.c b/tests/gem_exec_schedule.c
index 69379ec..40f2ebd 100644
--- a/tests/gem_exec_schedule.c
+++ b/tests/gem_exec_schedule.c
@@ -40,6 +40,7 @@
 #define NOISE 2
 
 #define MAX_PRIO 1023
+#define MIN_PRIO -1023
 
 #define BUSY_QLEN 8
 
@@ -280,10 +281,10 @@
 	uint32_t ctx[2];
 
 	ctx[LO] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[LO], -MAX_PRIO);
+	ctx_set_priority(fd, ctx[LO], MIN_PRIO);
 
 	ctx[HI] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[HI], flags & EQUAL ? -MAX_PRIO : 0);
+	ctx_set_priority(fd, ctx[HI], flags & EQUAL ? MIN_PRIO : 0);
 
 	scratch = gem_create(fd, 4096);
 
@@ -324,13 +325,13 @@
 	uint32_t ctx[3];
 
 	ctx[LO] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[LO], -MAX_PRIO);
+	ctx_set_priority(fd, ctx[LO], MIN_PRIO);
 
 	ctx[HI] = gem_context_create(fd);
 	ctx_set_priority(fd, ctx[HI], 0);
 
 	ctx[NOISE] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[NOISE], -MAX_PRIO/2);
+	ctx_set_priority(fd, ctx[NOISE], MIN_PRIO/2);
 
 	result = gem_create(fd, 4096);
 	dep = gem_create(fd, 4096);
@@ -386,7 +387,7 @@
 	uint32_t ctx[2];
 
 	ctx[LO] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[LO], -MAX_PRIO);
+	ctx_set_priority(fd, ctx[LO], MIN_PRIO);
 
 	ctx[HI] = gem_context_create(fd);
 	ctx_set_priority(fd, ctx[HI], MAX_PRIO);
@@ -395,7 +396,7 @@
 		if (flags & NEW_CTX) {
 			gem_context_destroy(fd, ctx[LO]);
 			ctx[LO] = gem_context_create(fd);
-			ctx_set_priority(fd, ctx[LO], -MAX_PRIO);
+			ctx_set_priority(fd, ctx[LO], MIN_PRIO);
 		}
 		spin[n] = __igt_spin_batch_new(fd, ctx[LO], ring, 0);
 		igt_debug("spin[%d].handle=%d\n", n, spin[n]->handle);
@@ -437,7 +438,7 @@
 	 */
 
 	ctx[LO] = gem_context_create(fd);
-	ctx_set_priority(fd, ctx[LO], -MAX_PRIO);
+	ctx_set_priority(fd, ctx[LO], MIN_PRIO);
 
 	ctx[NOISE] = gem_context_create(fd);
 
@@ -497,7 +498,7 @@
 	ctx[HI] = gem_context_create(fd);
 
 	n = 0;
-	ctx_set_priority(fd, ctx[HI], -MAX_PRIO);
+	ctx_set_priority(fd, ctx[HI], MIN_PRIO);
 	for_each_engine(fd, other) {
 		spin[n] = __igt_spin_batch_new(fd, ctx[NOISE], other, 0);
 		store_dword(fd, ctx[HI], other,
@@ -754,7 +755,7 @@
 	if (gen < 6)
 		execbuf.flags |= I915_EXEC_SECURE;
 
-	for (int n = -MAX_PRIO, x = 1; n <= MAX_PRIO; n++, x++) {
+	for (int n = MIN_PRIO, x = 1; n <= MAX_PRIO; n++, x++) {
 		uint32_t *batch;
 
 		execbuf.rsvd1 = gem_context_create(fd);
@@ -855,7 +856,7 @@
 	execbuf.buffer_count = 1;
 	execbuf.flags = engine;
 	execbuf.rsvd1 = gem_context_create(fd);
-	ctx_set_priority(fd, execbuf.rsvd1, -MAX_PRIO);
+	ctx_set_priority(fd, execbuf.rsvd1, MIN_PRIO);
 
 	gem_execbuf(fd, &execbuf);
 	gem_sync(fd, obj[1].handle);