VirtualBox

Changeset 62932 in vbox for trunk/src/VBox/Devices/Graphics


Ignore:
Timestamp:
Aug 3, 2016 4:50:15 PM (8 years ago)
Author:
vboxsync
Message:

Devices: warnings

Location:
trunk/src/VBox/Devices/Graphics
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Devices/Graphics/DevVGA-SVGA.cpp

    r62618 r62932  
    11601160                AssertRCReturn(rc, rc);
    11611161            }
     1162# ifdef LOG_ENABLED
    11621163            Log(("configured=%d busy=%d\n", pThis->svga.fConfigured, pThis->svga.pFIFOR3[SVGA_FIFO_BUSY]));
    11631164            uint32_t *pFIFO = pThis->svga.pFIFOR3;
    11641165            Log(("next %x stop %x\n", pFIFO[SVGA_FIFO_NEXT_CMD], pFIFO[SVGA_FIFO_STOP]));
     1166# endif
    11651167
    11661168            /* Disable or enable dirty page tracking according to the current fTraces value. */
    … …  
    26312633            uint32_t cbPayload = 0;
    26322634            uint32_t u32IrqStatus = 0;
    2633             bool     fTriggerIrq = false;
    26342635
    26352636            Assert(offCurrentCmd < offFifoMax && offCurrentCmd >= offFifoMin);
    … …  
    27812782                    switch (cmd)
    27822783                    {
    2783                     case SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS:
    2784                     {
    2785                         SVGAEscapeVideoSetRegs *pVideoCmd = (SVGAEscapeVideoSetRegs *)(pEscape + 1);
    2786                         AssertBreak(pEscape->size >= sizeof(pVideoCmd->header));
    2787                         uint32_t cRegs = (pEscape->size - sizeof(pVideoCmd->header)) / sizeof(pVideoCmd->items[0]);
    2788 
    2789                         Log(("SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS: stream %x\n", pVideoCmd->header.streamId));
    2790                         for (uint32_t iReg = 0; iReg < cRegs; iReg++)
     2784                        case SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS:
    27912785                        {
    2792                             Log(("SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS: reg %x val %x\n", pVideoCmd->items[iReg].registerId, pVideoCmd->items[iReg].value));
     2786                            SVGAEscapeVideoSetRegs *pVideoCmd = (SVGAEscapeVideoSetRegs *)(pEscape + 1);
     2787                            AssertBreak(pEscape->size >= sizeof(pVideoCmd->header));
     2788                            uint32_t cRegs = (pEscape->size - sizeof(pVideoCmd->header)) / sizeof(pVideoCmd->items[0]);
     2789
     2790                            Log(("SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS: stream %x\n", pVideoCmd->header.streamId));
     2791                            for (uint32_t iReg = 0; iReg < cRegs; iReg++)
     2792                                Log(("SVGA_ESCAPE_VMWARE_VIDEO_SET_REGS: reg %x val %x\n", pVideoCmd->items[iReg].registerId, pVideoCmd->items[iReg].value));
     2793
     2794                            RT_NOREF_PV(pVideoCmd);
     2795                            break;
     2796
    27932797                        }
    2794                         break;
    2795                     }
    2796 
    2797                     case SVGA_ESCAPE_VMWARE_VIDEO_FLUSH:
    2798                         SVGAEscapeVideoFlush *pVideoCmd = (SVGAEscapeVideoFlush *)(pEscape + 1);
    2799                         AssertBreak(pEscape->size >= sizeof(*pVideoCmd));
    2800                         Log(("SVGA_ESCAPE_VMWARE_VIDEO_FLUSH: stream %x\n", pVideoCmd->streamId));
    2801                         break;
     2798
     2799                        case SVGA_ESCAPE_VMWARE_VIDEO_FLUSH:
     2800                        {
     2801                            SVGAEscapeVideoFlush *pVideoCmd = (SVGAEscapeVideoFlush *)(pEscape + 1);
     2802                            AssertBreak(pEscape->size >= sizeof(*pVideoCmd));
     2803                            Log(("SVGA_ESCAPE_VMWARE_VIDEO_FLUSH: stream %x\n", pVideoCmd->streamId));
     2804                            RT_NOREF_PV(pVideoCmd);
     2805                            break;
     2806                        }
    28022807                    }
    28032808                }
    … …  
    29022907                    uint32_t             iDescriptor = 0;
    29032908                    RTGCPHYS             GCPhys;
    2904                     PVMSVGAGMRDESCRIPTOR paDescOld = NULL;
    29052909                    bool                 fGCPhys64 = !!(pCmd->flags & SVGA_REMAP_GMR2_PPN64);
    29062910
    … …  
    36943698static DECLCALLBACK(int) vmsvgaFIFOLoopWakeUp(PPDMDEVINS pDevIns, PPDMTHREAD pThread)
    36953699{
     3700    RT_NOREF(pDevIns);
    36963701    PVGASTATE pThis = (PVGASTATE)pThread->pvUser;
    36973702    Log(("vmsvgaFIFOLoopWakeUp\n"));
    … …  
    39253930static DECLCALLBACK(void) vmsvgaR3Info(PPDMDEVINS pDevIns, PCDBGFINFOHLP pHlp, const char *pszArgs)
    39263931{
     3932    RT_NOREF(pszArgs);
    39273933    PVGASTATE       pThis = PDMINS_2_DATA(pDevIns, PVGASTATE);
    39283934    PVMSVGAR3STATE  pSVGAState = pThis->svga.pSvgaR3State;

  • trunk/src/VBox/Devices/Graphics/DevVGA-SVGA3d-info.cpp

    r62514 r62932  
    814814                        uint32_t cchMaxX, uint32_t cchMaxY)
    815815{
     816    RT_NOREF(cbImage);
     817
    816818    /*
    817819     * Skip stuff we can't or won't need to handle.

  • trunk/src/VBox/Devices/Graphics/DevVGA-SVGA3d-internal.h

    r62679 r62932  
    137137        { \
    138138            BOOL fMakeCurrentRc = wglMakeCurrent((pContext)->hdc, (pContext)->hglrc); \
    139             Assert(fMakeCurrentRc == TRUE); \
     139            Assert(fMakeCurrentRc == TRUE); RT_NOREF_PV(fMakeCurrentRc); \
    140140            LogFlowFunc(("Changing context: %#x -> %#x\n", (pState)->idActiveContext, (pContext)->id)); \
    141141            (pState)->idActiveContext = (pContext)->id; \
    … …  
    161161                                                 (pContext)->window, \
    162162                                                 (pContext)->glxContext); \
    163             Assert(fMakeCurrentRc == True); \
     163            Assert(fMakeCurrentRc == True); RT_NOREF_PV(fMakeCurrentRc); \
    164164            LogFlowFunc(("Changing context: %#x -> %#x\n", (pState)->idActiveContext, (pContext)->id)); \
    165165            (pState)->idActiveContext = (pContext)->id; \
    … …  
    10411041
    10421042/* DevVGA-SVGA3d-shared.cpp: */
    1043 uint32_t vmsvga3dSaveShaderConst(PVMSVGA3DCONTEXT pContext, uint32_t reg, SVGA3dShaderType type, SVGA3dShaderConstType ctype,
    1044                                  uint32_t val1, uint32_t val2, uint32_t val3, uint32_t val4);
     1043int vmsvga3dSaveShaderConst(PVMSVGA3DCONTEXT pContext, uint32_t reg, SVGA3dShaderType type, SVGA3dShaderConstType ctype,
     1044                            uint32_t val1, uint32_t val2, uint32_t val3, uint32_t val4);
    10451045
    10461046

  • trunk/src/VBox/Devices/Graphics/DevVGA-SVGA3d-savedstate.cpp

    r62514 r62932  
    149149int vmsvga3dLoadExec(PVGASTATE pThis, PSSMHANDLE pSSM, uint32_t uVersion, uint32_t uPass)
    150150{
     151    RT_NOREF(uVersion, uPass);
    151152    PVMSVGA3DSTATE pState = pThis->svga.p3dState;
    152153    AssertReturn(pState, VERR_NO_MEMORY);
    … …  
    427428static int vmsvga3dSaveContext(PVGASTATE pThis, PSSMHANDLE pSSM, PVMSVGA3DCONTEXT pContext)
    428429{
     430    RT_NOREF(pThis);
    429431    uint32_t cid = pContext->id;
    430432
    … …  
    846848}
    847849
    848 uint32_t vmsvga3dSaveShaderConst(PVMSVGA3DCONTEXT pContext, uint32_t reg, SVGA3dShaderType type, SVGA3dShaderConstType ctype, uint32_t val1, uint32_t val2, uint32_t val3, uint32_t val4)
     850int vmsvga3dSaveShaderConst(PVMSVGA3DCONTEXT pContext, uint32_t reg, SVGA3dShaderType type, SVGA3dShaderConstType ctype,
     851                            uint32_t val1, uint32_t val2, uint32_t val3, uint32_t val4)
    849852{
    850853    /* Choose a sane upper limit. */

  • trunk/src/VBox/Devices/Graphics/DevVGA-SVGA3d-shared.cpp

    r62514 r62932  
    7878 *
    7979 * @returns VBox status code.
    80  * @param   pDevIns     The VGA device instance.
    81  * @param   pThread     The send thread.
     80 * @param   hThreadSelf     This thread.
     81 * @param   pvUser          Request sempahore handle.
    8282 */
    83 DECLCALLBACK(int) vmsvga3dWindowThread(RTTHREAD ThreadSelf, void *pvUser)
    84 {
     83DECLCALLBACK(int) vmsvga3dWindowThread(RTTHREAD hThreadSelf, void *pvUser)
     84{
     85    RT_NOREF(hThreadSelf);
    8586    RTSEMEVENT      WndRequestSem = (RTSEMEVENT)pvUser;
    8687    WNDCLASSEX      wc;
    … …  
    170171            if (msg.message == WM_VMSVGA3D_DESTROYWINDOW)
    171172            {
    172                 BOOL ret = DestroyWindow((HWND)msg.wParam);
    173                 Assert(ret);
     173                BOOL fRc = DestroyWindow((HWND)msg.wParam);
     174                Assert(fRc); NOREF(fRc);
    174175                /* Signal to the caller that we're done. */
    175176                RTSemEventSignal(WndRequestSem);
    … …  
    192193                pCS->cy = rectClient.bottom - rectClient.top;
    193194#endif
    194                 BOOL ret = SetWindowPos(hwnd, 0, pCS->x, pCS->y, pCS->cx, pCS->cy, SWP_NOZORDER | SWP_NOMOVE);
    195                 Assert(ret);
     195                BOOL fRc = SetWindowPos(hwnd, 0, pCS->x, pCS->y, pCS->cx, pCS->cy, SWP_NOZORDER | SWP_NOMOVE);
     196                Assert(fRc); NOREF(fRc);
    196197
    197198                /* Signal to the caller that we're done. */

  • trunk/src/VBox/Devices/Graphics/HGSMI/HGSMIHost.cpp

    r62514 r62932  
    9191#define VBOXHGSMI_STATE_FIFOSTOP_MAGIC 0x1fedcba9
    9292
    93 #define VBOXHGSMI_SAVE_START(_pSSM)     do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_START_MAGIC);      AssertRC(rc2);}while(0)
    94 #define VBOXHGSMI_SAVE_STOP(_pSSM)      do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_STOP_MAGIC);       AssertRC(rc2);}while(0)
    95 #define VBOXHGSMI_SAVE_FIFOSTART(_pSSM) do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_FIFOSTART_MAGIC);  AssertRC(rc2);}while(0)
    96 #define VBOXHGSMI_SAVE_FIFOSTOP(_pSSM)  do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_FIFOSTOP_MAGIC);   AssertRC(rc2);}while(0)
     93#define VBOXHGSMI_SAVE_START(_pSSM)     do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_START_MAGIC);     AssertRC(rc2); }while(0)
     94#define VBOXHGSMI_SAVE_STOP(_pSSM)      do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_STOP_MAGIC);      AssertRC(rc2); }while(0)
     95#define VBOXHGSMI_SAVE_FIFOSTART(_pSSM) do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_FIFOSTART_MAGIC); AssertRC(rc2); }while(0)
     96#define VBOXHGSMI_SAVE_FIFOSTOP(_pSSM)  do{ int rc2 = SSMR3PutU32(_pSSM, VBOXHGSMI_STATE_FIFOSTOP_MAGIC);  AssertRC(rc2); }while(0)
    9797
    9898#define VBOXHGSMI_LOAD_CHECK(_pSSM, _v) \
    … …  
    304304        if (RTListIsEmpty(&pIns->guestCmdCompleted))
    305305        {
    306             if(pIns->pHGFlags)
    307             {
     306            if (pIns->pHGFlags)
    308307                ASMAtomicAndU32(&pIns->pHGFlags->u32HostFlags, ~HGSMIHOSTFLAGS_GCOMMAND_COMPLETED);
    309             }
    310308        }
    311309
    … …  
    355353
    356354    int rc = hgsmiFIFOLock(pIns);
    357     if(RT_SUCCESS(rc))
     355    if (RT_SUCCESS(rc))
    358356    {
    359357        /* Search the Read list for the given buffer offset. */
    … …  
    392390
    393391        hgsmiFIFOUnlock(pIns);
    394         if(!bCompleteFirst)
     392        if (!bCompleteFirst)
    395393            LogRel(("HGSMI[%s]: ignored invalid write to the host FIFO: 0x%08X!!!\n", pIns->pszName, offBuffer));
    396394    }
    … …  
    420418    VM_ASSERT_EMT(pIns->pVM);
    421419
    422     AssertPtrReturn(pIns->pHGFlags, VERR_WRONG_ORDER);
     420    AssertPtrReturn(pIns->pHGFlags, HGSMIOFFSET_VOID);
    423421    int rc = hgsmiFIFOLock(pIns);
    424422    AssertRC(rc);
    425     if(RT_SUCCESS(rc))
     423    if (RT_SUCCESS(rc))
    426424    {
    427425        /* Get the host FIFO head entry. */
    … …  
    454452            return pEntry->offBuffer;
    455453        }
     454
    456455        hgsmiFIFOUnlock(pIns);
    457456    }
    … …  
    723722    HGSMIINSTANCE *pIns = pEntry->pIns;
    724723    int rc = hgsmiFIFOLock (pIns);
    725     if(RT_SUCCESS(rc))
     724    if (RT_SUCCESS(rc))
    726725    {
    727726        RTListNodeRemove(&pEntry->nodeEntry);
    … …  
    731730
    732731        rc = hgsmiHostHeapLock (pIns);
    733         if(RT_SUCCESS(rc))
     732        if (RT_SUCCESS(rc))
    734733        {
    735734            /* Deallocate the host heap memory. */
    … …  
    12921291              hgsmiHostHeapHandleLocationOffset(&pIns->hostHeap);
    12931292    rc = SSMR3PutU32 (pSSM, off);
    1294     if(off != HGSMIOFFSET_VOID)
     1293    if (off != HGSMIOFFSET_VOID)
    12951294    {
    12961295        SSMR3PutU32 (pSSM, hgsmiHostHeapOffset(&pIns->hostHeap));
    … …  
    12991298        SSMR3PutU64 (pSSM, (uint64_t)(uintptr_t)pIns->area.pu8Base);
    13001299        rc = hgsmiFIFOLock (pIns);
    1301         if(RT_SUCCESS(rc))
     1300        if (RT_SUCCESS(rc))
    13021301        {
    13031302            rc = hgsmiHostSaveFifoLocked (&pIns->hostFIFO, pSSM); AssertRC(rc);
    … …  
    13271326int HGSMIHostLoadStateExec (PHGSMIINSTANCE pIns, PSSMHANDLE pSSM, uint32_t u32Version)
    13281327{
    1329     if(u32Version < VGA_SAVEDSTATE_VERSION_HGSMI)
     1328    if (u32Version < VGA_SAVEDSTATE_VERSION_HGSMI)
    13301329        return VINF_SUCCESS;
    13311330
    … …  
    13461345    pIns->pHGFlags = (off != HGSMIOFFSET_VOID) ? (HGSMIHOSTFLAGS*)HGSMIOffsetToPointer (&pIns->area, off) : NULL;
    13471346
    1348     HGSMIHOSTHEAP hHeap = pIns->hostHeap;
    13491347    rc = SSMR3GetU32(pSSM, &off);
    13501348    AssertLogRelRCReturn(rc, rc);
    1351     if(off != HGSMIOFFSET_VOID)
     1349    if (off != HGSMIOFFSET_VOID)
    13521350    {
    13531351        /* There is a saved heap. */
    … …  
    13701368        {
    13711369            rc = hgsmiFIFOLock (pIns);
    1372             if(RT_SUCCESS(rc))
     1370            if (RT_SUCCESS(rc))
    13731371            {
    13741372                rc = hgsmiHostLoadFifoLocked (pIns, &pIns->hostFIFO, pSSM);
    … …  
    14401438 */
    14411439
    1442 static int hgsmiChannelMapCreate (PHGSMIINSTANCE pIns,
    1443                                   const char *pszChannel,
    1444                                   uint8_t *pu8Channel)
    1445 {
    1446     /* @todo later */
     1440static int hgsmiChannelMapCreate(PHGSMIINSTANCE pIns, const char *pszChannel, uint8_t *pu8Channel)
     1441{
     1442    RT_NOREF(pIns, pszChannel, pu8Channel);
     1443    /** @todo later */
    14471444    return VERR_NOT_SUPPORTED;
    14481445}
    … …  
    14781475/* Register a new HGSMI channel by name.
    14791476 */
    1480 int HGSMIChannelRegisterName (PHGSMIINSTANCE pIns,
    1481                               const char *pszChannel,
    1482                               PFNHGSMICHANNELHANDLER pfnChannelHandler,
    1483                               void *pvChannelHandler,
    1484                               uint8_t *pu8Channel)
     1477int HGSMIChannelRegisterName(PHGSMIINSTANCE pIns,
     1478                             const char *pszChannel,
     1479                             PFNHGSMICHANNELHANDLER pfnChannelHandler,
     1480                             void *pvChannelHandler,
     1481                             uint8_t *pu8Channel)
    14851482{
    14861483    LogFlowFunc(("pIns %p, pszChannel %s, pfnChannelHandler %p, pvChannelHandler %p, pu8Channel %p\n",
    … …  
    15911588
    15921589            HGSMIBUFFERLOCATION *pLoc = (HGSMIBUFFERLOCATION *)pvBuffer;
    1593             if(pLoc->cbLocation != sizeof(HGSMIHOSTFLAGS))
     1590            if (pLoc->cbLocation != sizeof(HGSMIHOSTFLAGS))
    15941591            {
    15951592                rc = VERR_INVALID_PARAMETER;
    … …  
    17051702{
    17061703    uint32_t flags = 0;
    1707     if(pIns->pHGFlags)
     1704    if (pIns->pHGFlags)
    17081705    {
    17091706        /* treat the abandoned commands as read.. */
    1710         while(HGSMIHostRead (pIns) != HGSMIOFFSET_VOID)  {}
     1707        while (HGSMIHostRead(pIns) != HGSMIOFFSET_VOID)
     1708        {}
    17111709        flags = pIns->pHGFlags->u32HostFlags;
    17121710        pIns->pHGFlags->u32HostFlags = 0;
    … …  
    17141712
    17151713    /* .. and complete them */
    1716     while(hgsmiProcessHostCmdCompletion (pIns, 0, true)) {}
     1714    while (hgsmiProcessHostCmdCompletion(pIns, 0, true))
     1715    {}
    17171716
    17181717#ifdef VBOX_WITH_WDDM
    1719     while(hgsmiProcessGuestCmdCompletion(pIns) != HGSMIOFFSET_VOID) {}
     1718    while(hgsmiProcessGuestCmdCompletion(pIns) != HGSMIOFFSET_VOID)
     1719    {}
    17201720#endif
    17211721
    … …  
    17941794    if (RT_SUCCESS (rc))
    17951795    {
    1796         if(bDoIrq)
     1796        if (bDoIrq)
    17971797        {
    17981798            /* Now guest can read the FIFO, the notification is informational. */
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette