On Wed, Nov 28, 2018 at 02:38:10PM +0100, Erik Skultety wrote: > Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1628892. > The problem is that we didn't put the DRI device into the namespace for QEMU to > access, but that was only a part of the issue. The other part of the issue is > that QEMU doesn't support specifying 'rendernode' for egl-headless yet (patches > are already merged upstream for 3.1) Instead, QEMU's been autoselecting the > DRI device on its own. There's no compelling reason for libvirt not doing that > instead and thus prevent any permission-related issues. > > Since v1: > - updated capabilities to 3.1.0-rc2 containing the necessary QEMU patches > - provided more test cases as requested > - added a new XML sub-element <gl> for egl-headless graphics type > > Erik Skultety (13): > util: Introduce virHostGetDRMRenderNode helper > conf: Introduce virDomainGraphics-related helpers > qemu: process: spice: Pick the first available DRM render node > qemu: command: Introduce qemuBuildGraphicsEGLHeadlessCommandLine > helper > qemu: caps: Introduce QEMU_EGL_HEADLESS_RENDERNODE capability > squash into rendernode capability patch > conf: gfx: Add egl-headless as a member to virDomainGraphicsDef struct > conf: gfx: egl-headless: Introduce a new <gl> subelement > qemu: process: gfx: egl-headless: Pick the first available DRM node > qemu: domain: egl-headless: Add the DRI device into the namespace > qemu: cgroup: gfx: egl-headless: Add the DRI device into the cgroup > list > security: dac: gfx: egl-headless: Relabel the DRI device > qemu: command: gfx: egl-headless: Add 'rendernode' option to the > cmdline You can ignore this version, I made a small changes in couple of the patches and I need to rebase on to the 3.1.0 QEMU caps anyway and I also converted the tests to use DO_TEST_CAPS_LATEST. I'll re-spin in a while. Erik -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
