Changeset 57006 in vbox for trunk/src/VBox/VMM

Timestamp:

Jul 19, 2015 1:36:21 AM (10 years ago)

Author:

vboxsync

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

101721

Message:

VMM,*: Annotated format strings in the VMM APIs and dealt with the fallout.

Location:

trunk/src/VBox/VMM

Files:

• VMMR0/GMMR0.cpp (modified) (1 diff)
• VMMR3/CPUM.cpp (modified) (1 diff)
• VMMR3/PGMDbg.cpp (modified) (2 diffs)
• VMMR3/VM.cpp (modified) (1 diff)

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

@@ -1460 +1460 @@
         {
             SUPR0Printf("gmmR0CleanupVMScanChunk: Chunk %p/%#x has bogus stats - free=%d/%d private=%d/%d shared=%d/%d\n",
-                        pChunk->cFree, cFree, pChunk->cPrivate, cPrivate, pChunk->cShared, cShared);
+                        pChunk, pChunk->Core.Key, pChunk->cFree, cFree, pChunk->cPrivate, cPrivate, pChunk->cShared, cShared);
             pChunk->cFree = cFree;
             pChunk->cPrivate = cPrivate;

trunk/src/VBox/VMM/VMMR3/CPUM.cpp

@@ -1895 +1895 @@
                 for (unsigned i = 0; i < RT_ELEMENTS(pFpuCtx->au32RsrvdRest); i++)
                     if (pFpuCtx->au32RsrvdRest[i])
-                        pHlp->pfnPrintf(pHlp, "%sRsrvdRest[i]=%RX32 (offset=%#x)\n",
+                        pHlp->pfnPrintf(pHlp, "%sRsrvdRest[%u]=%RX32 (offset=%#x)\n",
                                         pszPrefix, i, pFpuCtx->au32RsrvdRest[i], RT_OFFSETOF(X86FXSTATE, au32RsrvdRest[i]) );
             }

trunk/src/VBox/VMM/VMMR3/PGMDbg.cpp

@@ -1043 +1043 @@
         {
             pState->pHlp->pfnPrintf(pState->pHlp, "%0*llx error! PT mapping %s at HCPhys=%RHp was not found in the page pool!\n",
-                                    pState->cchAddress, pState->u64Address, HCPhys);
+                                    pState->cchAddress, pState->u64Address, pszDesc, HCPhys);
             return VERR_INVALID_PARAMETER;
         }
@@ -2354 +2354 @@
                                     ,
                                     cch, "", cch, "", cch, "", cch, "", cch, "", cch, "", cch, "",
-                                    cch, "", cch, "", cch, "", cch, "", cch, "", cch, "", cch, "Address");
+                                    cch, "", cch, "", cch, "", cch, "", cch, "", cch, "Address");
         if (pState->fLme)
             rc = pgmR3DumpHierarchyGstPaePML4(pState, cr3 & cr3Mask, cMaxDepth);

trunk/src/VBox/VMM/VMMR3/VM.cpp

@@ -3452 +3452 @@
             LogRel((" failed, because the VM state is actually %s\n", VMR3GetStateName(enmStateCur)));
             VMSetError(pVM, VERR_VM_INVALID_VM_STATE, RT_SRC_POS,
-                       N_("%s failed because the current VM state, %s, was not found in the state transition table"),
+                       N_("%s failed because the current VM state, %s, was not found in the state transition table (old state %s)"),
                        pszWho, VMR3GetStateName(enmStateCur), VMR3GetStateName(enmStateOld));
             AssertMsgFailed(("%s - state=%s, see release log for full details. Check the cTransitions passed us.\n",