[PATCH 0/5] Introduce 'virDomainQemuMonitorCommandWithFiles'

Peter Krempa posted 5 patches 2 years, 2 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1643899820.git.pkrempa@redhat.com
docs/manpages/virsh.rst             |   6 +-
include/libvirt/libvirt-qemu.h      |   6 ++
src/driver-hypervisor.h             |   8 +++
src/libvirt-qemu.c                  |  71 ++++++++++++++++++
src/libvirt_qemu.syms               |   5 ++
src/qemu/qemu_driver.c              |  34 +++++++--
src/qemu/qemu_monitor.c             |   7 +-
src/qemu/qemu_monitor.h             |   1 +
src/qemu/qemu_monitor_json.c        |   6 +-
src/qemu/qemu_monitor_json.h        |   2 +
src/qemu/qemu_monitor_text.c        |   8 +--
src/qemu_protocol-structs           |   9 +++
src/remote/qemu_protocol.x          |  20 +++++-
src/remote/remote_daemon_dispatch.c |  42 +++++++++++
src/remote/remote_driver.c          |  40 +++++++++++
tools/virsh-domain.c                | 108 ++++++++++++++++------------
16 files changed, 313 insertions(+), 60 deletions(-)
[PATCH 0/5] Introduce 'virDomainQemuMonitorCommandWithFiles'
Posted by Peter Krempa 2 years, 2 months ago
I needed to do some tests where I wanted to pass FDs to qemu to see
how 'add-fd' and 'getfd' behave.

It may be useful for others who wish for any reason to pass FDs to QMP
commands.

Peter Krempa (5):
  lib: Introduce 'virDomainQemuMonitorCommandWithFiles'
  cmdStartGetFDs: Modernize
  virsh-domain: Move and rename cmdStartGetFDs to virshFetchPassFdsList
  virsh: Implement support for virDomainQemuMonitorCommandWithFiles
  qemu: Implement qemuDomainQemuMonitorCommandWithFiles

 docs/manpages/virsh.rst             |   6 +-
 include/libvirt/libvirt-qemu.h      |   6 ++
 src/driver-hypervisor.h             |   8 +++
 src/libvirt-qemu.c                  |  71 ++++++++++++++++++
 src/libvirt_qemu.syms               |   5 ++
 src/qemu/qemu_driver.c              |  34 +++++++--
 src/qemu/qemu_monitor.c             |   7 +-
 src/qemu/qemu_monitor.h             |   1 +
 src/qemu/qemu_monitor_json.c        |   6 +-
 src/qemu/qemu_monitor_json.h        |   2 +
 src/qemu/qemu_monitor_text.c        |   8 +--
 src/qemu_protocol-structs           |   9 +++
 src/remote/qemu_protocol.x          |  20 +++++-
 src/remote/remote_daemon_dispatch.c |  42 +++++++++++
 src/remote/remote_driver.c          |  40 +++++++++++
 tools/virsh-domain.c                | 108 ++++++++++++++++------------
 16 files changed, 313 insertions(+), 60 deletions(-)

-- 
2.34.1

Re: [PATCH 0/5] Introduce 'virDomainQemuMonitorCommandWithFiles'
Posted by Ján Tomko 2 years, 2 months ago
On a Thursday in 2022, Peter Krempa wrote:
>I needed to do some tests where I wanted to pass FDs to qemu to see
>how 'add-fd' and 'getfd' behave.
>
>It may be useful for others who wish for any reason to pass FDs to QMP
>commands.
>
>Peter Krempa (5):
>  lib: Introduce 'virDomainQemuMonitorCommandWithFiles'
>  cmdStartGetFDs: Modernize
>  virsh-domain: Move and rename cmdStartGetFDs to virshFetchPassFdsList
>  virsh: Implement support for virDomainQemuMonitorCommandWithFiles
>  qemu: Implement qemuDomainQemuMonitorCommandWithFiles
>
> docs/manpages/virsh.rst             |   6 +-
> include/libvirt/libvirt-qemu.h      |   6 ++
> src/driver-hypervisor.h             |   8 +++
> src/libvirt-qemu.c                  |  71 ++++++++++++++++++
> src/libvirt_qemu.syms               |   5 ++
> src/qemu/qemu_driver.c              |  34 +++++++--
> src/qemu/qemu_monitor.c             |   7 +-
> src/qemu/qemu_monitor.h             |   1 +
> src/qemu/qemu_monitor_json.c        |   6 +-
> src/qemu/qemu_monitor_json.h        |   2 +
> src/qemu/qemu_monitor_text.c        |   8 +--
> src/qemu_protocol-structs           |   9 +++
> src/remote/qemu_protocol.x          |  20 +++++-
> src/remote/remote_daemon_dispatch.c |  42 +++++++++++
> src/remote/remote_driver.c          |  40 +++++++++++
> tools/virsh-domain.c                | 108 ++++++++++++++++------------
> 16 files changed, 313 insertions(+), 60 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano