[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC PATCH v2 05/16] qdev-monitor: prevent conflicts between qmp/device_
From: |
Damien Hedde |
Subject: |
[RFC PATCH v2 05/16] qdev-monitor: prevent conflicts between qmp/device_add and cli/-device |
Date: |
Wed, 22 Sep 2021 18:13:54 +0200 |
This commit prepares to extend device_add qmp command when using
-preconfig option.
In order to avoid conflicts with the cli -device option handling, we
need to handle some special case with the QemuOpts.
The qemu_device_opts is traversed when switching from
MACHINE_INIT_PHASE_INITIALIZED to MACHINE_INIT_PHASE_READY in order
to create any device specified by cli -device. Until now any
device_add qmp command was issued after that point so there was no
problem.
If we execute the qmp command before the MACHINE_INIT_PHASE_READY
phase we need to discard the QemuOpts from the qemu_device_opts in
order to avoid the cli -device code to try to create the device
again.
This commit preserves the opts behavior regarding the devices added
in 'ready' phase by the QMP command device_add.
Signed-off-by: Damien Hedde <damien.hedde@greensocs.com>
---
Although we keep the original behavior for QMP commands issued when
the machine is ready (only authorized case so far), we are not sure
it is necessary: keeping the opts in the list is not needed anymore
to ensure the id uniqueness of devices but it has the 2 following
consequences:
1. the device opts stay in the QemuOptsList. Is this list needed
after traversing the device cli options?
2. the DeviceState "opts" field is set. Do we need to keep it after
the device is realized ?
Any information on this will be appreciated.
---
softmmu/qdev-monitor.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/softmmu/qdev-monitor.c b/softmmu/qdev-monitor.c
index 0007698ff3..834f2b56b5 100644
--- a/softmmu/qdev-monitor.c
+++ b/softmmu/qdev-monitor.c
@@ -848,6 +848,23 @@ void qmp_device_add(QDict *qdict, QObject **ret_data,
Error **errp)
if (!dev) {
qemu_opts_del(opts);
return;
+ } else if (!phase_check(MACHINE_INIT_PHASE_READY)) {
+ /*
+ * Always delete the related opts in case the device was created
+ * before handling of cli -device arguments:
+ * We do not want a device added by the qmp command to be handled
+ * again by the cli -device creation code. This does not break
+ * the ID uniqueness because it is checked in qdev_device_add().
+ *
+ * Note: We check the phase in order to keep the legacy behavior:
+ * in the machine ready phase case, the QemuOpts remains in the list
+ * (and the dev->opts field is kept).
+ * If it happens it was done only to ensure the ID uniqueness and
+ * the QemuOpts is never used after this point: then we could
+ * remove QemuOpts in any phase.
+ */
+ dev->opts = NULL;
+ qemu_opts_del(opts);
}
object_unref(OBJECT(dev));
}
--
2.33.0
- Re: [RFC PATCH v2 02/16] qapi: Implement query-machine-phase QMP command, (continued)
[RFC PATCH v2 01/16] rename MachineInitPhase enum constants for QAPI compatibility, Damien Hedde, 2021/09/22
[RFC PATCH v2 07/16] hw/core/machine: add machine_class_is_dynamic_sysbus_dev_allowed, Damien Hedde, 2021/09/22
[RFC PATCH v2 03/16] qapi: Implement x-machine-init QMP command, Damien Hedde, 2021/09/22
[RFC PATCH v2 09/16] hw/core/machine: Remove the dynamic sysbus devices type check, Damien Hedde, 2021/09/22
[RFC PATCH v2 04/16] softmmu/qdev-monitor: add error handling in qdev_set_id, Damien Hedde, 2021/09/22
[RFC PATCH v2 06/16] qapi: Allow device_add to execute in machine initialized phase, Damien Hedde, 2021/09/22
[RFC PATCH v2 05/16] qdev-monitor: prevent conflicts between qmp/device_add and cli/-device,
Damien Hedde <=
[RFC PATCH v2 08/16] qdev-monitor: Check sysbus device type before creating it, Damien Hedde, 2021/09/22
[RFC PATCH v2 11/16] softmmu/memory: add memory_region_try_add_subregion function, Damien Hedde, 2021/09/22
[RFC PATCH v2 14/16] docs/system: add doc about the initialized machine phase and an example, Damien Hedde, 2021/09/22
[RFC PATCH v2 12/16] add x-sysbus-mmio-map qmp command, Damien Hedde, 2021/09/22
[RFC PATCH v2 10/16] qdev-monitor: allow adding any sysbus device before machine is ready, Damien Hedde, 2021/09/22