Changeset 39078 in vbox for trunk/src/VBox/VMM/VMMAll/EMAll.cpp

Timestamp:

Oct 21, 2011 2:18:22 PM (13 years ago)

Author:

vboxsync

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

74507

Message:

VMM: -Wunused-parameter

File:

• trunk/src/VBox/VMM/VMMAll/EMAll.cpp (modified) (45 diffs)

trunk/src/VBox/VMM/VMMAll/EMAll.cpp

@@ -402 +402 @@
     if (IOM_SUCCESS(rcStrict))
         pCtxCore->rip += cbOp;
+    NOREF(pVCpu);
     return rcStrict;
 #else
+    NOREF(pVM); NOREF(pVCpu); NOREF(pCtxCore); NOREF(pDis); NOREF(cbOp);
     AssertReleaseMsgFailed(("not implemented\n"));
     return VERR_NOT_IMPLEMENTED;
@@ -421 +423 @@
      * instruction and it either flushed the TLB or the CPU reused it.
      */
+#else
+    NOREF(pVM);
 #endif
     return PGMPhysInterpretedReadNoHandlers(pVCpu, pCtxCore, pvDst, GCPtrSrc, cb, /*fMayTrap*/ false);
@@ -430 +434 @@
     /* Don't use MMGCRamWrite here as it does not respect zero pages, shared
        pages or write monitored pages. */
+    NOREF(pVM);
     return PGMPhysInterpretedWriteNoHandlers(pVCpu, pCtxCore, GCPtrDst, pvSrc, cb, /*fMayTrap*/ false);
 }
@@ -508 +513 @@
 {
     OP_PARAMVAL param1, param2;
+    NOREF(pvFault);
 
     /* Source to make DISQueryParamVal read the register value - ugly hack */
@@ -642 +648 @@
 {
     OP_PARAMVAL param1;
+    NOREF(pvFault);
 
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_DEST);
@@ -711 +718 @@
     Assert(pDis->mode != CPUMODE_64BIT);    /** @todo check */
     OP_PARAMVAL param1;
+    NOREF(pvFault);
+
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_DEST);
     if(RT_FAILURE(rc))
@@ -791 +800 @@
 {
     OP_PARAMVAL param1, param2;
+    NOREF(pvFault);
 
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_DEST);
@@ -887 +897 @@
     void *pvParam1;
     OP_PARAMVAL param1, param2;
+    NOREF(pvFault);
 
 #if HC_ARCH_BITS == 32 && !defined(VBOX_WITH_HYBRID_32BIT_KERNEL_IN_R0)
@@ -958 +969 @@
                              PFNEMULATEPARAM3 pfnEmulate)
 {
+    NOREF(pvFault);
     OP_PARAMVAL param1, param2;
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_DEST);
@@ -1205 +1217 @@
 static int emInterpretMov(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pvFault);
     OP_PARAMVAL param1, param2;
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_DEST);
@@ -1343 +1356 @@
     uint64_t cTransfers;
     int      offIncrement;
+    NOREF(pvFault);
 
     /* Don't support any but these three prefix bytes. */
@@ -1493 +1507 @@
 {
     OP_PARAMVAL param1, param2;
+    NOREF(pvFault);
 
 #if HC_ARCH_BITS == 32 && !defined(VBOX_WITH_HYBRID_32BIT_KERNEL_IN_R0)
@@ -1564 +1579 @@
     Assert(pDis->mode != CPUMODE_64BIT);    /** @todo check */
     OP_PARAMVAL param1;
+    NOREF(pvFault);
 
     /* Source to make DISQueryParamVal read the register value - ugly hack */
@@ -1619 +1635 @@
     void *pvParamReg2;
     size_t cbParamReg2;
+    NOREF(pvFault);
 
     /* Source to make DISQueryParamVal read the register value - ugly hack */
@@ -1746 +1763 @@
 {
     /* only allow direct calls to EMInterpretIret for now */
+    NOREF(pVM); NOREF(pVCpu); NOREF(pDis); NOREF(pRegFrame); NOREF(pvFault); NOREF(pcbSize);
     return VERR_EM_INTERPRETER;
 }
@@ -1755 +1773 @@
 {
     /* Nothing to do. */
+    NOREF(pVM); NOREF(pVCpu); NOREF(pDis); NOREF(pRegFrame); NOREF(pvFault); NOREF(pcbSize);
     return VINF_SUCCESS;
 }
@@ -1774 +1793 @@
      * (in absence of segment override prefixes)????
      */
+    NOREF(pVM); NOREF(pRegFrame);
 #ifdef IN_RC
     LogFlow(("RC: EMULATE: invlpg %RGv\n", pAddrGC));
@@ -1795 +1815 @@
     OP_PARAMVAL param1;
     RTGCPTR     addr;
+    NOREF(pvFault); NOREF(pVM); NOREF(pcbSize);
 
     VBOXSTRICTRC rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_SOURCE);
@@ -1843 +1864 @@
 {
     uint32_t iLeaf = pRegFrame->eax;
+    NOREF(pVM);
 
     /* cpuid clears the high dwords of the affected 64 bits registers. */
@@ -1862 +1884 @@
 static int emInterpretCpuId(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pVM); NOREF(pVCpu); NOREF(pDis); NOREF(pRegFrame); NOREF(pvFault); NOREF(pcbSize);
     int rc = EMInterpretCpuId(pVM, pVCpu, pRegFrame);
     return rc;
@@ -1883 +1906 @@
     int rc = CPUMGetGuestCRx(pVCpu, SrcRegCrx, &val64);
     AssertMsgRCReturn(rc, ("CPUMGetGuestCRx %d failed\n", SrcRegCrx), VERR_EM_INTERPRETER);
+    NOREF(pVM);
 
     if (CPUMIsGuestIn64BitCode(pVCpu, pRegFrame))
@@ -1909 +1933 @@
 VMMDECL(int) EMInterpretCLTS(PVM pVM, PVMCPU pVCpu)
 {
+    NOREF(pVM);
     uint64_t cr0 = CPUMGetGuestCR0(pVCpu);
     if (!(cr0 & X86_CR0_TS))
@@ -1920 +1945 @@
 static int emInterpretClts(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pRegFrame); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretCLTS(pVM, pVCpu);
 }
@@ -1940 +1966 @@
     uint64_t msrEFER;
     int      rc, rc2;
+    NOREF(pVM);
 
     /** @todo Clean up this mess. */
@@ -2123 +2150 @@
     OP_PARAMVAL param1;
     uint32_t    val;
+    NOREF(pvFault); NOREF(pcbSize);
 
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_SOURCE);
@@ -2202 +2230 @@
 static int emInterpretMovCRx(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pvFault); NOREF(pcbSize);
     if ((pDis->param1.flags == USE_REG_GEN32 || pDis->param1.flags == USE_REG_GEN64) && pDis->param2.flags == USE_REG_CR)
         return EMInterpretCRxRead(pVM, pVCpu, pRegFrame, pDis->param1.base.reg_gen, pDis->param2.base.reg_ctrl);
@@ -2227 +2256 @@
     uint64_t val;
     int      rc;
+    NOREF(pVM);
 
     if (CPUMIsGuestIn64BitCode(pVCpu, pRegFrame))
@@ -2265 +2295 @@
 {
     uint64_t val64;
+    NOREF(pVM);
 
     int rc = CPUMGetGuestDRx(pVCpu, SrcRegDrx, &val64);
@@ -2288 +2319 @@
 {
     int rc = VERR_EM_INTERPRETER;
+    NOREF(pvFault); NOREF(pcbSize);
 
     if((pDis->param1.flags == USE_REG_GEN32 || pDis->param1.flags == USE_REG_GEN64) && pDis->param2.flags == USE_REG_DBG)
@@ -2312 +2344 @@
     OP_PARAMVAL param1;
     RTSEL       sel;
+    NOREF(pVM); NOREF(pvFault); NOREF(pcbSize);
 
     int rc = DISQueryParamVal(pRegFrame, pDis, &pDis->param1, &param1, PARAM_SOURCE);
@@ -2360 +2393 @@
     RTGCPTR     pParam1;
     X86XDTR32   dtr32;
+    NOREF(pvFault); NOREF(pcbSize);
 
     Log(("Emulate %s at %RGv\n", emGetMnemonic(pDis), (RTGCPTR)pRegFrame->rip));
@@ -2404 +2438 @@
 static int emInterpretSti(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pcbSize);
     PPATMGCSTATE pGCState = PATMQueryGCState(pVM);
 
@@ -2430 +2465 @@
 emInterpretHlt(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pVM); NOREF(pVCpu); NOREF(pDis); NOREF(pRegFrame); NOREF(pvFault); NOREF(pcbSize);
     return VINF_EM_HALT;
 }
@@ -2456 +2492 @@
     pRegFrame->rdx = (uTicks >> 32ULL);
 
+    NOREF(pVM);
     return VINF_SUCCESS;
 }
@@ -2498 +2535 @@
 static int emInterpretRdtsc(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretRdtsc(pVM, pVCpu, pRegFrame);
 }
@@ -2526 +2564 @@
     pRegFrame->rdx = 0;
     /** @todo We should trigger a #GP here if the cpu doesn't support the index in ecx. */
+
+    NOREF(pVM);
     return VINF_SUCCESS;
 }
@@ -2534 +2574 @@
 static int emInterpretRdpmc(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretRdpmc(pVM, pVCpu, pRegFrame);
 }
@@ -2543 +2584 @@
 {
     uint32_t u32Dummy, u32ExtFeatures, cpl;
+    NOREF(pVM);
 
     if (pRegFrame->ecx != 0)
@@ -2568 +2610 @@
 static int emInterpretMonitor(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretMonitor(pVM, pVCpu, pRegFrame);
 }
@@ -2578 +2621 @@
 {
     uint32_t u32Dummy, u32ExtFeatures, cpl, u32MWaitFeatures;
+    NOREF(pVM);
 
     /* Get the current privilege level. */
@@ -2621 +2665 @@
 static VBOXSTRICTRC emInterpretMWait(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretMWait(pVM, pVCpu, pRegFrame);
 }
@@ -2734 +2779 @@
     /** @todo According to the Intel manuals, there's a REX version of RDMSR that is slightly different.
      *  That version clears the high dwords of both RDX & RAX */
+    NOREF(pVM);
 
     /* Get the current privilege level. */
@@ -2761 +2807 @@
              different, so we play safe by completely disassembling the instruction. */
     Assert(!(pDis->prefix & PREFIX_REX));
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretRdmsr(pVM, pVCpu, pRegFrame);
 }
@@ -2787 +2834 @@
     LogFlow(("EMInterpretWrmsr %s (%x) val=%RX64\n", emMSRtoString(pRegFrame->ecx), pRegFrame->ecx,
              RT_MAKE_U64(pRegFrame->eax, pRegFrame->edx)));
+    NOREF(pVM);
     return rc;
 }
@@ -2796 +2844 @@
 static int emInterpretWrmsr(PVM pVM, PVMCPU pVCpu, PDISCPUSTATE pDis, PCPUMCTXCORE pRegFrame, RTGCPTR pvFault, uint32_t *pcbSize)
 {
+    NOREF(pDis); NOREF(pvFault); NOREF(pcbSize);
     return EMInterpretWrmsr(pVM, pVCpu, pRegFrame);
 }