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@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list