From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
Looks like we have to do a lot of work just to count the number of
modes...
Caught by Valgrind.
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
---
tests/kms_flip.c | 17 +++++++++++++----
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index c43a26f..7e5e355 100644
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1129,6 +1129,16 @@ static unsigned event_loop(struct test_output *o, unsigned duration_ms)
return end - start;
}
+static void free_test_output(struct test_output *o)
+{
+ int i;
+
+ for (i = 0; i < o->count; i++) {
+ drmModeFreeEncoder(o->kencoder[i]);
+ drmModeFreeConnector(o->kconnector[i]);
+ }
+}
+
static void run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
int crtc_count, int duration_ms)
{
@@ -1231,10 +1241,7 @@ out:
last_connector = NULL;
- for (i = 0; i < o->count; i++) {
- drmModeFreeEncoder(o->kencoder[i]);
- drmModeFreeConnector(o->kconnector[i]);
- }
+ free_test_output(o);
}
static int run_test(int duration, int flags)
@@ -1261,6 +1268,7 @@ static int run_test(int duration, int flags)
if (o.mode_valid)
modes++;
+ free_test_output(&o);
}
}
@@ -1318,6 +1326,7 @@ static int run_pair(int duration, int flags)
if (o.mode_valid)
modes++;
+ free_test_output(&o);
}
}
}
--
1.8.3.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
