[PATCH driver/mga] Disable HW cursor by default on G200 server chips

Egbert Eich eich at freedesktop.org
Mon Jul 29 04:02:21 PDT 2013


Disable HW cursor by default on G200 server chips as these chips a
re often used with a remote graphics link which cannot display
the HW cursor.
This can be overridden by a config option.
Most desktops today use ARGB cursors anyhow which are not
supported by this driver anyhow. Thus the performance penalty
should be irrelevant.

Signed-off-by: Egbert Eich <eich at freedesktop.org>
---
 src/mga_driver.c | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

diff --git a/src/mga_driver.c b/src/mga_driver.c
index 94faa07..b6760dd 100644
--- a/src/mga_driver.c
+++ b/src/mga_driver.c
@@ -1679,7 +1679,27 @@ MGAPreInit(ScrnInfoPtr pScrn, int flags)
 	}
     }
 
-    pMga->HWCursor = TRUE;
+    /*
+     * Disable HW cursor by default on G200 server chips as these
+     * chips are often used with a remote graphics link which cannot
+     * display the HW cursor.
+     */
+    switch (pMga->Chipset) {
+    case PCI_CHIP_MGAG200_SE_A_PCI:
+    case PCI_CHIP_MGAG200_SE_B_PCI:
+    case PCI_CHIP_MGAG200_EV_PCI:
+    case PCI_CHIP_MGAG200_ER_PCI:
+    case PCI_CHIP_MGAG200_WINBOND_PCI:
+    case PCI_CHIP_MGAG200_EH_PCI:
+	pMga->HWCursor = FALSE;
+	xf86DrvMsg(pScrn->scrnIndex, X_INFO,
+		   "HW cursor is not supported with video redirection on"
+		   "G200 server chips.\n If you don't intend to use video "
+		   "redirection enable with Option \"HWCursor\" \"On\"\n");
+	break;
+    default:
+	pMga->HWCursor = TRUE;
+    }
     from = X_DEFAULT;
 
     /*
-- 
1.8.1.4



More information about the xorg-devel mailing list