Changeset 57860 in vbox

Timestamp:

Sep 22, 2015 2:57:16 PM (9 years ago)

Author:

vboxsync

Message:

VMM: warnings.

Location:

trunk/src/VBox/VMM

Files:

• Makefile.kmk (modified) (2 diffs)
• VMMAll/EMAll.cpp (modified) (3 diffs)
• VMMAll/IOMAllMMIO.cpp (modified) (4 diffs)
• VMMAll/MMAllHyper.cpp (modified) (3 diffs)
• VMMR0/HMVMXR0.cpp (modified) (1 diff)
• VMMR0/PDMR0Device.cpp (modified) (1 diff)

trunk/src/VBox/VMM/Makefile.kmk

@@ -553 +553 @@
 
  ifeq ($(VBOX_LDR_FMT),elf)
-  VMMR0_CXXFLAGS += $(filter-out -Wno-unused,$(TEMPLATE_VBoxR0_CXXFLAGS)) -Wunused-variable
+  VMMR0_CXXFLAGS += $(filter-out -Wno-unused,$(TEMPLATE_VBoxR0_CXXFLAGS)) -Wunused-variable -Wno-unused-parameter
  endif
 
@@ -733 +733 @@
  VMMAll/PGMAllPool.cpp_CXXFLAGS   = $(VBOX_GCC_Wno-array_bounds)
  VMMAll/PGMAll.cpp_CXXFLAGS       = -Wno-unused-function
- VMMAll/IEMAll.cpp_CXXFLAGS       = -Wno-unused-parameter -Wno-unused-function
- VMMAll/IEMAllAImplC.cpp_CXXFLAGS = -Wno-unused-parameter
- VMMAll/CPUMAllMsrs.cpp_CXXFLAGS  = -Wno-unused-parameter
+ VMMAll/IEMAll.cpp_CXXFLAGS       = -Wno-unused-function
  VMMR0/GMMR0.cpp_CXXFLAGS         = -Wno-unused-value
 endif

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

@@ -476 +476 @@
 {
     NOREF(pVM);
-    Assert(pCtxCore == CPUMGetGuestCtxCore(pVCpu));
+    Assert(pCtxCore == CPUMGetGuestCtxCore(pVCpu)); NOREF(pCtxCore);
     DISCPUMODE enmCpuMode = CPUMGetGuestDisMode(pVCpu);
     /** @todo Deal with too long instruction (=> \#GP), opcode read errors (=>
@@ -2599 +2599 @@
             Log2(("emInterpret%s: pvFault=%RGv pParam1=%RGv val2=%x\n", emGetMnemonic(pDis), pvFault, pParam1, valpar2));
             pParam1 = (RTGCPTR)((RTGCUINTPTR)pParam1 + valpar2/8);
-            EM_ASSERT_FAULT_RETURN((RTGCPTR)((RTGCUINTPTR)pParam1 & ~3) == pvFault, VERR_EM_INTERPRETER);
+            EM_ASSERT_FAULT_RETURN((RTGCPTR)((RTGCUINTPTR)pParam1 & ~3) == pvFault, VERR_EM_INTERPRETER); NOREF(pvFault);
             rc = emRamRead(pVM, pVCpu, pRegFrame, &valpar1, pParam1, 1);
             if (RT_FAILURE(rc))
@@ -2675 +2675 @@
 
     Log2(("emInterpretLockBitTest %s: pvFault=%RGv GCPtrPar1=%RGv imm=%RX64\n", emGetMnemonic(pDis), pvFault, GCPtrPar1, ValPar2));
+    NOREF(pvFault);
 
     /* Try emulate it with a one-shot #PF handler in place. (RC) */

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

@@ -452 +452 @@
 # endif
     STAM_PROFILE_START(&pStats->CTX_SUFF_Z(ProfWrite), a);
+#else
+    NOREF(pVCpu);
 #endif
 
@@ -669 +671 @@
 # endif
     STAM_PROFILE_START(&pStats->CTX_SUFF_Z(ProfRead), a);
+#else
+    NOREF(pVCpu);
 #endif
 
@@ -1758 +1762 @@
     VBOXSTRICTRC rcStrict = IEMExecOne(pVCpu);
 
+    NOREF(pCtxCore); NOREF(GCPhysFault);
     STAM_PROFILE_STOP(&pVM->iom.s.StatRZMMIOHandler, a);
     PDMCritSectLeave(pDevIns->CTX_SUFF(pCritSectRo));
@@ -1932 +1937 @@
 {
     LogFlow(("iomMmioPfHandler: GCPhys=%RGp uErr=%#x pvFault=%RGv rip=%RGv\n",
-             GCPhysFault, (uint32_t)uErrorCode, pvFault, (RTGCPTR)pCtxCore->rip));
+             GCPhysFault, (uint32_t)uErrorCode, pvFault, (RTGCPTR)pCtxCore->rip)); NOREF(pvFault);
     return iomMmioCommonPfHandler(pVM, pVCpu, (uint32_t)uErrorCode, pCtxCore, GCPhysFault, pvUser);
 }

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

@@ -262 +262 @@
         return VERR_MM_HYPER_NO_MEMORY;
     }
+#else
+    NOREF(enmTag);
 #endif
     if (uAlignment < PAGE_SIZE)
@@ -837 +839 @@
                     ("%p: u32Magic=%#x\n", pv, pHeap->u32Magic),
                     VERR_INVALID_POINTER);
-    Assert(pHeap == pVM->mm.s.CTX_SUFF(pHyperHeap));
+    Assert(pHeap == pVM->mm.s.CTX_SUFF(pHyperHeap)); NOREF(pVM);
 
     /* Some more verifications using additional info from pHeap. */
@@ -1221 +1223 @@
     mmHyperHeapCheck(pVM->mm.s.CTX_SUFF(pHyperHeap));
     mmHyperUnlock(pVM);
+#else
+    NOREF(pVM);
 #endif
 }

trunk/src/VBox/VMM/VMMR0/HMVMXR0.cpp

@@ -7406 +7406 @@
 DECLINLINE(void) hmR0VmxSetPendingDebugXcpt(PVMCPU pVCpu, PCPUMCTX pMixedCtx)
 {
-    Assert(HMVMXCPU_GST_IS_UPDATED(pVCpu, HMVMX_UPDATED_GUEST_RFLAGS));
+    Assert(HMVMXCPU_GST_IS_UPDATED(pVCpu, HMVMX_UPDATED_GUEST_RFLAGS)); NOREF(pVCpu);
     if (pMixedCtx->eflags.Bits.u1TF)    /* We don't have any IA32_DEBUGCTL MSR for guests. Treat as all bits 0. */
     {

trunk/src/VBox/VMM/VMMR0/PDMR0Device.cpp

@@ -302 +302 @@
 
     AssertFailed();
-    NOREF(GCPhys); NOREF(pCachedData);
+    NOREF(GCPhys); NOREF(pCachedData); NOREF(pDevIns);
 
 /*    return PATMSetMMIOPatchInfo(pDevIns->Internal.s.pVMR0, GCPhys, pCachedData); */