[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC PATCH v3 0/5] QMP support for cold-plugging devices
From: |
Damien Hedde |
Subject: |
[RFC PATCH v3 0/5] QMP support for cold-plugging devices |
Date: |
Wed, 17 Nov 2021 15:46:58 +0100 |
Hi all,
This series adds support for cold-plugging devices using QMP
commands. It is a step towards machine configuration using QMP, but
it does not allow the user to add more devices than he could do with
the CLI options before.
Right now we can add a device using 2 ways:
+ giving "-device" CLI option. In that case the device is
cold-plugged: it is created before the machine becomes ready.
+ issuing device_add QMP command. In that case the device is
hot-plugged (the command can not be issued before the machine is
ready).
This series allows to issue device_add QMP to cold-plug a device
like we do with "-device" CLI option. All added QMP commands are
marked as 'unstable' in qapi as they are part of preconfig.
We achieve this by introducing a new 'x-machine-init' command to
stop the VM creation at a point where we can cold-plug devices.
We are aware of the ongoing discussion about preconfig future (see
[1]). This patchset includes no major modifications from the v2 (but
the scope is reduced) and "x-machine-init" simply stops the
configuration between qemu_board_init() and qemu_create_cli_devices()
function calls.
As a consequence, in the current state, the timeline is:
+ "x-machine-init" command
+ "device_add" cold-plug commands (no fw_cfg legacy order support)
+ "x-exit-preconfig" command will then trigger the following
+ "-soundhw" CLI options
+ "-fw_cfg" CLI options
+ usb devices creation
+ "-device" CLI options (with fw_cfg legacy order support)
+ some other devices creation (with fw_cfg legacy order support)
We don't know if the differences between -device/device_add are
acceptable or not. To reduce/remove them we could move the
"x-machine-init" stopping point. What do you think ?
Patches 1, 3 and 5 miss a review.
The series is organized as follow:
+ Patches 1 and 2 converts the MachinePhase enum to a qapi definition
and add the 'query-machine-phase'. It allows to introspect the
current machine phase during preconfig as we will now be able to
reach several machine phases using QMP.
+ Patch 3 adds the 'x-machine-init' QMP command to stop QEMU at
machine-initialized phase during preconfig.
+ Patch 4 allows issuing device_add QMP command during the
machine-initialized phase.
+ Patch 5 improves the doc about preconfig in consequence.
[1]:
https://lore.kernel.org/qemu-devel/b31f442d28920447690a6b8cee865bdbacde1283.1635160056.git.mprivozn@redhat.com
Thanks for your feedback.
---
Damien
v3:
+ extracted patches related to cold-plugging devices from the v2 rfc
+ updated for rebase (qapi 'unstable' feature addition and 7.0 version bump)
+ fix check for x-machine-init command (and drop Alistair's ack-by on
patch 4)
+ extracted only a bit of the doc patch
+ drop qdev_set_id patch because it was merged in Kevin's
"qdev: Add JSON -device" series which did a lot of cleanups in
device_add related functions:
https://lore.kernel.org/qemu-devel/20211008133442.141332-1-kwolf@redhat.com
v2 was part of this rfc:
https://lore.kernel.org/qemu-devel/20210922161405.140018-1-damien.hedde@greensocs.com
Damien Hedde (1):
docs/system: improve doc about preconfig
Mirela Grujic (4):
rename MachineInitPhase enum constants for QAPI compatibility
qapi: Implement query-machine-phase QMP command
qapi: Implement x-machine-init QMP command
qapi: Allow device_add to execute in machine initialized phase
docs/system/managed-startup.rst | 20 +++++++-
qapi/machine.json | 87 +++++++++++++++++++++++++++++++++
qapi/qdev.json | 3 +-
include/hw/qdev-core.h | 30 +-----------
hw/core/machine-qmp-cmds.c | 11 ++++-
hw/core/machine.c | 6 +--
hw/core/qdev.c | 7 ++-
hw/pci/pci.c | 2 +-
hw/usb/core.c | 2 +-
hw/virtio/virtio-iommu.c | 2 +-
monitor/hmp.c | 2 +-
monitor/misc.c | 2 +-
softmmu/qdev-monitor.c | 15 ++++--
softmmu/vl.c | 23 ++++++---
ui/console.c | 3 +-
15 files changed, 164 insertions(+), 51 deletions(-)
--
2.33.0
- [RFC PATCH v3 0/5] QMP support for cold-plugging devices,
Damien Hedde <=
- [RFC PATCH v3 1/5] rename MachineInitPhase enum constants for QAPI compatibility, Damien Hedde, 2021/11/17
- [RFC PATCH v3 2/5] qapi: Implement query-machine-phase QMP command, Damien Hedde, 2021/11/17
- [RFC PATCH v3 3/5] qapi: Implement x-machine-init QMP command, Damien Hedde, 2021/11/17
- [RFC PATCH v3 4/5] qapi: Allow device_add to execute in machine initialized phase, Damien Hedde, 2021/11/17
- [RFC PATCH v3 5/5] docs/system: improve doc about preconfig, Damien Hedde, 2021/11/17
- Re: [RFC PATCH v3 0/5] QMP support for cold-plugging devices, Markus Armbruster, 2021/11/20