fix fsync test on 5.11

tchaloupka · Nov 6, 2021 · 6554d1b · 6554d1b
1 parent 73b4927
commit 6554d1b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/tests/fsync.d b/tests/fsync.d
@@ -67,9 +67,7 @@ unittest
     {
         e.prepFsync(fd, FsyncFlags.DATASYNC);
         e.user_data = 2;
-        // TODO: Works with IO_LINK but not without it: See: https://github.com/axboe/liburing/issues/33
-        e.flags = cast(SubmissionEntryFlags)(SubmissionEntryFlags.IO_DRAIN | SubmissionEntryFlags.IO_LINK);
-        // e.flags = SubmissionEntryFlags.IO_DRAIN;
+        e.flags = SubmissionEntryFlags.IO_DRAIN;
     })(fd);
 
     auto ret = io.submit(NUM_WRITES + 1);
@@ -88,9 +86,10 @@ unittest
     }
     else assert(ret == NUM_WRITES + 1);
 
-    assert(io.length == NUM_WRITES + 1);
+    // assert(io.length == NUM_WRITES + 1); // TODO: fails on GH actions with 5.11
     foreach (i; 0..NUM_WRITES + 1)
     {
+        if (io.empty) io.wait(1); // TODO: workaround for above, similar is used here: https://github.com/axboe/liburing/blob/0b6b5bc79a85bc3a461c6f3ba9c0ce0dba696d4c/test/fsync.c#L111
         if (io.front.res == -EINVAL)
         {
             version (D_BetterC)