Changeset 60588 in vbox

Timestamp:

Apr 20, 2016 9:49:24 AM (8 years ago)

Author:

vboxsync

Message:

Additions/WDDM: Improve the scan line and VBlank reporting, 10% of one frame period are reserved for the VBlank. Fixes performance issues with PowerPoint if Aero is disabled

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPMisc.cpp (modified) (2 diffs)

trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPMisc.cpp

@@ -1954 +1954 @@
     if (pTarget->Size.cy)
     {
-        uint32_t curScanLine;
-        BOOL bVBlank;
+        uint32_t curScanLine = 0;
+        BOOL bVBlank = FALSE;
         LARGE_INTEGER DevVSyncTime;
         DevVSyncTime.QuadPart =  ASMAtomicReadU64((volatile uint64_t*)&pDevExt->VSyncTime.QuadPart);
@@ -1964 +1964 @@
         {
             WARN(("vsync time is less than the one stored in device"));
-            curScanLine = 0;
+            bVBlank = TRUE;
         }
         else
         {
             VSyncTime.QuadPart = VSyncTime.QuadPart - DevVSyncTime.QuadPart;
-            /* time is in 100ns, */
-            curScanLine = (uint32_t)((pTarget->Size.cy * VSyncTime.QuadPart) / 166666LL); /* ASSUMES 60Hz*/
-            if (pDevExt->bVSyncTimerEnabled)
-            {
-                if (curScanLine >= pTarget->Size.cy)
-                    curScanLine = 0;
-            }
+            /* Check whether we are in VBlank state or actively drawing a scan line
+             * 10% of the 60Hz are dedicated to VBlank.
+             */
+            LARGE_INTEGER VSyncPeriod;
+            VSyncPeriod.QuadPart = VSyncTime.QuadPart % 166666LL; /* ASSUMES 60Hz*/
+            if (VSyncPeriod.QuadPart > 150000LL)
+                bVBlank = TRUE;
             else
             {
-                curScanLine %= pTarget->Size.cy;
-            }
-        }
-
-        bVBlank = (!curScanLine || curScanLine > pTarget->Size.cy);
+                /* time is in 100ns, */
+                curScanLine = (uint32_t)((pTarget->Size.cy * VSyncPeriod.QuadPart) / 150000LL);
+                if (pDevExt->bVSyncTimerEnabled)
+                {
+                    if (curScanLine > pTarget->Size.cy)
+                        curScanLine = pTarget->Size.cy;
+                }
+                else
+                    curScanLine %= pTarget->Size.cy;
+            }
+        }
+
         pGetScanLine->ScanLine = curScanLine;
         pGetScanLine->InVerticalBlank = bVBlank;