Changeset 92392 in vbox for trunk/src/VBox/VMM/VMMRZ

Timestamp:

Nov 12, 2021 10:39:56 AM (3 years ago)

Author:

vboxsync

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

148246

Message:

VMM: Removed the callring-3 API and some of the associated stuff. bugref:10093

File:

• trunk/src/VBox/VMM/VMMRZ/VMMRZ.cpp (modified) (2 diffs)

trunk/src/VBox/VMM/VMMRZ/VMMRZ.cpp

@@ -29 +29 @@
 #include <iprt/asm-amd64-x86.h>
 #include <iprt/string.h>
-
-
-/**
- * Calls the ring-3 host code.
- *
- * @returns VBox status code of the ring-3 call.
- * @retval  VERR_VMM_RING3_CALL_DISABLED if called at the wrong time. This must
- *          be passed up the stack, or if that isn't possible then VMMRZCallRing3
- *          needs to change it into an assertion.
- *
- *
- * @param   pVM             The cross context VM structure.
- * @param   pVCpu           The cross context virtual CPU structure of the calling EMT.
- * @param   enmOperation    The operation.
- * @param   uArg            The argument to the operation.
- */
-VMMRZDECL(int) VMMRZCallRing3(PVMCC pVM, PVMCPUCC pVCpu, VMMCALLRING3 enmOperation, uint64_t uArg)
-{
-    VMCPU_ASSERT_EMT(pVCpu);
-
-    /*
-     * Check if calling ring-3 has been disabled and only let let fatal calls thru.
-     */
-    if (RT_UNLIKELY(    pVCpu->vmm.s.cCallRing3Disabled != 0
-                    &&  enmOperation != VMMCALLRING3_VM_R0_ASSERTION))
-    {
-#ifndef IN_RING0
-        /*
-         * In most cases, it's sufficient to return a status code which
-         * will then be propagated up the code usually encountering several
-         * AssertRC invocations along the way. Hitting one of those is more
-         * helpful than stopping here.
-         *
-         * However, some doesn't check the status code because they are called
-         * from void functions, and for these we'll turn this into a ring-0
-         * assertion host call.
-         */
-        if (enmOperation != VMMCALLRING3_REM_REPLAY_HANDLER_NOTIFICATIONS)
-            return VERR_VMM_RING3_CALL_DISABLED;
-#endif
-#ifdef IN_RC
-        RTStrPrintf(g_szRTAssertMsg1, sizeof(pVM->vmm.s.szRing0AssertMsg1),
-                    "VMMRZCallRing3: enmOperation=%d uArg=%#llx idCpu=%#x cCallRing3Disabled=%#x\n",
-                    enmOperation, uArg, pVCpu->idCpu, pVCpu->vmm.s.cCallRing3Disabled);
-#endif
-        RTStrPrintf(pVM->vmm.s.szRing0AssertMsg1, sizeof(pVM->vmm.s.szRing0AssertMsg1),
-                    "VMMRZCallRing3: enmOperation=%d uArg=%#llx idCpu=%#x cCallRing3Disabled=%#x\n",
-                    enmOperation, uArg, pVCpu->idCpu, pVCpu->vmm.s.cCallRing3Disabled);
-        enmOperation = VMMCALLRING3_VM_R0_ASSERTION;
-    }
-
-    /*
-     * The normal path.
-     */
-/** @todo profile this! */
-    pVCpu->vmm.s.enmCallRing3Operation = enmOperation;
-    pVCpu->vmm.s.u64CallRing3Arg = uArg;
-    pVCpu->vmm.s.rcCallRing3 = VERR_VMM_RING3_CALL_NO_RC;
-#ifdef IN_RC
-    pVM->vmm.s.pfnRCToHost(VINF_VMM_CALL_HOST);
-#else
-    int rc;
-    if (pVCpu->vmm.s.pfnCallRing3CallbackR0)
-    {
-        rc = pVCpu->vmm.s.pfnCallRing3CallbackR0(pVCpu, enmOperation, pVCpu->vmm.s.pvCallRing3CallbackUserR0);
-        if (RT_FAILURE(rc))
-            return rc;
-    }
-    rc = vmmR0CallRing3LongJmp(&pVCpu->vmm.s.CallRing3JmpBufR0, VINF_VMM_CALL_HOST);
-    if (RT_FAILURE(rc))
-        return rc;
-#endif
-    return pVCpu->vmm.s.rcCallRing3;
-}
-
-
-/**
- * Simple wrapper that adds the pVCpu argument.
- *
- * @returns VBox status code of the ring-3 call.
- * @retval  VERR_VMM_RING3_CALL_DISABLED if called at the wrong time. This must
- *          be passed up the stack, or if that isn't possible then VMMRZCallRing3
- *          needs to change it into an assertion.
- *
- * @param   pVM             The cross context VM structure.
- * @param   enmOperation    The operation.
- * @param   uArg            The argument to the operation.
- */
-VMMRZDECL(int) VMMRZCallRing3NoCpu(PVMCC pVM, VMMCALLRING3 enmOperation, uint64_t uArg)
-{
-    return VMMRZCallRing3(pVM, VMMGetCpu(pVM), enmOperation, uArg);
-}
 
 
@@ -194 +102 @@
 }
 
-
-/**
- * Sets the ring-0 callback before doing the ring-3 call.
- *
- * @param   pVCpu         The cross context virtual CPU structure.
- * @param   pfnCallback   Pointer to the callback.
- * @param   pvUser        The user argument.
- *
- * @return VBox status code.
- */
-VMMRZDECL(int) VMMRZCallRing3SetNotification(PVMCPUCC pVCpu, R0PTRTYPE(PFNVMMR0CALLRING3NOTIFICATION) pfnCallback, RTR0PTR pvUser)
-{
-    AssertPtrReturn(pVCpu, VERR_INVALID_POINTER);
-    AssertPtrReturn(pfnCallback, VERR_INVALID_POINTER);
-
-    if (pVCpu->vmm.s.pfnCallRing3CallbackR0)
-        return VERR_ALREADY_EXISTS;
-
-    pVCpu->vmm.s.pfnCallRing3CallbackR0    = pfnCallback;
-    pVCpu->vmm.s.pvCallRing3CallbackUserR0 = pvUser;
-    return VINF_SUCCESS;
-}
-
-
-/**
- * Removes the ring-0 callback.
- *
- * @param   pVCpu   The cross context virtual CPU structure.
- */
-VMMRZDECL(void) VMMRZCallRing3RemoveNotification(PVMCPUCC pVCpu)
-{
-    pVCpu->vmm.s.pfnCallRing3CallbackR0 = NULL;
-}
-
-
-/**
- * Checks whether there is a ring-0 callback notification active.
- *
- * @param   pVCpu   The cross context virtual CPU structure.
- * @returns true if there the notification is active, false otherwise.
- */
-VMMRZDECL(bool) VMMRZCallRing3IsNotificationSet(PVMCPUCC pVCpu)
-{
-    return pVCpu->vmm.s.pfnCallRing3CallbackR0 != NULL;
-}
-