Re: [libvirt PATCH v3 00/16] Add QEMU "-display dbus" support

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

 



On Fri, Dec 31, 2021 at 1:27 AM Marc-André Lureau
<marcandre.lureau@xxxxxxxxxx> wrote:
>
> Hi Neal
>
> On Fri, Dec 31, 2021 at 6:09 AM Neal Gompa <ngompa13@xxxxxxxxx> wrote:
> >
> > On Wed, Dec 22, 2021 at 2:44 PM <marcandre.lureau@xxxxxxxxxx> wrote:
> > >
> > > From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> > >
> > > Hi,
> > >
> > > This series implements supports for the uQEMU "-display dbus" support, that
> > > landed earlier this week for 7.0.
> > >
> > > By default, libvirt will start a private VM bus (sharing and reusing the
> > > existing "vmstate" VM bus & code).
> > >
> > > The feature set should cover the needs to replace Spice as local client of choice,
> > > including 3daccel/dmabuf, audio, clipboard sharing, usb redirection, and arbitrary
> > > chardev/channels (for serial etc).
> > >
> > > The test Gtk4 client is also in progress, currently in development at
> > > https://gitlab.com/marcandre.lureau/qemu-display/. A few dependencies, such as
> > > zbus, require an upcoming release. virt-viewer & boxes will need a port to Gtk4
> > > to make use of the shared widget.
> > >
> > > Comments welcome, as we can still adjust the QEMU side etc.
> > >
> > > thanks
> > >
> > > v3: after QEMU 7.0 dev cycle opening and merge
> > >  - rebased
> > >  - add 7.0 x86-64 capabilities (instead of tweaking 6.2)
> > >  - fix version annotations
> > >
> > > Marc-André Lureau (16):
> > >   qemu: add chardev-vdagent capability check
> > >   qemu: add -display dbus capability check
> > >   qemucapabilitiestest: Add x64 test data for the qemu-7.0 development
> > >     cycle
> > >   conf: add <graphics type='dbus'>
> > >   qemu: start the D-Bus daemon for the display
> > >   qemu: add -display dbus support
> > >   virsh: refactor/split cmdDomDisplay()
> > >   virsh: report the D-Bus bus URI for domdisplay
> > >   conf: add <audio type='dbus'> support
> > >   qemu: add audio type 'dbus'
> > >   conf: add dbus <clipboard>
> > >   qemu: add dbus clipboard sharing
> > >   conf: add <serial type='dbus'>
> > >   qemu: add -chardev dbus support
> > >   qemu: add usbredir type 'dbus'
> > >   docs: document <graphics> type dbus
> > >
> > >  NEWS.rst                                      |     7 +-
> > >  docs/formatdomain.rst                         |    43 +-
> > >  docs/schemas/basictypes.rng                   |     7 +
> > >  docs/schemas/domaincommon.rng                 |    71 +
> > >  src/bhyve/bhyve_command.c                     |     1 +
> > >  src/conf/domain_conf.c                        |   141 +-
> > >  src/conf/domain_conf.h                        |    15 +
> > >  src/conf/domain_validate.c                    |    41 +-
> > >  src/libxl/libxl_conf.c                        |     1 +
> > >  src/qemu/qemu_capabilities.c                  |     8 +
> > >  src/qemu/qemu_capabilities.h                  |     4 +
> > >  src/qemu/qemu_command.c                       |    77 +-
> > >  src/qemu/qemu_domain.c                        |     1 +
> > >  src/qemu/qemu_driver.c                        |    10 +-
> > >  src/qemu/qemu_extdevice.c                     |    13 +
> > >  src/qemu/qemu_hotplug.c                       |     1 +
> > >  src/qemu/qemu_monitor_json.c                  |    10 +
> > >  src/qemu/qemu_process.c                       |    41 +-
> > >  src/qemu/qemu_validate.c                      |    33 +
> > >  src/security/security_dac.c                   |     2 +
> > >  src/vmx/vmx.c                                 |     1 +
> > >  .../domaincapsdata/qemu_7.0.0-q35.x86_64.xml  |   231 +
> > >  .../domaincapsdata/qemu_7.0.0-tcg.x86_64.xml  |   237 +
> > >  tests/domaincapsdata/qemu_7.0.0.x86_64.xml    |   231 +
> > >  .../caps_6.1.0.x86_64.xml                     |     1 +
> > >  .../caps_6.2.0.aarch64.xml                    |     1 +
> > >  .../caps_6.2.0.x86_64.xml                     |     1 +
> > >  .../caps_7.0.0.x86_64.replies                 | 37335 ++++++++++++++++
> > >  .../caps_7.0.0.x86_64.xml                     |  3720 ++
> > >  .../graphics-dbus-address.args                |    30 +
> > >  .../graphics-dbus-address.xml                 |    35 +
> > >  .../qemuxml2argvdata/graphics-dbus-audio.args |    33 +
> > >  .../qemuxml2argvdata/graphics-dbus-audio.xml  |    45 +
> > >  .../graphics-dbus-chardev.args                |    32 +
> > >  .../graphics-dbus-chardev.xml                 |    43 +
> > >  .../graphics-dbus-clipboard.args              |    31 +
> > >  .../graphics-dbus-clipboard.xml               |    35 +
> > >  tests/qemuxml2argvdata/graphics-dbus-p2p.args |    30 +
> > >  tests/qemuxml2argvdata/graphics-dbus-p2p.xml  |    33 +
> > >  .../graphics-dbus-usbredir.args               |    34 +
> > >  .../graphics-dbus-usbredir.xml                |    30 +
> > >  tests/qemuxml2argvdata/graphics-dbus.args     |    30 +
> > >  tests/qemuxml2argvdata/graphics-dbus.xml      |    33 +
> > >  tests/qemuxml2argvtest.c                      |    22 +
> > >  .../graphics-dbus-address.xml                 |     1 +
> > >  .../graphics-dbus-audio.xml                   |     1 +
> > >  .../graphics-dbus-chardev.xml                 |     1 +
> > >  .../graphics-dbus-clipboard.xml               |     1 +
> > >  .../qemuxml2xmloutdata/graphics-dbus-p2p.xml  |     1 +
> > >  tests/qemuxml2xmloutdata/graphics-dbus.xml    |     1 +
> > >  tests/qemuxml2xmltest.c                       |    20 +
> > >  tools/virsh-domain.c                          |   366 +-
> > >  52 files changed, 42981 insertions(+), 192 deletions(-)
> > >  create mode 100644 tests/domaincapsdata/qemu_7.0.0-q35.x86_64.xml
> > >  create mode 100644 tests/domaincapsdata/qemu_7.0.0-tcg.x86_64.xml
> > >  create mode 100644 tests/domaincapsdata/qemu_7.0.0.x86_64.xml
> > >  create mode 100644 tests/qemucapabilitiesdata/caps_7.0.0.x86_64.replies
> > >  create mode 100644 tests/qemucapabilitiesdata/caps_7.0.0.x86_64.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-address.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-address.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-audio.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-audio.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-chardev.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-chardev.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-clipboard.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-clipboard.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-p2p.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-p2p.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-usbredir.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus-usbredir.xml
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus.args
> > >  create mode 100644 tests/qemuxml2argvdata/graphics-dbus.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus-address.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus-audio.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus-chardev.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus-clipboard.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus-p2p.xml
> > >  create mode 120000 tests/qemuxml2xmloutdata/graphics-dbus.xml
> > >
> > > --
> > > 2.34.1.8.g35151cf07204
> > >
> > >
> >
> > How will this work for the web (cockpit-machines, oVirt/RHV, and
> > KubeVirt/OpenShift Virtualization)?
>
> For remoting, an external VNC/Spice/RDP server should export the display.
>
> A simple VNC server is implemented here
> https://gitlab.com/marcandre.lureau/qemu-display/-/tree/master/qemu-vnc.
>
> I plan to work on an RDP server at some point.
>

You might want to look at some of the prior art for QEMU RDP
support[1][2]. It's probably not terribly helpful if you've got a
different approach now, but at least there was something in the past.

[1]: https://pagure.io/virt-with-rdp
[2]: https://github.com/datto/rdpmux

> >
> > Personally, I use virt-manager/virt-viewer *regularly*, so where's the
> > plan to have this supported there? What about QVirtManager? And so on?
>
> I have been working on a common gtk4 remote display widget which
> various applications can use (https://gitlab.gnome.org/malureau/rdw).
> Similarly for Qt apps, they will have to implement their own widget.
>

Hmm, okay...


-- 
真実はいつも一つ!/ Always, there's only one truth!





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux