Changeset 29504 in vbox for trunk/src/VBox/Devices/Storage

Timestamp:

May 16, 2010 1:43:18 PM (15 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

61641

Message:

AHCI: Always use an intermediate buffer for now until we have native S/G operations (fixes OS X read performance)

File:

• trunk/src/VBox/Devices/Storage/DevAHCI.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Storage/DevAHCI.cpp

@@ -4286 +4286 @@
     /*
      * Create a safe mapping when doing post processing because the size of the
-     * data to transfer and the amount of guest memory reserved can differ
+     * data to transfer and the amount of guest memory reserved can differ.
+     *
+     * @fixme: Read performance is really bad on OS X hosts because there is no
+     *         S/G support and the I/O manager has to create a newrequest
+     *         for every segment. The default limit of active requests is 16 on OS X
+     *         which causes a the bad read performance (writes are not affected
+     *         because of the writeback cache).
+     *         For now we will always use an intermediate buffer until
+     *         there is support for host S/G operations.
      */
-    if (pAhciPortTaskState->pfnPostProcess)
+    if (pAhciPortTaskState->pfnPostProcess || true)
     {
         ahciLog(("%s: Request with post processing.\n", __FUNCTION__));