[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC PATCH v3 1/5] rename MachineInitPhase enum constants for QAPI compa
From: |
Damien Hedde |
Subject: |
[RFC PATCH v3 1/5] rename MachineInitPhase enum constants for QAPI compatibility |
Date: |
Wed, 17 Nov 2021 15:46:59 +0100 |
From: Mirela Grujic <mirela.grujic@greensocs.com>
This commit is a preparation to switch to a QAPI definition
of the MachineInitPhase enum.
QAPI will generate enumeration constants prefixed with the
MACHINE_INIT_PHASE_, so rename values accordingly.
Signed-off-by: Mirela Grujic <mirela.grujic@greensocs.com>
Signed-off-by: Damien Hedde <damien.hedde@greensocs.com>
---
include/hw/qdev-core.h | 10 +++++-----
hw/core/machine-qmp-cmds.c | 2 +-
hw/core/machine.c | 6 +++---
hw/core/qdev.c | 2 +-
hw/pci/pci.c | 2 +-
hw/usb/core.c | 2 +-
hw/virtio/virtio-iommu.c | 2 +-
monitor/hmp.c | 2 +-
softmmu/qdev-monitor.c | 9 +++++----
softmmu/vl.c | 6 +++---
ui/console.c | 3 ++-
11 files changed, 24 insertions(+), 22 deletions(-)
diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h
index 20d3066595..ef2d612d39 100644
--- a/include/hw/qdev-core.h
+++ b/include/hw/qdev-core.h
@@ -849,30 +849,30 @@ bool qdev_should_hide_device(const QDict *opts, bool
from_json, Error **errp);
typedef enum MachineInitPhase {
/* current_machine is NULL. */
- PHASE_NO_MACHINE,
+ MACHINE_INIT_PHASE_NO_MACHINE,
/* current_machine is not NULL, but current_machine->accel is NULL. */
- PHASE_MACHINE_CREATED,
+ MACHINE_INIT_PHASE_MACHINE_CREATED,
/*
* current_machine->accel is not NULL, but the machine properties have
* not been validated and machine_class->init has not yet been called.
*/
- PHASE_ACCEL_CREATED,
+ MACHINE_INIT_PHASE_ACCEL_CREATED,
/*
* machine_class->init has been called, thus creating any embedded
* devices and validating machine properties. Devices created at
* this time are considered to be cold-plugged.
*/
- PHASE_MACHINE_INITIALIZED,
+ MACHINE_INIT_PHASE_INITIALIZED,
/*
* QEMU is ready to start CPUs and devices created at this time
* are considered to be hot-plugged. The monitor is not restricted
* to "preconfig" commands.
*/
- PHASE_MACHINE_READY,
+ MACHINE_INIT_PHASE_READY,
} MachineInitPhase;
extern bool phase_check(MachineInitPhase phase);
diff --git a/hw/core/machine-qmp-cmds.c b/hw/core/machine-qmp-cmds.c
index 4f4ab30f8c..ddbdc5212f 100644
--- a/hw/core/machine-qmp-cmds.c
+++ b/hw/core/machine-qmp-cmds.c
@@ -148,7 +148,7 @@ HotpluggableCPUList *qmp_query_hotpluggable_cpus(Error
**errp)
void qmp_set_numa_node(NumaOptions *cmd, Error **errp)
{
- if (phase_check(PHASE_MACHINE_INITIALIZED)) {
+ if (phase_check(MACHINE_INIT_PHASE_INITIALIZED)) {
error_setg(errp, "The command is permitted only before the machine has
been created");
return;
}
diff --git a/hw/core/machine.c b/hw/core/machine.c
index 26ec54e726..8560bb4c42 100644
--- a/hw/core/machine.c
+++ b/hw/core/machine.c
@@ -1178,7 +1178,7 @@ void machine_run_board_init(MachineState *machine)
accel_init_interfaces(ACCEL_GET_CLASS(machine->accelerator));
machine_class->init(machine);
- phase_advance(PHASE_MACHINE_INITIALIZED);
+ phase_advance(MACHINE_INIT_PHASE_INITIALIZED);
}
static NotifierList machine_init_done_notifiers =
@@ -1187,7 +1187,7 @@ static NotifierList machine_init_done_notifiers =
void qemu_add_machine_init_done_notifier(Notifier *notify)
{
notifier_list_add(&machine_init_done_notifiers, notify);
- if (phase_check(PHASE_MACHINE_READY)) {
+ if (phase_check(MACHINE_INIT_PHASE_READY)) {
notify->notify(notify, NULL);
}
}
@@ -1210,7 +1210,7 @@ void qdev_machine_creation_done(void)
* ok, initial machine setup is done, starting from now we can
* only create hotpluggable devices
*/
- phase_advance(PHASE_MACHINE_READY);
+ phase_advance(MACHINE_INIT_PHASE_READY);
qdev_assert_realized_properly();
/* TODO: once all bus devices are qdevified, this should be done
diff --git a/hw/core/qdev.c b/hw/core/qdev.c
index 84f3019440..ccfd6f0dc4 100644
--- a/hw/core/qdev.c
+++ b/hw/core/qdev.c
@@ -674,7 +674,7 @@ static void device_initfn(Object *obj)
{
DeviceState *dev = DEVICE(obj);
- if (phase_check(PHASE_MACHINE_READY)) {
+ if (phase_check(MACHINE_INIT_PHASE_READY)) {
dev->hotplugged = 1;
qdev_hot_added = true;
}
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index e5993c1ef5..f77d9e8d57 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -1102,7 +1102,7 @@ static PCIDevice *do_pci_register_device(PCIDevice
*pci_dev,
address_space_init(&pci_dev->bus_master_as,
&pci_dev->bus_master_container_region, pci_dev->name);
- if (phase_check(PHASE_MACHINE_READY)) {
+ if (phase_check(MACHINE_INIT_PHASE_READY)) {
pci_init_bus_master(pci_dev);
}
pci_dev->irq_state = 0;
diff --git a/hw/usb/core.c b/hw/usb/core.c
index 975f76250a..7a9a81c4fe 100644
--- a/hw/usb/core.c
+++ b/hw/usb/core.c
@@ -97,7 +97,7 @@ void usb_wakeup(USBEndpoint *ep, unsigned int stream)
USBDevice *dev = ep->dev;
USBBus *bus = usb_bus_from_device(dev);
- if (!phase_check(PHASE_MACHINE_READY)) {
+ if (!phase_check(MACHINE_INIT_PHASE_READY)) {
/*
* This is machine init cold plug. No need to wakeup anyone,
* all devices will be reset anyway. And trying to wakeup can
diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c
index 1b23e8e18c..b777a80be2 100644
--- a/hw/virtio/virtio-iommu.c
+++ b/hw/virtio/virtio-iommu.c
@@ -948,7 +948,7 @@ static int
virtio_iommu_set_page_size_mask(IOMMUMemoryRegion *mr,
* accept it. Having a different masks is possible but the guest will use
* sub-optimal block sizes, so warn about it.
*/
- if (phase_check(PHASE_MACHINE_READY)) {
+ if (phase_check(MACHINE_INIT_PHASE_READY)) {
int new_granule = ctz64(new_mask);
int cur_granule = ctz64(cur_mask);
diff --git a/monitor/hmp.c b/monitor/hmp.c
index b20737e63c..3275e7aeed 100644
--- a/monitor/hmp.c
+++ b/monitor/hmp.c
@@ -217,7 +217,7 @@ static bool cmd_can_preconfig(const HMPCommand *cmd)
static bool cmd_available(const HMPCommand *cmd)
{
- return phase_check(PHASE_MACHINE_READY) || cmd_can_preconfig(cmd);
+ return phase_check(MACHINE_INIT_PHASE_READY) || cmd_can_preconfig(cmd);
}
static void help_cmd_dump_one(Monitor *mon,
diff --git a/softmmu/qdev-monitor.c b/softmmu/qdev-monitor.c
index 01f3834db5..1d6a1c4716 100644
--- a/softmmu/qdev-monitor.c
+++ b/softmmu/qdev-monitor.c
@@ -250,7 +250,7 @@ static DeviceClass *qdev_get_device_class(const char
**driver, Error **errp)
dc = DEVICE_CLASS(oc);
if (!dc->user_creatable ||
- (phase_check(PHASE_MACHINE_READY) && !dc->hotpluggable)) {
+ (phase_check(MACHINE_INIT_PHASE_READY) && !dc->hotpluggable)) {
error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "driver",
"a pluggable device type");
return NULL;
@@ -660,7 +660,8 @@ DeviceState *qdev_device_add_from_qdict(const QDict *opts,
return NULL;
}
- if (phase_check(PHASE_MACHINE_READY) && bus && !qbus_is_hotpluggable(bus))
{
+ if (phase_check(MACHINE_INIT_PHASE_READY) && bus &&
+ !qbus_is_hotpluggable(bus)) {
error_setg(errp, QERR_BUS_NO_HOTPLUG, bus->name);
return NULL;
}
@@ -674,7 +675,7 @@ DeviceState *qdev_device_add_from_qdict(const QDict *opts,
dev = qdev_new(driver);
/* Check whether the hotplug is allowed by the machine */
- if (phase_check(PHASE_MACHINE_READY)) {
+ if (phase_check(MACHINE_INIT_PHASE_READY)) {
if (!qdev_hotplug_allowed(dev, errp)) {
goto err_del_dev;
}
@@ -1040,7 +1041,7 @@ int qemu_global_option(const char *str)
bool qmp_command_available(const QmpCommand *cmd, Error **errp)
{
- if (!phase_check(PHASE_MACHINE_READY) &&
+ if (!phase_check(MACHINE_INIT_PHASE_READY) &&
!(cmd->options & QCO_ALLOW_PRECONFIG)) {
error_setg(errp, "The command '%s' is permitted only after machine
initialization has completed",
cmd->name);
diff --git a/softmmu/vl.c b/softmmu/vl.c
index 1159a64bce..df19b911e6 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -2732,7 +2732,7 @@ static void qemu_machine_creation_done(void)
void qmp_x_exit_preconfig(Error **errp)
{
- if (phase_check(PHASE_MACHINE_INITIALIZED)) {
+ if (phase_check(MACHINE_INIT_PHASE_INITIALIZED)) {
error_setg(errp, "The command is permitted only before machine
initialization");
return;
}
@@ -3715,14 +3715,14 @@ void qemu_init(int argc, char **argv, char **envp)
qemu_apply_legacy_machine_options(machine_opts_dict);
qemu_apply_machine_options(machine_opts_dict);
qobject_unref(machine_opts_dict);
- phase_advance(PHASE_MACHINE_CREATED);
+ phase_advance(MACHINE_INIT_PHASE_MACHINE_CREATED);
/*
* Note: uses machine properties such as kernel-irqchip, must run
* after qemu_apply_machine_options.
*/
configure_accelerators(argv[0]);
- phase_advance(PHASE_ACCEL_CREATED);
+ phase_advance(MACHINE_INIT_PHASE_ACCEL_CREATED);
/*
* Beware, QOM objects created before this point miss global and
diff --git a/ui/console.c b/ui/console.c
index 29a3e3f0f5..df66536a79 100644
--- a/ui/console.c
+++ b/ui/console.c
@@ -1295,7 +1295,8 @@ static QemuConsole *new_console(DisplayState *ds,
console_type_t console_type,
if (QTAILQ_EMPTY(&consoles)) {
s->index = 0;
QTAILQ_INSERT_TAIL(&consoles, s, next);
- } else if (console_type != GRAPHIC_CONSOLE ||
phase_check(PHASE_MACHINE_READY)) {
+ } else if (console_type != GRAPHIC_CONSOLE ||
+ phase_check(MACHINE_INIT_PHASE_READY)) {
QemuConsole *last = QTAILQ_LAST(&consoles);
s->index = last->index + 1;
QTAILQ_INSERT_TAIL(&consoles, s, next);
--
2.33.0
- [RFC PATCH v3 0/5] QMP support for cold-plugging devices, Damien Hedde, 2021/11/17
- [RFC PATCH v3 1/5] rename MachineInitPhase enum constants for QAPI compatibility,
Damien Hedde <=
- [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