Changeset 68115 in vbox for trunk

Timestamp:

Jul 26, 2017 8:44:38 AM (8 years ago)

Author:

vboxsync

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

117178

Message:

Additions/WINNT/Video: log more reasons why setting a graphics mode fails

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPVModes.cpp (modified) (1 diff)

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

@@ -230 +230 @@
 
     /* prevent potensial overflow */
-    if (pResolution->cx > 0x7fff
-            || pResolution->cy > 0x7fff)
-    {
-        WARN(("too big resolution"));
-        return VERR_INVALID_PARAMETER;
-    }
+    if (   pResolution->cx > 0x7fff
+        || pResolution->cy > 0x7fff)
+    {
+        WARN(("resolution %dx%d insane", pResolution->cx, pResolution->cy));
+        return VERR_INVALID_PARAMETER;
+    }
+
     uint32_t cbSurfMem = pResolution->cx * pResolution->cy * 4;
     if (cbSurfMem > vramSize)
+    {
+        WARN(("resolition %dx%d too big for available VRAM (%d bytes)\n", pResolution->cx, pResolution->cy, vramSize));
         return VERR_NOT_SUPPORTED;
+    }
 
     if (!VBoxLikesVideoMode(u32Target, pResolution->cx, pResolution->cy, 32))
+    {
+        WARN(("resolution %dx%d not accepted by the frontend\n", pResolution->cx, pResolution->cy));
         return VERR_NOT_SUPPORTED;
+    }
 
     if (pModes->aTransientResolutions[u32Target] == CR_RSIZE2U64(*pResolution))