Bug 1097

Created attachment 650 [details] [review]
Patch to add pseudo-xinerama support to the mga driver

Created attachment 651 [details] [review]
Patch to add pseudo-xinerama support to the mga driver

Added the wrong file before, this should be the right one...

Erm... don't want to appear picky but that code was actually written by me. Alex
took it (including the comments) from the SiS driver.

Sorry, didn't want to credit the wrong person. Thank you for the initial
implementation, only that makes MergedFB really usable.

Created attachment 4138 [details] [review]
mga-pseudoxinerama-1.patch

rediff against modular.

adding patch keyword. Also has this patch been merged to trunk?

Unfortunately, in xorg 7.2 the patch no longer works. Could someone 
knowledgeable adapt it? It was quite useful (and stable, used it for over a 
year without problems!)

Created attachment 8216 [details] [review]
Rediff against current git repository

Rediff against current code from git repository. Created by me without any real
knowledge of X11, just adapted the patch to fit the new code so use with care!

One thing though: Some files were missing includes to "config.h", this could be
a bug on its own.

To use this patch, one has to manually insert "#define USEMGAHAL" in config.h
after running autogen.sh and configure, as I have no idea of autoconf and how
to adapt configure.ac (or how to do this properly another way :)

After your review, i suggest to include pseudo-xinerama permanently in HEAD,
the patch looks quite non-intrusive to me (that is, a complete xorg-outsider ;)

Getting the HAL module working with our current driver (as opposed to the stuff
Matrox ships) is a PITA anyway -- so that's why we this bug isn't really of high
priority to me.

IMO this patch can go in once we have figured out how to mergedfb on our own.

Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.

MergedFB is dead, we have working RandR support now.

(In reply to comment #11)
> MergedFB is dead, we have working RandR support now.

Can you please add a pointer to a demo configuration using randr with dual head setup working?

I would use RandR with joy, just it doesn't work with a G550 - this patch always has.

Using X 7.3-96.1 RandR reports only one output:

> xrandr
Screen 0: minimum 640 x 350, current 1280 x 1024, maximum 2560 x 1024
default connected 1280x1024+0+0 0mm x 0mm
   1280x1024      85.0*    75.0     60.0  
   1280x960       85.0     60.0  
   1152x864       85.0     75.0  
   1152x768       55.0  
   1024x768       85.0     75.0     70.0     60.0  
   832x624        75.0  
   960x529        75.0  
   800x600        85.0     75.0     72.0     60.0     56.0  
   640x480        85.0     75.0     73.0     60.0  
   720x400        88.0     70.0     85.0  
   640x400        85.0  
   640x350        85.0  
   2560x1024      85.0

For xorg.conf, see below.

So for me this bug is still open, as I can not get accelerated dual-head on my G550 in any way.

Any chance that this could be fixed on the MergedFB route? If not, should I open a RandR bug for my G550 setup?

xorg.conf:

Section "Files"
  FontPath     "/usr/share/fonts/misc:unscaled"
  FontPath     "/usr/share/fonts/local"
  FontPath     "/usr/share/fonts/75dpi:unscaled"
  FontPath     "/usr/share/fonts/100dpi:unscaled"
  FontPath     "/usr/share/fonts/Type1"
  FontPath     "/usr/share/fonts/URW"
  FontPath     "/usr/share/fonts/Speedo"
  FontPath     "/usr/share/fonts/PEX"
  FontPath     "/usr/share/fonts/cyrillic"
  FontPath     "/usr/share/fonts/latin2/misc:unscaled"
  FontPath     "/usr/share/fonts/latin2/75dpi:unscaled"
  FontPath     "/usr/share/fonts/latin2/100dpi:unscaled"
  FontPath     "/usr/share/fonts/latin2/Type1"
  FontPath     "/usr/share/fonts/latin7/75dpi:unscaled"
  FontPath     "/usr/share/fonts/baekmuk:unscaled"
  FontPath     "/usr/share/fonts/japanese:unscaled"
  FontPath     "/usr/share/fonts/kwintv"
  FontPath     "/usr/share/fonts/truetype"
  FontPath     "/usr/share/fonts/uni:unscaled"
  FontPath     "/usr/share/fonts/CID"
  FontPath     "/usr/share/fonts/ucs/misc:unscaled"
  FontPath     "/usr/share/fonts/ucs/75dpi:unscaled"
  FontPath     "/usr/share/fonts/ucs/100dpi:unscaled"
  FontPath     "/usr/share/fonts/hellas/misc:unscaled"
  FontPath     "/usr/share/fonts/hellas/75dpi:unscaled"
  FontPath     "/usr/share/fonts/hellas/100dpi:unscaled"
  FontPath     "/usr/share/fonts/hellas/Type1"
  FontPath     "/usr/share/fonts/misc/sgi:unscaled"
  FontPath     "/usr/share/fonts/xtest"
  FontPath     "/opt/kde3/share/fonts"
  InputDevices "/dev/gpmdata"
  InputDevices "/dev/input/mice"
EndSection

Section "ServerFlags"
  Option       "AllowMouseOpenFail" "on"
  Option       "ZapWarning" "on"
EndSection

Section "Module"
  Load         "dbe"
  Load         "type1"
  Load         "freetype"
  Load         "extmod"
  Load         "glx"
  Load         "dri"
EndSection

Section "InputDevice"
  Driver       "kbd"
  Identifier   "Keyboard[0]"
  Option       "Protocol" "Standard"
  Option       "XkbLayout" "de"
  Option       "XkbModel" "microsoftpro"
  Option       "XkbRules" "xfree86"
  Option       "XkbVariant" "nodeadkeys"
EndSection

Section "InputDevice"
  Driver       "mouse"
  Identifier   "Mouse[1]"
  Option       "Buttons" "9"
  Option       "Device" "/dev/input/mice"
  Option       "Name" "ImExPS/2 Logitech Wheel Mouse"
  Option       "Protocol" "explorerps/2"
  Option       "Vendor" "Sysp"
  Option       "ZAxisMapping" "4 5"
EndSection

Section "Monitor"
  Identifier   "Monitor[0]"
  VendorName   "IBM"
  ModelName    "IBM P260"
  DisplaySize  388 291
  Option       "DPMS"
  Option       "PreferredMode" "1280x1024"
EndSection

Section "Screen"
  Identifier   "Screen[0]"
  Device       "Device[0]"
  Monitor      "Monitor[0]"
  DefaultDepth 24
  SubSection "Display"
    Depth      24
    Virtual    2560 1024
  EndSubSection
EndSection

Section "Device"
  BoardName    "G550"
  BusID        "1:0:0"
  Driver       "mga"
  Identifier   "Device[0]"
  VendorName   "Matrox"
EndSection

Section "ServerLayout"
  Identifier   "Layout[all]"
  InputDevice  "Keyboard[0]" "CoreKeyboard"
  InputDevice  "Mouse[1]" "CorePointer"
  Screen       "Screen[0]"
EndSection

Section "DRI"
    Group      "video"
    Mode       0660
EndSection

Section "Extensions"
EndSection

Over to ajax since I think he's the only xorg dev with hardware for this.

The comments indicate that the patch no longer applies.  If someone rebases it 
against the current driver, please send it to xorg-devel for review and comment 
in this ticket.  If this bug doesn't get any motion in the next month or so, 
I'll probably close it as NTBF.  Thanks.

I don't intend to fix this, short of porting mga to new enough randr where you get xinerama protocol for free.