Go to file
Zoltán Böszörményi 42aaf37241 Fix modesetting device matching through kmsdev device path
xf86platformProbeDev didn't check the device path, fix it.

This is a problem when trying to set up a non-PCI device via
explicit xorg.conf.d configuration.

An USB DisplayLink device, being non-PCI was always set up
as a GPU device assigned to screen 0 instead of a regular
framebuffer, potentially having its own dedicated screen,
despite such configuration as below. Only the relevant parts
of the configuration are quoted, it's part of a larger context
with an Intel chip that has 3 outputs:
* DP1 connected to an LCD panel,
* VGA1 connected to an external monitor,
* HDMI1 unconnected and having no user visible connector

Section "ServerFlags"
        Option          "AutoBindGPU" "false"
EndSection

...

Section "Device"
        Identifier      "Intel2"
        Driver          "intel"
        BusID           "PCI:0:2:0"
        Screen          2
        Option          "Monitor-HDMI1" "HDMI1"
        Option          "ZaphodHeads" "HDMI1"
EndSection

Section "Device"
        Identifier      "UDL"
        Driver          "modesetting"
        Option          "kmsdev" "/dev/dri/card0"
        #BusID          "usb:0:1.2:1.0"
        Option          "Monitor-DVI-I-1" "DVI-I-1"
        Option          "ShadowFB" "on"
        Option          "DoubleShadow" "on"
EndSection

...

Section "Screen"
        Identifier      "SCREEN2"
        Option          "AutoServerLayout" "on"
        Device          "UDL"
        GPUDevice       "Intel2"
        Monitor         "Monitor-DVI-I-1"
        SubSection      "Display"
                Modes   "1024x768"
                Depth   24
        EndSubSection
EndSection

Section "ServerLayout"
        Identifier      "LAYOUT"
        Option          "AutoServerLayout" "on"
        Screen          0 "SCREEN"
        Screen          1 "SCREEN1" RightOf "SCREEN"
        Screen          2 "SCREEN2" RightOf "SCREEN1"
EndSection

On the particular machine I was trying to set up an UDL device,
I found the following structure was being used to match
the device to a platform device while I was debugging the issue:

xf86_platform_devices[0] == Intel, /dev/dri/card1, primary platform device
xf86_platform_devices[1] == UDL, /dev/dri/card0

devList[0] == "Intel0", ZaphodHeads: DP1
devList[1] == "Intel1", ZaphodHeads: VGA1
devList[2] == "UDL"
devList[3] == "Intel2", ZaphodHeads: HDMI1 (intended GPU device to UDL)

When xf86platformProbeDev() matched the UDL device, the BusID
check failed in both cases of:
* BusID "usb:0:1.2:1.0" was specified
* Option "kmsdev" "/dev/dri/card0" was specified

As a result, xf86platformProbeDev() went on to call probeSingleDevice()
with xf86_platform_devices[0] and devList[2], resulting in the
UDL device being set up as a GPU device assigned to the first screen
instead of as a framebuffer on the third screen as the configuration
specified.

Checking Option "kmsdev" in code code may be a layering violation.
But the modesetting driver is actually part of the Xorg sources
instead of being an external driver, so he "kmsdev" path knowledge
may be used here.

Signed-off-by: Böszörményi Zoltán <zboszor@pr.hu>
2020-02-12 21:29:52 +00:00
.gitlab-ci gitlab-ci: Install libnvidia-egl-wayland-dev package 2020-01-06 15:52:33 +01:00
Xext os: Don't crash in AttendClient if the client is gone 2019-11-19 10:15:05 -08:00
Xi Xi: return AlreadyGrabbed for key grabs > 255 2019-06-12 16:58:53 +00:00
composite composite: Stop wrapping GetImage/GetSpans 2019-10-30 16:26:01 +00:00
config udev: Fixed NULL pointer argument of strcmp 2019-09-20 05:33:47 +00:00
damageext Use ARRAY_SIZE all over the tree 2017-10-30 13:45:20 -04:00
dbe meson: Distribute more SDK headers 2018-04-02 13:42:08 -04:00
dix os: Don't crash in AttendClient if the client is gone 2019-11-19 10:15:05 -08:00
doc mi: Add a default no-op miSourceValidate 2019-10-30 16:26:01 +00:00
dri3 dri3: Fix XACE access mode for open and get_supported_modifiers 2019-01-16 20:05:43 +00:00
exa Restrict 1x1 pixmap filling optimization to GXcopy 2020-01-27 17:43:33 +00:00
fb fboverlay: move bpp checks above malloc 2018-09-12 12:26:48 +10:00
glamor glamor_egl: Reject OpenGL < 2.1 early on 2020-01-08 15:20:10 +00:00
glx Revert "dri2: Don't make reference to noClientException" 2020-01-28 13:26:41 -05:00
hw Fix modesetting device matching through kmsdev device path 2020-02-12 21:29:52 +00:00
include configure: Define GLAMOR_HAS_EGL_QUERY_DRIVER when available 2020-01-14 12:50:53 -08:00
m4 Add ax_pthread.m4 to m4/ 2016-05-29 19:20:51 -07:00
man dix: Remove -to option to set the default connection timeout 2019-10-30 16:17:04 +00:00
mi misprite: Stop wrapping GetImage/GetSpans 2019-10-30 16:26:01 +00:00
miext mi: Add a default no-op miSourceValidate 2019-10-30 16:26:01 +00:00
os os: Ignore dying client in ResetCurrentRequest 2020-01-29 21:39:53 +00:00
present present/wnmd: Relax assertion on CRTC on abort_vblank() 2019-11-18 18:15:07 +01:00
pseudoramiX Unvalidated lengths 2017-10-10 23:33:34 +02:00
randr randr: auto-bind of GPU is a config change 2020-02-10 16:07:41 +01:00
record os: Don't crash in AttendClient if the client is gone 2019-11-19 10:15:05 -08:00
render dix: Call SourceValidate before GetImage 2019-10-30 16:26:01 +00:00
test tests/misc: Drop redundant screenInfo declaration 2020-01-28 20:35:06 +00:00
xfixes xfixes: Restore monitoring of animated cursors 2018-02-14 13:52:45 -05:00
xkb Fix crash on XkbSetMap 2019-07-01 02:33:26 +02:00
.appveyor.yml Revert "appveyor: Drop building Xorg server" 2019-10-08 18:26:33 +00:00
.dir-locals.el .dir-locals.el: Add missing final newline 2019-10-01 17:05:28 +00:00
.gitignore .gitignore: Add new autotools file 'test-driver' 2014-04-21 13:41:42 -07:00
.gitlab-ci.yml gitlab-ci: Add meson build job with glamor disabled 2020-02-11 15:59:18 +01:00
.travis.yml travis: Add OSX meson build to matrix 2019-05-02 15:42:58 +00:00
COPYING modesetting: Merge modesetting's COPYING into the xserver's. 2014-09-15 12:46:02 -07:00
Makefile.am xfree86: Merge vbe into int10 2019-10-02 10:03:26 -04:00
README.md Update README for gitlab migration 2018-11-25 20:55:11 +00:00
autogen.sh autogen: Set a default subject prefix for patches 2016-02-08 17:41:38 -05:00
configure.ac configure: Define GLAMOR_HAS_EGL_QUERY_DRIVER when available 2020-01-14 12:50:53 -08:00
devbook.am doc: Create a script to filter xmlto output 2015-01-05 14:24:06 -08:00
docbook.am docbook.am: embed css styles inside the HTML HEAD element 2011-09-21 14:07:49 -07:00
manpages.am man: Fix automake seddery 2018-05-08 12:15:30 -04:00
meson.build meson: add prefix to serverconfigdir 2020-01-15 17:36:06 +00:00
meson_options.txt meson: Add sha1 library options 2019-10-30 16:58:39 +00:00
xorg-server.m4 xorg-server.m4: just all cflags instead of just sdkdir 2018-09-20 20:12:24 +01:00
xorg-server.pc.in xfree86: link modules against Xorg symbols on Cygwin 2012-04-05 21:57:07 -05:00
xserver.ent.in doc: relocate xserver.ent in the package root directory 2011-05-14 11:22:26 -07:00

README.md

X Server

The X server accepts requests from client applications to create windows, which are (normally rectangular) "virtual screens" that the client program can draw into.

Windows are then composed on the actual screen by the X server (or by a separate composite manager) as directed by the window manager, which usually communicates with the user via graphical controls such as buttons and draggable titlebars and borders.

For a comprehensive overview of X Server and X Window System, consult the following article: https://en.wikipedia.org/wiki/X_server

All questions regarding this software should be directed at the Xorg mailing list:

https://lists.freedesktop.org/mailman/listinfo/xorg

The master development code repository can be found at:

https://gitlab.freedesktop.org/xorg/xserver

For patch submission instructions, see:

https://www.x.org/wiki/Development/Documentation/SubmittingPatches

As with other projects hosted on freedesktop.org, X.Org follows its Code of Conduct, based on the Contributor Covenant. Please conduct yourself in a respectful and civilized manner when using the above mailing lists, bug trackers, etc:

https://www.freedesktop.org/wiki/CodeOfConduct