Changeset 49877 in vbox for trunk/src/VBox/Additions/x11/vboxvideo

Timestamp:

Dec 11, 2013 4:27:39 PM (11 years ago)

Author:

vboxsync

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

91244

Message:

Additions/x11: make vboxvideo dynamic resizing compatible with what the KMS/modesetting drivers can do.

Location:

trunk/src/VBox/Additions/x11/vboxvideo

Files:

• Makefile.kmk (modified) (2 diffs)
• setmode.c (modified) (1 diff)
• vboxvideo.c (modified) (6 diffs)

trunk/src/VBox/Additions/x11/vboxvideo/Makefile.kmk

@@ -30 +30 @@
 ifeq ($(KBUILD_TARGET),solaris) # don't use .solaris or anything here.
  vboxvideo_70_DEFS += __EXTENSIONS__  ## @todo Why this?
+endif
+ifdef VBOX_WITH_GUEST_KMS_DRIVER
+ vboxvideo_70_DEFS += VBOX_WITH_GUEST_KMS_DRIVER
 endif
 vboxvideo_13_DEFS := $(vboxvideo_70_DEFS) VBOXVIDEO_13
@@ -77 +80 @@
         IN_MODULE XFree86Module IN_XF86_MODULE IN_RT_STATIC
 vboxvideo_drv_DEFS += memset=xf86memset memcpy=xf86memcpy
+ifdef VBOX_WITH_GUEST_KMS_DRIVER
+ vboxvideo_drv_DEFS += VBOX_WITH_GUEST_KMS_DRIVER
+endif
 vboxvideo_drv_INCS = \
         $(VBOX_PATH_X11_XFREE_4_3)/include \

trunk/src/VBox/Additions/x11/vboxvideo/setmode.c

@@ -122 +122 @@
                                   vboxBPP(pScrn), 0, x, y);
     /* Tell the host we support graphics */
+#ifndef VBOX_WITH_GUEST_KMS_DRIVER
     if (vbox_device_available(pVBox))
         vboxEnableGraphicsCap(pVBox);
+#endif
     if (pVBox->fHaveHGSMI)
     {

trunk/src/VBox/Additions/x11/vboxvideo/vboxvideo.c

@@ -520 +520 @@
     ScrnInfoPtr pScrn = output->scrn;
     VBOXPtr pVBox = VBOXGetRec(pScrn);
-    TRACE_LOG("property=%d, value->type=%d, value->format=%d, value->size=%ld\n",
-              (int)property, (int)value->type, value->format, value->size);
+    TRACE_LOG("property=%d, value->type=%d, value->format=%d, value->size=%d\n",
+              (int)property, (int)value->type, (int)value->format, (int)value->size);
     if (property == vboxAtomVBoxMode())
     {
         uint32_t cDisplay = (uintptr_t)output->driver_private;
-        char sz[256] = { 0 };
         int w, h;
 
-        if (   value->type != XA_STRING
-            || (unsigned) value->size > (sizeof(sz) - 1))
+        if (   value->type != XA_INTEGER
+            || value->format != 32
+            || value->size != 1)
             return FALSE;
-        strncpy(sz, value->data, value->size);
-        TRACE_LOG("screen=%u, property value=%s\n", cDisplay, sz);
-        if (sscanf(sz, "%dx%d", &w, &h) != 2)
-            return FALSE;
+        w = (*(uint32_t *)value->data) >> 16;
+        h = (*(uint32_t *)value->data) & 0xffff;
+        TRACE_LOG("screen=%u, property value=%dx%d\n", cDisplay, w, h);
         pVBox->aPreferredSize[cDisplay].cx = w;
         pVBox->aPreferredSize[cDisplay].cy = h;
@@ -1054 +1053 @@
     if (vbox_open (pScrn, pScreen, pVBox)) {
         vboxEnableVbva(pScrn);
+#ifndef VBOX_WITH_GUEST_KMS_DRIVER
         vboxEnableGraphicsCap(pVBox);
+#endif
     }
 
@@ -1073 +1074 @@
 
             /* Set up our virtual outputs. */
-            snprintf(szOutput, sizeof(szOutput), "VBOX%u", i);
+            snprintf(szOutput, sizeof(szOutput), "VGA-%u", i);
             pVBox->paOutputs[i] = xf86OutputCreate(pScrn, &VBOXOutputFuncs,
                                                    szOutput);
@@ -1112 +1113 @@
         for (i = 0; i < pVBox->cScreens; ++i)
         {
-            char csz[] = "0x0";
+            INT32 value = 0;
             RRChangeOutputProperty(pVBox->paOutputs[i]->randr_output,
-                                   vboxAtomVBoxMode(), XA_STRING, 8,
-                                   PropModeReplace, sizeof(csz), csz, TRUE,
+                                   vboxAtomVBoxMode(), XA_INTEGER, 32,
+                                   PropModeReplace, 1, &value, TRUE,
                                    FALSE);
 
@@ -1222 +1223 @@
         vboxDisableVbva(pScrn);
     vboxClearVRAM(pScrn, 0, 0);
+#ifndef VBOX_WITH_GUEST_KMS_DRIVER
     vboxDisableGraphicsCap(pVBox);
+#endif
 #ifdef VBOX_DRI_OLD
     if (pVBox->useDRI)
@@ -1245 +1248 @@
         if (pVBox->fHaveHGSMI)
             vboxDisableVbva(pScrn);
+#ifndef VBOX_WITH_GUEST_KMS_DRIVER
         if (pScrn->vtSema)
             vboxDisableGraphicsCap(pVBox);
+#endif
         vboxClearVRAM(pScrn, 0, 0);
     }