Changeset 29111 in vbox for trunk/src/VBox

Timestamp:

May 5, 2010 8:28:30 PM (15 years ago)

Author:

vboxsync

Message:

IPRT/fileaio-linux: Support flushes, not supported by any filesystem yet (returns an error)

File:

• trunk/src/VBox/Runtime/r3/linux/fileaio-linux.cpp (modified) (3 diffs)

trunk/src/VBox/Runtime/r3/linux/fileaio-linux.cpp

@@ -77 +77 @@
 enum
 {
-    LNXKAIO_IOCB_CMD_READ = 0,
-    LNXKAIO_IOCB_CMD_WRITE
+    LNXKAIO_IOCB_CMD_READ   = 0,
+    LNXKAIO_IOCB_CMD_WRITE  = 1,
+    LNXKAIO_IOCB_CMD_FSYNC  = 2,
+    LNXKAIO_IOCB_CMD_FDSYNC = 3
 };
 
@@ -365 +367 @@
     RTFILEAIOREQ_NOT_STATE_RETURN_RC(pReqInt, SUBMITTED, VERR_FILE_AIO_IN_PROGRESS);
     Assert(hFile != NIL_RTFILE);
-    AssertPtr(pvBuf);
-    Assert(off >= 0);
-    Assert(cbTransfer > 0);
+
+    if (uTransferDirection != LNXKAIO_IOCB_CMD_FSYNC)
+    {
+        AssertPtr(pvBuf);
+        Assert(off >= 0);
+        Assert(cbTransfer > 0);
+    }
 
     /*
@@ -409 +415 @@
     RTFILEAIOREQ_NOT_STATE_RETURN_RC(pReqInt, SUBMITTED, VERR_FILE_AIO_IN_PROGRESS);
 
-    /** @todo: Flushing is not neccessary on Linux because O_DIRECT is mandatory
-     *         which disables caching.
-     *         We could setup a fake request which isn't really executed
-     *         to avoid platform dependent code in the caller.
-     */
-#if 0
-    return rtFileAsyncPrepareTransfer(pRequest, File, TRANSFERDIRECTION_FLUSH,
-                                      0, NULL, 0, pvUser);
-#endif
-    return VERR_NOT_IMPLEMENTED;
+    return rtFileAioReqPrepareTransfer(pReqInt, hFile, LNXKAIO_IOCB_CMD_FSYNC,
+                                       0, NULL, 0, pvUser);
 }