[Intel-gfx] [PATCH igt] igt/gem_ringfill: Add exercising the default-ring to basic testing

[Chris Wilson]

ringfill generates a few very common errors when submitting requests,
and historically these have been where we have had many implementation
bugs, repeated over and over again.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
 tests/gem_ringfill.c | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/tests/gem_ringfill.c b/tests/gem_ringfill.c
index bd1f371..f81a70b 100644
--- a/tests/gem_ringfill.c
+++ b/tests/gem_ringfill.c
@@ -189,30 +189,27 @@ static void run_test(int fd, unsigned ring, unsigned flags)
 igt_main
 {
 	const struct {
-		const char *prefix;
 		const char *suffix;
 		unsigned flags;
 	} modes[] = {
-		{ "basic-", "", 0 },
-		{ "", "-interruptible", INTERRUPTIBLE },
-		{ "", "-hang", HANG },
-		{ "", "-child", CHILD },
-		{ "", "-forked", FORKED },
-		{ "", "-bomb", BOMB | INTERRUPTIBLE },
-		{ NULL, NULL, 0 }
+		{ "", 0 },
+		{ "-interruptible", INTERRUPTIBLE },
+		{ "-hang", HANG },
+		{ "-child", CHILD },
+		{ "-forked", FORKED },
+		{ "-bomb", BOMB | INTERRUPTIBLE },
+		{ NULL, 0 }
 	}, *mode;
 	const struct intel_execution_engine *e;
 	int fd;
 
-	igt_skip_on_simulation();
-
 	igt_fixture
 		fd = drm_open_driver_master(DRIVER_INTEL);
 
-	for (mode = modes; mode->prefix; mode++) {
+	for (mode = modes; mode->suffix; mode++) {
 		for (e = intel_execution_engines; e->name; e++) {
 			igt_subtest_f("%s%s%s",
-				      e->exec_id || (mode->flags & ~INTERRUPTIBLE) ? "" : mode->prefix,
+				      e->exec_id ? "" : "basic-",
 				      e->name,
 				      mode->suffix)
 				run_test(fd, e->exec_id | e->flags, mode->flags);
-- 
2.7.0