Changeset 63771 in vbox for trunk/src/VBox/Additions/linux/drm/vbox_mode.c

Timestamp:

Sep 8, 2016 6:46:39 PM (8 years ago)

Author:

vboxsync

Message:

bugref:8584: HGSMISetHostGuestFlags() assertion: make the Linux vboxvideo driver report the host flags location to the host before enabling interrupts, as the host sets flags when sending interrupts.

File:

• trunk/src/VBox/Additions/linux/drm/vbox_mode.c (modified) (2 diffs)

trunk/src/VBox/Additions/linux/drm/vbox_mode.c

@@ -107 +107 @@
                                 pitch, width, height,
                                 vbox_crtc->blanked ? 0 : bpp, flags);
-    VBoxHGSMIReportFlagsLocation(&vbox->submit_info,   vbox->vram_map_start
-                                                     + vbox->host_flags_offset);
     LogFunc(("vboxvideo: %d\n", __LINE__));
 }
@@ -507 +505 @@
     vbox_connector = to_vbox_connector(connector);
     vbox = connector->dev->dev_private;
+    /* Heuristic: we do not want to tell the host that we support dynamic
+     * resizing unless we feel confident that the user space client using
+     * the video driver can handle hot-plug events.  So the first time modes
+     * are queried after a "master" switch we tell the host that we do not,
+     * and immediately after we send the client a hot-plug notification as
+     * a test to see if they will respond and query again.
+     * That is also the reason why capabilities are reported to the host at
+     * this place in the code rather than elsewhere.
+     * We need to report the flags location before reporting the IRQ
+     * capability. */
+    VBoxHGSMIReportFlagsLocation(&vbox->submit_info,   vbox->vram_map_start
+                                                     + vbox->host_flags_offset);
     if (vbox_connector->vbox_crtc->crtc_id == 0)
-        vbox_enable_caps(vbox);
+        vbox_report_caps(vbox);
     if (!vbox->initial_mode_queried) {
         if (vbox_connector->vbox_crtc->crtc_id == 0) {