Changeset 75504 in vbox for trunk/src

Timestamp:

Nov 16, 2018 2:03:43 AM (6 years ago)

Author:

vboxsync

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

126661

Message:

VMMDev/HGCM: Changed hgcmCompleted to call hgcmCompletedWorker directly rather than bothering the EMT with it. Tests with a debug build indicates that this may save quite some time.

File:

• trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp

@@ -1240 +1240 @@
 DECLCALLBACK(void) hgcmCompleted(PPDMIHGCMPORT pInterface, int32_t result, PVBOXHGCMCMD pCmd)
 {
+#if 0 /* This seems to be significantly slower.  Half of MsgTotal time seems to be spend here. */
     PVMMDEV pThis = RT_FROM_MEMBER(pInterface, VMMDevState, IHGCMPort);
     STAM_GET_TS(pCmd->tsComplete);
@@ -1251 +1252 @@
                                    (PFNRT)hgcmCompletedWorker, 3, pInterface, result, pCmd);
     AssertRC(rc);
+#else
+    STAM_GET_TS(pCmd->tsComplete);
+    VBOXDD_HGCMCALL_COMPLETED_REQ(pCmd, result);
+    hgcmCompletedWorker(pInterface, result, pCmd);
+#endif
 }