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

Timestamp:

Feb 15, 2010 3:36:01 AM (15 years ago)

Author:

vboxsync

Message:

VMM,asm.h,Config.kmk: win.amd64 warnings; fixed the ASMAtomicSubU32 signature.

Location:

trunk/src/VBox/VMM

Files:

• Makefile.kmk (modified) (1 diff)
• PDMAsyncCompletionFile.cpp (modified) (2 diffs)
• PDMAsyncCompletionFileCache.cpp (modified) (2 diffs)
• PDMAsyncCompletionFileNormal.cpp (modified) (1 diff)
• SSM.cpp (modified) (7 diffs)

trunk/src/VBox/VMM/Makefile.kmk

@@ -181 +181 @@
         PDMAsyncCompletionFileCache.cpp
 endif
+if defined(VBOX_WITH_WARNINGS_AS_ERRORS) && "$(KBUILD_TARGET)" == "win"
+PDMAsyncCompletionFileCache.cpp_CXXFLAGS = -WX-
+endif
 
 ifdef VBOX_WITH_RAW_MODE

trunk/src/VBox/VMM/PDMAsyncCompletionFile.cpp

@@ -326 +326 @@
     else
     {
-        uint32_t uOld = ASMAtomicSubU32(&pTaskFile->cbTransferLeft, pTask->DataSeg.cbSeg);
+        Assert((uint32_t)pTask->DataSeg.cbSeg == pTask->DataSeg.cbSeg && (int32_t)pTask->DataSeg.cbSeg >= 0);
+        uint32_t uOld = ASMAtomicSubS32(&pTaskFile->cbTransferLeft, (int32_t)pTask->DataSeg.cbSeg);
 
         if (!(uOld - pTask->DataSeg.cbSeg)
@@ -347 +348 @@
            || (enmTransfer == PDMACTASKFILETRANSFER_WRITE));
 
-    ASMAtomicWriteS32(&pTaskFile->cbTransferLeft, cbTransfer);
+    Assert((uint32_t)cbTransfer == cbTransfer && (uint32_t)cbTransfer >= 0);
+    ASMAtomicWriteS32(&pTaskFile->cbTransferLeft, (int32_t)cbTransfer);
     ASMAtomicWriteBool(&pTaskFile->fCompleted, false);
 

trunk/src/VBox/VMM/PDMAsyncCompletionFileCache.cpp

@@ -605 +605 @@
     PPDMACFILETASKSEG pNext = pTaskSeg->pNext;
 
-    uint32_t uOld = ASMAtomicSubU32(&pTaskSeg->pTask->cbTransferLeft, pTaskSeg->cbTransfer);
+    uint32_t uOld = ASMAtomicSubS32(&pTaskSeg->pTask->cbTransferLeft, pTaskSeg->cbTransfer);
     AssertMsg(uOld >= pTaskSeg->cbTransfer, ("New value would overflow\n"));
     if (!(uOld - pTaskSeg->cbTransfer)
@@ -1771 +1771 @@
                 LogFlow(("Evicted enough bytes (%u requested). Creating new cache entry\n", cbToWrite));
 
-                uint8_t *pbBuf;
                 PPDMACFILECACHEENTRY pEntryNew;
 

trunk/src/VBox/VMM/PDMAsyncCompletionFileNormal.cpp

@@ -374 +374 @@
 static int pdmacFileAioMgrNormalReqsEnqueue(PPDMACEPFILEMGR pAioMgr,
                                             PPDMASYNCCOMPLETIONENDPOINTFILE pEndpoint,
-                                            PRTFILEAIOREQ pahReqs, size_t cReqs)
+                                            PRTFILEAIOREQ pahReqs, unsigned cReqs)
 {
     int rc;

trunk/src/VBox/VMM/SSM.cpp

@@ -559 +559 @@
              * @{ */
             /** The size of the file header. */
-            size_t          cbFileHdr;
+            uint32_t        cbFileHdr;
             /** The major version number. */
             uint16_t        u16VerMajor;
@@ -2346 +2346 @@
     Assert(pStrm->pCur);
     Assert(pStrm->off + cb <= RT_SIZEOFMEMB(SSMSTRMBUF, abData));
-    pStrm->off += cb;
+    pStrm->off += (uint32_t)cb;
     return VINF_SUCCESS;
 }
@@ -3408 +3408 @@
     while (cbToFill > 0)
     {
-        size_t cb = RT_MIN(sizeof(g_abZero), cbToFill);
+        uint32_t cb = RT_MIN(sizeof(g_abZero), cbToFill);
         int rc = ssmR3DataWrite(pSSM, g_abZero, cb);
         if (RT_FAILURE(rc))
@@ -5798 +5798 @@
                     /* Spill the remainer into the data buffer. */
                     memset(&pSSM->u.Read.abDataBuffer[0], 0, cbToRead - cbBuf);
-                    pSSM->u.Read.cbDataBuffer  = cbToRead - cbBuf;
+                    pSSM->u.Read.cbDataBuffer  = cbToRead - (uint32_t)cbBuf;
                     pSSM->u.Read.offDataBuffer = 0;
                     cbToRead = (uint32_t)cbBuf;
@@ -6902 +6902 @@
                 while (pSSM->u.Read.cbRecLeft)
                 {
-                    uint8_t abBuf[8192];
-                    size_t  cbToRead = RT_MIN(pSSM->u.Read.cbRecLeft, sizeof(abBuf));
+                    uint8_t  abBuf[8192];
+                    uint32_t cbToRead = RT_MIN(pSSM->u.Read.cbRecLeft, sizeof(abBuf));
                     int rc = ssmR3DataReadV2Raw(pSSM, abBuf, cbToRead);
                     if (RT_FAILURE(rc))
@@ -7142 +7142 @@
     {
         char        szMagic[sizeof(SSMFILEHDR_MAGIC_V2_0)];
-        size_t      cbHdr;
+        uint32_t    cbHdr;
         unsigned    uFmtVerMajor;
         unsigned    uFmtVerMinor;
@@ -8465 +8465 @@
     int rc = ssmR3StrmPeekAt(&pSSM->Strm, offDir, pDir, cbDir, NULL);
     AssertLogRelRCReturn(rc, rc);
-    rc = ssmR3ValidateDirectory(pDir, cbDir, offDir, cDirEntries, pSSM->u.Read.cbFileHdr, pSSM->u.Read.u32SvnRev);
+    rc = ssmR3ValidateDirectory(pDir, (uint32_t)cbDir, offDir, cDirEntries, pSSM->u.Read.cbFileHdr, pSSM->u.Read.u32SvnRev);
     if (RT_FAILURE(rc))
         return rc;