Changeset 39150 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Video/disp

Timestamp:

Oct 31, 2011 1:37:06 PM (13 years ago)

Author:

vboxsync

Message:

wddm: avoid unnecessary window creation (caused img flickering for MacOS guest), etc.

Location:

trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm

Files:

• VBoxDispD3D.cpp (modified) (7 diffs)
• VBoxDispD3D.h (modified) (2 diffs)
• VBoxDispDbg.h (modified) (2 diffs)

trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm/VBoxDispD3D.cpp

@@ -3097 +3097 @@
 static HRESULT vboxWddmSwapchainPresent(PVBOXWDDMDISP_DEVICE pDevice, PVBOXWDDMDISP_ALLOCATION pBbAlloc)
 {
+    /* we currently *assume* that presenting shared resource is only possible when 3d app is rendering with composited desktop on,
+     * no need to do anything else since dwm will present everything for us */
+    if (pBbAlloc->hSharedHandle)
+    {
+        VBOXVDBG_ASSERT_IS_DWM(FALSE);
+        return S_OK;
+    }
+
     BOOL bNeedPresent;
     PVBOXWDDMDISP_SWAPCHAIN pSwapchain = vboxWddmSwapchainFindCreate(pDevice, pBbAlloc, &bNeedPresent);
@@ -5304 +5312 @@
                 if (SUCCEEDED(hr))
                 {
-                    if (pResource->Pool != D3DDDIPOOL_SYSTEMMEM)
-                    {
-                        vboxWddmMemsetRc(pRc, 0);
-                    }
-                    else
+                    if (pResource->Pool == D3DDDIPOOL_SYSTEMMEM)
                     {
                         vboxWddmSurfSynchMem(pRc);
@@ -5347 +5351 @@
                 if (SUCCEEDED(hr))
                 {
-                    if (pResource->Pool != D3DDDIPOOL_SYSTEMMEM)
-                    {
-                        vboxWddmMemsetRc(pRc, 0);
-                    }
-                    else
+                    if (pResource->Pool == D3DDDIPOOL_SYSTEMMEM)
                     {
                         vboxWddmSurfSynchMem(pRc);
@@ -5392 +5392 @@
                 if (SUCCEEDED(hr))
                 {
-                    if (pResource->Pool != D3DDDIPOOL_SYSTEMMEM)
-                    {
-                        vboxWddmMemsetRc(pRc, 0);
-                    }
-                    else
+                    if (pResource->Pool == D3DDDIPOOL_SYSTEMMEM)
                     {
                         vboxWddmSurfSynchMem(pRc);
@@ -5568 +5564 @@
                 if (SUCCEEDED(hr))
                 {
-                    if (pResource->Pool != D3DDDIPOOL_SYSTEMMEM)
-                    {
-                        vboxWddmMemsetRc(pRc, 0);
-                    }
-                    else
+                    if (pResource->Pool == D3DDDIPOOL_SYSTEMMEM)
                     {
                         vboxWddmSurfSynchMem(pRc);
@@ -5639 +5631 @@
                     if (SUCCEEDED(hr))
                     {
-                        if (pResource->Pool != D3DDDIPOOL_SYSTEMMEM)
-                        {
-                            vboxWddmMemsetRc(pRc, 0);
-                        }
-                        else
+                        if (pResource->Pool == D3DDDIPOOL_SYSTEMMEM)
                         {
                             Assert(0);
@@ -5955 +5943 @@
     Assert(pRc->RcDesc.fFlags.Primary);
     Assert(pAlloc->hAllocation);
-//    PVBOXWDDMDISP_SCREEN pScreen = &pDevice->aScreens[pRc->RcDesc.VidPnSourceId];
-//    Assert(pScreen->hWnd);
-//    Assert(pScreen->pDevice9If);
     D3DDDICB_SETDISPLAYMODE DdiDm = {0};
     DdiDm.hPrimaryAllocation = pAlloc->hAllocation;
-//    DdiDm.PrivateDriverFormatAttribute = 0;
-//    Assert(pScreen->pRenderTargetRc == pRc);
-//    Assert(pScreen->iRenderTargetFrontBuf == pData->SubResourceIndex);
 
     {

trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm/VBoxDispD3D.h

@@ -164 +164 @@
     VBOXWDDMDISP_RENDERTGT aRTs[VBOXWDDMDISP_MAX_SWAPCHAIN_SIZE];
 } VBOXWDDMDISP_SWAPCHAIN, *PVBOXWDDMDISP_SWAPCHAIN;
-
-
-//typedef struct VBOXWDDMDISP_SCREEN
-//{
-//    RTLISTNODE SwapchainList;
-//    IDirect3DDevice9 *pDevice9If;
-////    struct VBOXWDDMDISP_RESOURCE *pDstSharedRc;
-//    uint32_t iRenderTargetFrontBuf;
-//    HWND hWnd;
-//} VBOXWDDMDISP_SCREEN, *PVBOXWDDMDISP_SCREEN;
 
 typedef struct VBOXWDDMDISP_DEVICE
@@ -205 +195 @@
     /* no lock is needed for this since we're guaranteed the per-device calls are not reentrant */
     RTLISTNODE DirtyAllocList;
+
     UINT cRTs;
     struct VBOXWDDMDISP_ALLOCATION * apRTs[1];

trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm/VBoxDispDbg.h

@@ -201 +201 @@
 #define VBOXVDBG_CHECK_EXE(_pszName) (vboxVDbgDoCheckExe(_pszName))
 #define VBOXVDBG_IS_DWM() (!!(g_VBoxVDbgFIsDwm >=0 ? g_VBoxVDbgFIsDwm : (g_VBoxVDbgFIsDwm = VBOXVDBG_CHECK_EXE("dwm.exe"))))
+
+#define VBOXVDBG_ASSERT_IS_DWM(_bDwm) do { \
+        Assert((!VBOXVDBG_IS_DWM()) == (!(_bDwm))); \
+    } while (0)
 
 #define VBOXVDBG_IS_DUMP_ALLOWED(_type) ( \
@@ -503 +507 @@
 #define VBOXVDBG_CHECK_BLT(_opBlt, _pSrcAlloc, _pSrcSurf, _pSrcRect, _pDstAlloc, _pDstSurf, _pDstRect) do { } while (0)
 #define VBOXVDBG_CHECK_TEXBLT(_opTexBlt, _pSrcRc, _pSrcRect, _pDstRc, _pDstPoint) do { } while (0)
+#define VBOXVDBG_ASSERT_IS_DWM(_bDwm) do {} while (0)
 #endif