Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • xserver xserver
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 931
    • Issues 931
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 99
    • Merge requests 99
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • xorg
  • xserverxserver
  • Issues
  • #466

Closed
Open
Created Dec 27, 2014 by Bugzilla Migration User@bugzilla-migration

xfree86: drm device access denied to armsoc driver due to systemd-logind (patch drafted)

Submitted by Alexei Colin

Assigned to Xorg Project Team

Link to original bug (#87762)

Description

Created attachment 111383 Patch (commit 1 of 2): factor out for reuse a function to lookup device by bus ID

Symptom: startx (and xinit) from a virtual console (as root) fails to start the X server with the following error:

(EE) ARMSOC(0): ERROR: Cannot set the DRM interface version. (EE) Screen(s) found, but none have a usable configuration. Fatal server error: (EE) no screens found(EE)

This is with xf86-video-armsoc (both Arch package xf86-video-armsoc-git 1:219.f16b5c8-2 and xf86-video-armsoc-odroid 237.3bdf799-1) on Linux 3.8.13.28-3-ARCH armv7h on Samsung Exynos 4412 on Hardkernel Odroid U3 board. However, the problem is not strictly platform-specific (see below).

The problem: kernel drm module debug output (drm.debug=1 on kernel command line) shows that set version ioctl (nr=0x07) to the drm device from the armsoc driver fails with EACCESS (-13), see attached kernel log. This is because the armsoc driver's device file descriptor does not have the 'master' priviledge. The server opens the /dev/dri/card0 device through systemd-logind, which sets the server's file descriptor to be the master via ioctl nr=0x1e from systemd-logind process. Later, the armsoc driver opens the device, but its file descriptor is not master because the server's descriptor is still open and is the master.

The device is opened via systemd-logind by the server during bus probing without regard to whether the driver SUPPORTS_SERVER_FD, i.e. does not explicitly open the device file but fetches the descriptor via an attribute initialized by the server. However, (only) if a non-SERVER_FD driver does implement the 'platformProbe', then the server releases the file descriptor through the systemd-logind before further probing in the driver. The armsoc driver does not implement 'platformProbe' and relies on fallback to 'Probe', the path on which the above release logic does not take place.

In summary, this problem seems to affect all drivers, like armsoc, that (1) do not SUPPORTS_SERVER_FD, and (2) do not implement the 'platformProbe' method but only the older 'Probe' method.

Potential solutions (both tested and make startx succeed): (A) A workaround is to build the server with './configure --disable-systemd-logind'. (B) Attached patchset moves the descriptor release logic slightly up the stack so that it applies to both drivers with 'platformProbe' and with 'Probe' instead of only to the former. Tested against recent xorg/xserver master 3b5be33f.

Details on patch:

Original callchain: InitOutput xf86BusProbe xf86platformProbe xf86PlatformDeviceProbe get_drm_info systemd_logind_take_fd xf86BusConfig xf86CallDriverProbe platformProbeDev [if drv->platformProbe exists] probeSingleDevice
doPlatformProbe systemd_logind_release_fd [if not SUPPORTS_SERVER_FD] drv->platformProbe drv->Probe [if drv->platformProbe does not exist]

Patched callchain: ... xf86CallDriverProbe systemd_logind_release_fd [if not SUPPORTS_SERVER_FD] platformProbeDev [if drv->platformProbe exists] probeSingleDevice
doPlatformProbe drv->platformProbe drv->Probe [if drv->platformProbe does not exist]

Attached is an Arch PKGBUILD for building the package and testing the patch (derived from https://projects.archlinux.org/svntogit/packages.git/tree/trunk?h=packages/xorg-server): makepkg -si PKGBUILD --asroot

Sidenotes:

(1) This problem is not fatal to display managers (LightDM), because the request for the file descriptor from the systemd-logind never succeeds in the first place, because association with a logind session fails: '(EE) systemd-logind: failed to get session: PID 8464 does not belong to any known session' in X server log.

(2) This problem happens to not manifest (as severely) if server is started on a virtual console different from the current session where startx is run, e.g. when running 'startx -- :0 vt3' on vt2. This is a sideeffect: the VT switch event triggers systemd-logind to 'pause' the drm device, which involves droping master, just in time before the armsoc driver needs it. The server does start but it ends up outside of an active logind session (the vt2 session goes from 'active' to 'online' state, and there is no other session created). As a result, input devices don't work. Running xserver in a non-controlling VT seems to run into separate issues (Bug 81932), in the present bug report the concern is with running it in the controlling VT.

(3) This problem affects LibDRM: the /dev/dri/card0 device is wrongly determined to be in use, because drmGetBusid returns a non-zero value, and is skipped, preventing server startup. See attached debug output from 'export LIBGL_DEBUG=verbose; startx'. To debug the original issue, libDRM was patched to override this 'in use' decision. This patch is not necessary once the underlying issue is fixed in the server.

Attachment 111383, "Patch (commit 1 of 2): factor out for reuse a function to lookup device by bus ID":
0001-xfree86-a-helper-func-for-finding-device-by-bus-ID.patch

Version: git

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking