[libvirt PATCH v2 6/6] Move src/keycodemapdb -> subprojects/keycodemapdb

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau Follow better meson build system conventions. This allows to find keymap-gen or CSV without explicitly setting the paths. Signed-off-by: Marc-André Lureau --- .gitmodules | 2 +- meson.build | 1 + src/util/meson.build

[libvirt PATCH v2 5/6] rpc/ssh: ssh_userauth_agent() is not supported on win32

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau The function does not exist on win32. Signed-off-by: Marc-André Lureau Signed-off-by: Michal Privoznik Reviewed-by: Michal Privoznik --- src/rpc/virnetlibsshsession.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git

[libvirt PATCH v2 4/6] meson: drop explicit python interpreter

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau meson wraps python scripts already on win32, so we end up with these failing commands: [1/359] "C:/msys64/ucrt64/bin/meson" "--internal" "exe" "--capture" "src/util/virkeycodetable_atset1.h" "--" "sh" "C:/msys64/home/marca/src/libvirt/scripts/meson-python.sh"

[libvirt PATCH v2 3/6] meson: don't hard code find_program() location

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau Use 'dirs' argument to locate the program. Signed-off-by: Marc-André Lureau --- src/util/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/meson.build b/src/util/meson.build index 2fe6f7699e..f055079d0b 100644 ---

[libvirt PATCH v2 2/6] keycodemapdb: update submodule for build fixes

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau A number of changes, but notably python script shebang fixing build issues in CI: Daniel P. Berrangé (1): Revert "Add local argparse for compat with python 2.6" Dawid Dziurla (1): Don't hardcode python3 path in shebang Eli Schwartz (1): make the

[libvirt PATCH v2 1/6] meson: don't look for unix paths on win32

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau Or meson will complain with: ../meson.build:770:2: ERROR: Search directory /sbin is not an absolute path. Signed-off-by: Marc-André Lureau Reviewed-by: Michal Privoznik --- meson.build | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git

[libvirt PATCH v2 0/6] Fix compilation on msys2, update keycodemapdb

2023-04-17 Thread marcandre . lureau
From: Marc-André Lureau Hi, libvirt fails to compile on msys2/win32. Also update keycodemapdb to fix CI build issues, and turn it into a subproject(), as that makes things simpler for meson. Fixes: https://gitlab.com/libvirt/libvirt/-/issues/453 v2: - update based on Michal version -

[libvirt PATCH 3/3] rpc/ssh: ssh_userauth_agent() is not supported on win32

2023-04-06 Thread marcandre . lureau
From: Marc-André Lureau The function does not exist on win32. Signed-off-by: Marc-André Lureau --- src/rpc/virnetlibsshsession.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/rpc/virnetlibsshsession.c b/src/rpc/virnetlibsshsession.c index

[libvirt PATCH 2/3] meson: drop explicit python interpreter

2023-04-06 Thread marcandre . lureau
From: Marc-André Lureau meson wraps python scripts already on win32, so we end up with these failing commands: [1/359] "C:/msys64/ucrt64/bin/meson" "--internal" "exe" "--capture" "src/util/virkeycodetable_atset1.h" "--" "sh" "C:/msys64/home/marca/src/libvirt/scripts/meson-python.sh"

[libvirt PATCH 1/3] meson: don't look for unix paths on win32

2023-04-06 Thread marcandre . lureau
From: Marc-André Lureau Or meson will complain with: ../meson.build:770:2: ERROR: Search directory /sbin is not an absolute path. Signed-off-by: Marc-André Lureau --- meson.build | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/meson.build b/meson.build index

[libvirt PATCH 0/3] RFC: fix compilation on msys2

2023-04-06 Thread marcandre . lureau
From: Marc-André Lureau Hi, libvirt fails to compile on msys2/win32. Here is a few patches that solve it, but the way python scripts are being handled in general is a bit odd, so this is RFC. (fwiw, a lot of tests fail though, I can send the log if anyone is interested - or attach it to a

[PATCH] RFC: char: deprecate usage of bidirectional pipe

2022-07-26 Thread marcandre . lureau
From: Marc-André Lureau As Ed Swierk explained back in 2006: https://lists.nongnu.org/archive/html/qemu-devel/2006-12/msg00160.html "When qemu writes into the pipe, it immediately reads back what it just wrote and treats it as a monitor command, endlessly breathing its own exhaust." This is

[libvirt PATCH v4 04/11] qemu: add -display dbus support

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau By default, libvirt will start a private bus and tell QEMU to connect to it. Instead, a D-Bus "address" to connect to can be specified, or the p2p mode enabled. D-Bus display works best with GL & a rendernode, which can be specified with child element. Signed-off-by:

[libvirt PATCH v4 11/11] docs: document type dbus

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- NEWS.rst | 5 + docs/formatdomain.rst | 43 ++- 2 files changed, 47 insertions(+), 1 deletion(-) diff --git a/NEWS.rst b/NEWS.rst index 7903449f9b..b32fd26167 100644 ---

[libvirt PATCH v4 10/11] qemu: add usbredir type 'dbus'

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau The USB device redirection works in a similar way as Spice. The underlying 'dbus' channel is set to "org.qemu.usbredir" by default for the client to identify the channel purpose (as specified in -display dbus documentation). Signed-off-by: Marc-André Lureau ---

[libvirt PATCH v4 01/11] qemu: add -display dbus capability check

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_7.0.0.x86_64.xml | 1 + tests/qemucapabilitiesdata/caps_7.1.0.x86_64.xml | 1 + 4 files

[libvirt PATCH v4 09/11] qemu: add -chardev dbus support

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 4 +++ src/qemu/qemu_monitor_json.c | 9 ++ .../graphics-dbus-chardev.args| 32 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v4 08/11] conf: add

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Like a Spice port, a dbus serial must specify an associated channel name. Signed-off-by: Marc-André Lureau --- src/conf/domain_conf.c| 25 +++ src/conf/domain_conf.h| 4 ++ src/conf/domain_validate.c

[libvirt PATCH v4 07/11] qemu: add audio type 'dbus'

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 12 +-- src/qemu/qemu_validate.c | 30 - .../qemuxml2argvdata/graphics-dbus-audio.args | 33 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v4 05/11] virsh: report the D-Bus bus URI for domdisplay

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau This implementation reports only Unix bus address using the URI format proposed in https://gitlab.freedesktop.org/dbus/dbus/-/issues/348. We prefer a URI form over the D-Bus address form, since all other display protocols use a URI, allowing to distinguish between

[libvirt PATCH v4 06/11] conf: add support

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/bhyve/bhyve_command.c | 1 + src/conf/domain_conf.c| 34 +++--- src/conf/domain_conf.h| 2 + src/conf/schemas/domaincommon.rng | 29

[libvirt PATCH v4 02/11] conf: add

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/conf/domain_conf.c| 78 ++- src/conf/domain_conf.h| 8 ++ src/conf/domain_validate.c| 9 ++- src/conf/schemas/basictypes.rng | 7

[libvirt PATCH v4 03/11] qemu: start the D-Bus daemon for the display

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Start the daemon if necessary (it is already stopped in qemuProcessStop) Signed-off-by: Marc-André Lureau --- src/qemu/qemu_extdevice.c | 13 + 1 file changed, 13 insertions(+) diff --git a/src/qemu/qemu_extdevice.c b/src/qemu/qemu_extdevice.c index

[libvirt PATCH v4 00/11] Add QEMU "-display dbus" support

2022-05-13 Thread marcandre . lureau
From: Marc-André Lureau Hi, This series implements supports for the QEMU "-display dbus" support, available since 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

[libvirt PATCH v3 16/16] docs: document type dbus

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- NEWS.rst | 7 ++- docs/formatdomain.rst | 43 ++- 2 files changed, 48 insertions(+), 2 deletions(-) diff --git a/NEWS.rst b/NEWS.rst index e7d53167219f..92042459653d 100644

[libvirt PATCH v3 15/16] qemu: add usbredir type 'dbus'

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau The USB device redirection works in a similar way as Spice. The underlying 'dbus' channel is set to "org.qemu.usbredir" by default for the client to identify the channel purpose (as specified in -display dbus documentation). Signed-off-by: Marc-André Lureau ---

[libvirt PATCH v3 14/16] qemu: add -chardev dbus support

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 4 +++ src/qemu/qemu_monitor_json.c | 9 ++ .../graphics-dbus-chardev.args| 32 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v3 13/16] conf: add

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Like a Spice port, a dbus serial must specify an associated channel name. Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 1 + src/conf/domain_conf.c| 26 +++ src/conf/domain_conf.h

[libvirt PATCH v3 12/16] qemu: add dbus clipboard sharing

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau This feature requires to setup a "-chardev qemu-vdagent,clipboard=on". "qemu-vdagent" is not a typical chardev that you can connect to another end, so it doesn't map to a or . Set it up along with "-display dbus" code instead. Signed-off-by: Marc-André Lureau ---

[libvirt PATCH v3 11/16] conf: add dbus

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 8 + src/conf/domain_conf.c| 13 ++- src/conf/domain_conf.h| 1 + .../graphics-dbus-clipboard.xml | 35

[libvirt PATCH v3 06/16] qemu: add -display dbus support

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau By default, libvirt will start a private bus and tell QEMU to connect to it. Instead, a D-Bus "address" to connect to can be specified, or the p2p mode enabled. D-Bus display works best with GL & a rendernode, which can be specified with child element. Signed-off-by:

[libvirt PATCH v3 10/16] qemu: add audio type 'dbus'

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 12 +-- src/qemu/qemu_validate.c | 30 - .../qemuxml2argvdata/graphics-dbus-audio.args | 33 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v3 09/16] conf: add support

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 29 src/bhyve/bhyve_command.c | 1 + src/conf/domain_conf.c| 34 +++--- src/conf/domain_conf.h|

[libvirt PATCH v3 08/16] virsh: report the D-Bus bus URI for domdisplay

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau This implementation reports only Unix bus address using the URI format proposed in https://gitlab.freedesktop.org/dbus/dbus/-/issues/348. We prefer a URI form over the D-Bus address form, since all other display protocols use a URI, allowing to distinguish between

[libvirt PATCH v3 07/16] virsh: refactor/split cmdDomDisplay()

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau The function is large and quite complex. Move the inner loop for each scheme in a separate function cmdDomDisplayScheme(). Signed-off-by: Marc-André Lureau --- tools/virsh-domain.c | 341 +-- 1 file changed, 169 insertions(+),

[libvirt PATCH v3 05/16] qemu: start the D-Bus daemon for the display

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Start the daemon if necessary (it is already stopped in qemuProcessStop) Signed-off-by: Marc-André Lureau --- src/qemu/qemu_extdevice.c | 13 + 1 file changed, 13 insertions(+) diff --git a/src/qemu/qemu_extdevice.c b/src/qemu/qemu_extdevice.c index

[libvirt PATCH v3 04/16] conf: add

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/basictypes.rng | 7 ++ docs/schemas/domaincommon.rng | 33 src/conf/domain_conf.c| 78 ++- src/conf/domain_conf.h

[libvirt PATCH v3 02/16] qemu: add -display dbus capability check

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 4 src/qemu/qemu_capabilities.h | 3 +++ 2 files changed, 7 insertions(+) diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index 1cf56ace71b2..f6ba51229841 100644 ---

[libvirt PATCH v3 01/16] qemu: add chardev-vdagent capability check

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_6.1.0.x86_64.xml | 1 + tests/qemucapabilitiesdata/caps_6.2.0.aarch64.xml | 1 +

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

2021-12-22 Thread marcandre . lureau
From: Marc-André Lureau 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

[libvirt PATCH v2 09/15] qemu: add audio type 'dbus'

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 12 +-- src/qemu/qemu_validate.c | 30 - .../qemuxml2argvdata/graphics-dbus-audio.args | 33 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v2 08/15] conf: add support

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 29 src/bhyve/bhyve_command.c | 1 + src/conf/domain_conf.c| 34 +++--- src/conf/domain_conf.h|

[libvirt PATCH v2 10/15] conf: add dbus

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 8 + src/conf/domain_conf.c| 13 ++- src/conf/domain_conf.h| 1 + .../graphics-dbus-clipboard.xml | 35

[libvirt PATCH v2 15/15] docs: document type dbus

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/formatdomain.rst | 43 ++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst index eb8c973cf1fc..ac298fbafa7f 100644 ---

[libvirt PATCH v2 14/15] qemu: add usbredir type 'dbus'

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau The USB device redirection works in a similar way as Spice. The underlying 'dbus' channel is set to "org.qemu.usbredir" by default for the client to identify the channel purpose (as specified in -display dbus documentation). Signed-off-by: Marc-André Lureau ---

[libvirt PATCH v2 12/15] conf: add

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Like a Spice port, a dbus serial must specify an associated channel name. Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 1 + src/conf/domain_conf.c| 26 +++ src/conf/domain_conf.h

[libvirt PATCH v2 13/15] qemu: add -chardev dbus support

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_command.c | 8 + src/qemu/qemu_monitor_json.c | 9 ++ .../graphics-dbus-chardev.args| 32 +++ tests/qemuxml2argvtest.c

[libvirt PATCH v2 11/15] qemu: add dbus clipboard sharing

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau This feature requires to setup a "-chardev qemu-vdagent,clipboard=on". "qemu-vdagent" is not a typical chardev that you can connect to another end, so it doesn't map to a or . Set it up along with "-display dbus" code instead. Signed-off-by: Marc-André Lureau ---

[libvirt PATCH v2 07/15] virsh: report the D-Bus bus URI for domdisplay

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau This implementation reports only Unix bus address using the URI format proposed in https://gitlab.freedesktop.org/dbus/dbus/-/issues/348. We prefer a URI form over the D-Bus address form, since all other display protocols use a URI, allowing to distinguish between

[libvirt PATCH v2 06/15] virsh: refactor/split cmdDomDisplay()

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau The function is large and quite complex. Move the inner loop for each scheme in a separate function cmdDomDisplayScheme(). Signed-off-by: Marc-André Lureau --- tools/virsh-domain.c | 341 +-- 1 file changed, 169 insertions(+),

[libvirt PATCH v2 03/15] conf: add

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/basictypes.rng | 7 ++ docs/schemas/domaincommon.rng | 33 src/conf/domain_conf.c| 78 ++- src/conf/domain_conf.h

[libvirt PATCH v2 05/15] qemu: add -display dbus support

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau By default, libvirt will start a private bus and tell QEMU to connect to it. Instead, a D-Bus "address" to connect to can be specified, or the p2p mode enabled. D-Bus display works best with GL & a rendernode, which can be specified with child element. Signed-off-by:

[libvirt PATCH v2 02/15] qemu: add -display dbus capability check (to update to 6.3)

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_6.2.0.x86_64.replies | 10 +-

[libvirt PATCH v2 04/15] qemu: start the D-Bus daemon for the display

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Start the daemon if necessary (it is already stopped in qemuProcessStop) Signed-off-by: Marc-André Lureau --- src/qemu/qemu_extdevice.c | 13 + 1 file changed, 13 insertions(+) diff --git a/src/qemu/qemu_extdevice.c b/src/qemu/qemu_extdevice.c index

[libvirt PATCH v2 01/15] qemu: add chardev-vdagent capability check

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_6.1.0.x86_64.xml | 1 + tests/qemucapabilitiesdata/caps_6.2.0.aarch64.xml | 1 +

[libvirt PATCH v2 00/15] WIP/RFC: add QEMU "-display dbus" support

2021-12-02 Thread marcandre . lureau
From: Marc-André Lureau Hi, This series implements supports for the upcoming QEMU "-display dbus" support. Development is still in progress, but I hope to land the QEMU support early in 6.3 (last version posted: https://patchew.org/QEMU/20211009210838.2219430-1-marcandre.lur...@redhat.com/).

[libvirt PATCH 7/9] qemu: add -chardev dbus

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 1 + src/conf/domain_conf.c| 26 + src/conf/domain_conf.h| 4 ++ src/conf/domain_validate.c| 32

[libvirt PATCH 9/9] docs: document type dbus

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/formatdomain.rst | 24 1 file changed, 24 insertions(+) diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst index eb8c973cf1fc..b477aec65f5c 100644 --- a/docs/formatdomain.rst +++

[libvirt PATCH 8/9] qemu: add usbredir type 'dbus'

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau The USB device redirection works in a similar way as Spice. The underlying 'dbus' channel is set to "org.qemu.usbredir" by default for the client to identify the channel purpose (as specified in -display dbus documentation). Signed-off-by: Marc-André Lureau ---

[libvirt PATCH 6/9] qemu: add dbus clipboard sharing

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau This feature requires to setup a "-chardev qemu-vdagent,clipboard=on". "qemu-vdagent" is not a typical chardev that you can connect to another end, so it doesn't map to a or . Set it up along with "-display dbus" code instead. Signed-off-by: Marc-André Lureau ---

[libvirt PATCH 5/9] qemu: add audio type 'dbus'

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- docs/schemas/domaincommon.rng | 29 + src/bhyve/bhyve_command.c | 1 + src/conf/domain_conf.c| 16 +++ src/conf/domain_conf.h| 2 +

[libvirt PATCH 4/9] qemu: add -display dbus support

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Without any extra option, libvirt will start and tell QEMU to connect to the private VM bus. Instead, a D-Bus "address" to connect to can be specified. Or the p2p mode enabled. D-Bus display best works with GL & a rendernode, which can be specified with child element.

[libvirt PATCH 3/9] qemu: add -display dbus capability check (to update to 6.3)

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_6.2.0.x86_64.replies | 10 +-

[libvirt PATCH 2/9] qemu: add chardev-vdagent capability check

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_6.1.0.x86_64.xml | 1 + tests/qemucapabilitiesdata/caps_6.2.0.aarch64.xml | 1 +

[libvirt PATCH 1/9] qemu: start the D-Bus daemon as needed

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau The daemon is started on daemon in external devices and hotplug code. Add a mechanism to start it before qemu, if qemu itself needs it. (it is already stopped in qemuProcessStop) Signed-off-by: Marc-André Lureau --- src/qemu/qemu_domain.h | 1 +

[libvirt PATCH 0/9] WIP/RFC: add QEMU "-display dbus" support

2021-11-05 Thread marcandre . lureau
From: Marc-André Lureau Hi, This series implements supports for the upcoming QEMU "-display dbus" support. Development is still in progress, but I hope to land the QEMU support early in 6.3 (last version posted: https://patchew.org/QEMU/20211009210838.2219430-1-marcandre.lur...@redhat.com/).

[libvirt PATCH 6/9] qemu_agent: add qemuAgentGetDisks

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau guest-get-disks is available since QEMU 5.2: https://wiki.qemu.org/ChangeLog/5.2#Guest_agent Note that the test response was manually edited based on a reply on my bare-metal computer. It shows partial results due to pcieport driver not being currently supported by QGA.

[libvirt PATCH 2/9] qemu_agent: export qemuAgentDiskAddressFree & add g_auto

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_agent.c | 2 +- src/qemu/qemu_agent.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_agent.c b/src/qemu/qemu_agent.c index beb42449ce..ba07d6bfdf 100644 --- a/src/qemu/qemu_agent.c

[libvirt PATCH 5/9] qemu: use virJSONValueObjectGetStringArray

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau There might be more potential users around, I haven't looked thoroughly. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_monitor_json.c | 34 ++ 1 file changed, 6 insertions(+), 28 deletions(-) diff --git

[libvirt PATCH 0/9] qemu: report guest disks informations

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Hi, The following series extends virDomainGetGuestInfo to report disk informations provided by the new QGA "guest-get-disks" command in QEMU 5.2. Please review, Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1899527 Marc-André Lureau (9): qemu_agent: rename

[libvirt PATCH 1/9] qemu_agent: rename qemuAgentDiskInfo->qemuAgentDiskAddress

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau To match the QGA schema name (we are introducing a qemuAgentDiskInfo struct again for different purpose). Signed-off-by: Marc-André Lureau --- src/qemu/qemu_agent.c | 10 +- src/qemu/qemu_agent.h | 8 src/qemu/qemu_driver.c | 4 ++-- 3 files

[libvirt PATCH 9/9] virsh: add --disk informations to guestinfo command

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- tools/virsh-domain.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index c999458d72..070b8d21fd 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-domain.c @@ -14219,6

[libvirt PATCH 8/9] qemu_driver: report guest disk informations

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_driver.c | 99 -- 1 file changed, 96 insertions(+), 3 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index a8760f82b1..8d3e09da69 100644 ---

[libvirt PATCH 7/9] domain: add disk informations to virDomainGetGuestInfo

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- include/libvirt/libvirt-domain.h | 1 + src/libvirt-domain.c | 17 + 2 files changed, 18 insertions(+) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index

[libvirt PATCH 4/9] util: json: add virJSONValueObjectGetStringArray convenience

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/libvirt_private.syms | 1 + src/util/virjson.c | 30 ++ src/util/virjson.h | 1 + 3 files changed, 32 insertions(+) diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index

[libvirt PATCH 3/9] qemu_agent: factor out qemuAgentGetDiskAddress

2020-11-20 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_agent.c | 83 ++- 1 file changed, 43 insertions(+), 40 deletions(-) diff --git a/src/qemu/qemu_agent.c b/src/qemu/qemu_agent.c index ba07d6bfdf..31f26eedd1 100644 ---

[libvirt PATCH] qemu: add qemuAgentSSH{Add,Remove,Get}AuthorizedKeys

2020-11-07 Thread marcandre . lureau
From: Marc-André Lureau In QEMU 5.2, the guest agent learned to manipulate a user ~/.ssh/authorized_keys. Bind the JSON API to libvirt. https://wiki.qemu.org/ChangeLog/5.2#Guest_agent Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1888537 Signed-off-by: Marc-André Lureau ---

[libvirt PATCH 5/6] qemu-dbus: register DBus bus to the VM cgroup

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau External devices are started before cgroup is created. Add the DBus daemon to the VM cgroup with the rest of the external devices. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 24 src/qemu/qemu_dbus.h | 3 +++

[libvirt PATCH 6/6] qemu-cgroup: remove unnecessary include

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau The file doesn't use virSystemd functions directly. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_cgroup.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index 2afe22f177..2e019b64af 100644 ---

[libvirt PATCH 3/6] qemu-dbus: prevent double start of the bus daemon

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau Allow calling qemuDBusStart() multiple times (as may be done by qemu-slirp already). Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/qemu/qemu_dbus.c b/src/qemu/qemu_dbus.c index

[libvirt PATCH 4/6] qemu-dbus: remove unused variable

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/qemu/qemu_dbus.c b/src/qemu/qemu_dbus.c index ae55bbb299..02dc3ebf07 100644 --- a/src/qemu/qemu_dbus.c +++ b/src/qemu/qemu_dbus.c @@ -142,13 +142,11 @@

[libvirt PATCH 1/6] slirp: leave the dbus daemon running on error

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau Don't stop the DBus daemon if a slirp helper failed to start, as it may be shared with other helpers. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_slirp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_slirp.c

[libvirt PATCH 2/6] slirp: add helper to VM cgroup

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau The slirp helper process should be associated with the VM cgroup, like other helpers. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_extdevice.c | 8 src/qemu/qemu_slirp.c | 14 ++ src/qemu/qemu_slirp.h | 3 +++ 3 files changed, 25

[libvirt PATCH 0/6] Some slirp/dbus related fixes

2020-04-08 Thread marcandre . lureau
From: Marc-André Lureau Hi, Here is a subset of the series posted earlier "RFC: basic CGroup support with qemu:///session", as requested by Pavel. Marc-André Lureau (6): slirp: leave the dbus daemon running on error slirp: add helper to VM cgroup qemu-dbus: prevent double start of the

[libvirt PATCH 11/15] systemd: register machines against session service

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau machined could quite easily support session management: https://github.com/systemd/systemd/pull/15312 Signed-off-by: Marc-André Lureau --- src/util/virsystemd.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git

[libvirt PATCH 12/15] cgroup: return directly if there is nothing to remove

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau This fixes an error path when virCgroupV2DevicesDetectProg() fails but wasn't needed in the first place on VM shutdown. Signed-off-by: Marc-André Lureau --- src/util/vircgroupv2devices.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[libvirt PATCH 13/15] qemu-cgroup: register ext devices when cpu/cpuset controller are missing

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau External devices registration can add processes to the process group, which allows basic process management or other tuning. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_cgroup.c | 9 - 1 file changed, 9 deletions(-) diff --git a/src/qemu/qemu_cgroup.c

[libvirt PATCH 10/15] systemd: only check for activitable machine1 service

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau The service is started on demand, checking if it's already registered on the bus defeat this purpose. Note: the same is probably true for login1, but I don't intent to look at the code path yet. Signed-off-by: Marc-André Lureau --- src/util/virsystemd.c | 2 --

[libvirt PATCH 15/15] qemu-cgroup: drop the need for privileges to use cgroup

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau CGroup delegation can allow various processes or users to use cgroup. Further checks should be done by the various backends. With this series, a qemu:///session VM can have basic CGroupv2 support with machined --user help. Signed-off-by: Marc-André Lureau ---

[libvirt PATCH 14/15] qemu: create cgroup regardless of controller support

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau This allow for basic process management, at least. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_process.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index 6b9f6fb860..0c33eab26c

[libvirt PATCH 06/15] qemu-cgroup: remove unnecessary include

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau The file doesn't use virSystemd functions directly. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_cgroup.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index 23e52fa218..cec0a03889 100644 ---

[libvirt PATCH 05/15] qemu-dbus: register DBus bus to the VM cgroup

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau External devices are started before cgroup is created. Add the DBus daemon to the VM cgroup with the rest of the external devices. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 24 src/qemu/qemu_dbus.h | 3 +++

[libvirt PATCH 09/15] systemd: check org.freedesktop.machine1 registration

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Since commit f10bd740e178c89f24d0b0298d0b5413537d0699 ("Cache the presence of machine1 service"), the code checks for systemd1 registration. Not totally unreasonable, but that seems odd since we actually check machined presence in this function. Signed-off-by: Marc-André

[libvirt PATCH 07/15] dbus: rename virDBusIs* -> virDBusSystemIs*

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Those functions talk to system bus and have a special handling of system bus missing. Signed-off-by: Marc-André Lureau --- src/util/virdbus.c | 12 ++-- src/util/virdbus.h | 4 ++-- src/util/virfirewalld.c | 2 +- src/util/virsystemd.c | 8

[libvirt PATCH 00/15] RFC: basic CGroup support with qemu:///session

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Hi, This is a small series that allows basic QEMU VM CGroup support with the help of machined --user: https://github.com/systemd/systemd/pull/15312 The first few patches are fixes to register dbus and slirp-helper correctly with the VM cgroup. A few changes are done to

[libvirt PATCH 02/15] slirp: add helper to VM cgroup

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau The slirp helper process should be associated with the VM cgroup, like other helpers. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_extdevice.c | 8 src/qemu/qemu_slirp.c | 14 ++ src/qemu/qemu_slirp.h | 3 +++ 3 files changed, 25

[libvirt PATCH 03/15] qemu-dbus: prevent double start of the bus daemon

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Allow calling qemuDBusStart() multiple times (as may be done by qemu-slirp already). Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/qemu/qemu_dbus.c b/src/qemu/qemu_dbus.c index

[libvirt PATCH 04/15] qemu-dbus: remove unused variable

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- src/qemu/qemu_dbus.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/qemu/qemu_dbus.c b/src/qemu/qemu_dbus.c index ae55bbb299..02dc3ebf07 100644 --- a/src/qemu/qemu_dbus.c +++ b/src/qemu/qemu_dbus.c @@ -142,13 +142,11 @@

[libvirt PATCH 01/15] slirp: leave the dbus daemon running on error

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Don't stop the DBus daemon if a slirp helper failed to start, as it may be shared with other helpers. Signed-off-by: Marc-André Lureau --- src/qemu/qemu_slirp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_slirp.c

[libvirt PATCH 08/15] dbus: add virDBusIsService{Enabled, Registered} with @conn argument

2020-04-06 Thread marcandre . lureau
From: Marc-André Lureau Learn to check presence of services on a given connection. Signed-off-by: Marc-André Lureau --- src/util/virdbus.c | 65 +++--- src/util/virdbus.h | 3 +++ 2 files changed, 53 insertions(+), 15 deletions(-) diff --git

  1   2   3   4   >