[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2 00/13] Autoselect a DRM node for egl-headless and add it to cmdline



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]