Changeset 82675 in vbox

Timestamp:

Jan 8, 2020 4:49:01 PM (5 years ago)

Author:

vboxsync

Message:

Devices/VMMDev: Use sane (monitor is disabled) initial state when we have no hint yet, otherwise all monitors will be unconditionally enabled with X11 GA at least in the VMSVGA case. Temporary solution, eventually we'll need a way to properly signal "don't have any hint for this monitor".

File:

• trunk/src/VBox/Devices/VMMDev/VMMDev.cpp (modified) (3 diffs)

trunk/src/VBox/Devices/VMMDev/VMMDev.cpp

@@ -4272 +4272 @@
     {
         DISPLAYCHANGEREQUEST *pRequest = &pThis->displayChangeData.aRequests[i];
-        memset (&pRequest->lastReadDisplayChangeRequest, 0, sizeof (pRequest->lastReadDisplayChangeRequest));
+        memset(&pRequest->lastReadDisplayChangeRequest, 0, sizeof(pRequest->lastReadDisplayChangeRequest));
+        pRequest->lastReadDisplayChangeRequest.fDisplayFlags = VMMDEV_DISPLAY_DISABLED;
     }
     pThis->displayChangeData.iCurrentMonitor = 0;
@@ -4457 +4458 @@
     vmmdevAllocFacilityStatusEntry(pThis, VBoxGuestFacilityType_Graphics,        true /*fFixed*/, &TimeStampNow);
     Assert(pThis->cFacilityStatuses == 5);
+
+    /* disable all screens (no better hints known yet). */
+    /** @todo r=klaus need a way to represent "no hint known" */
+    for (unsigned i = 0; i < RT_ELEMENTS(pThis->displayChangeData.aRequests); i++)
+    {
+        DISPLAYCHANGEREQUEST *pRequest = &pThis->displayChangeData.aRequests[i];
+        pRequest->displayChangeRequest.fDisplayFlags = VMMDEV_DISPLAY_DISABLED;
+        pRequest->lastReadDisplayChangeRequest.fDisplayFlags = VMMDEV_DISPLAY_DISABLED;
+    }
 
     /*
@@ -4919 +4929 @@
 };
 
-#endif /* !VBOX_DEV1679
-ICE_STRUCT_TESTCASE */
+#endif /* !VBOX_DEVICE_STRUCT_TESTCASE */