All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: Laurent Vivier <lvivier@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>,
	Thomas Huth <thuth@redhat.com>,
	Eduardo Habkost <ehabkost@redhat.com>,
	Sergio Lopez <slp@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Shannon Zhao <shannon.zhaosl@gmail.com>,
	qemu-arm@nongnu.org, Gerd Hoffmann <kraxel@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Igor Mammedov <imammedo@redhat.com>,
	Richard Henderson <rth@twiddle.net>
Subject: [PATCH v8 16/21] microvm: wire up hotplug
Date: Tue, 15 Sep 2020 14:09:04 +0200	[thread overview]
Message-ID: <20200915120909.20838-17-kraxel@redhat.com> (raw)
In-Reply-To: <20200915120909.20838-1-kraxel@redhat.com>

The cpu hotplug code handles the initialization of coldplugged cpus
too, so it is needed even in case cpu hotplug is not supported.

Wire cpu hotplug up for microvm.
Without this we get a broken MADT table.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Sergio Lopez <slp@redhat.com>
---
 hw/i386/microvm.c | 42 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 42 insertions(+)

diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
index 9df15354ce0f..8de8ca864963 100644
--- a/hw/i386/microvm.c
+++ b/hw/i386/microvm.c
@@ -320,6 +320,39 @@ static void microvm_fix_kernel_cmdline(MachineState *machine)
     g_free(cmdline);
 }
 
+static void microvm_device_pre_plug_cb(HotplugHandler *hotplug_dev,
+                                       DeviceState *dev, Error **errp)
+{
+    x86_cpu_pre_plug(hotplug_dev, dev, errp);
+}
+
+static void microvm_device_plug_cb(HotplugHandler *hotplug_dev,
+                                   DeviceState *dev, Error **errp)
+{
+    x86_cpu_plug(hotplug_dev, dev, errp);
+}
+
+static void microvm_device_unplug_request_cb(HotplugHandler *hotplug_dev,
+                                             DeviceState *dev, Error **errp)
+{
+    error_setg(errp, "unplug not supported by microvm");
+}
+
+static void microvm_device_unplug_cb(HotplugHandler *hotplug_dev,
+                                     DeviceState *dev, Error **errp)
+{
+    error_setg(errp, "unplug not supported by microvm");
+}
+
+static HotplugHandler *microvm_get_hotplug_handler(MachineState *machine,
+                                                   DeviceState *dev)
+{
+    if (object_dynamic_cast(OBJECT(dev), TYPE_CPU)) {
+        return HOTPLUG_HANDLER(machine);
+    }
+    return NULL;
+}
+
 static void microvm_machine_state_init(MachineState *machine)
 {
     MicrovmMachineState *mms = MICROVM_MACHINE(machine);
@@ -503,6 +536,7 @@ static void microvm_machine_initfn(Object *obj)
 static void microvm_class_init(ObjectClass *oc, void *data)
 {
     MachineClass *mc = MACHINE_CLASS(oc);
+    HotplugHandlerClass *hc = HOTPLUG_HANDLER_CLASS(oc);
 
     mc->init = microvm_machine_state_init;
 
@@ -524,6 +558,13 @@ static void microvm_class_init(ObjectClass *oc, void *data)
     /* Machine class handlers */
     mc->reset = microvm_machine_reset;
 
+    /* hotplug (for cpu coldplug) */
+    mc->get_hotplug_handler = microvm_get_hotplug_handler;
+    hc->pre_plug = microvm_device_pre_plug_cb;
+    hc->plug = microvm_device_plug_cb;
+    hc->unplug_request = microvm_device_unplug_request_cb;
+    hc->unplug = microvm_device_unplug_cb;
+
     object_class_property_add(oc, MICROVM_MACHINE_PIC, "OnOffAuto",
                               microvm_machine_get_pic,
                               microvm_machine_set_pic,
@@ -573,6 +614,7 @@ static const TypeInfo microvm_machine_info = {
     .class_size    = sizeof(MicrovmMachineClass),
     .class_init    = microvm_class_init,
     .interfaces = (InterfaceInfo[]) {
+         { TYPE_HOTPLUG_HANDLER },
          { }
     },
 };
-- 
2.27.0



  parent reply	other threads:[~2020-09-15 12:31 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-15 12:08 [PATCH v8 00/21] microvm: add acpi support Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 01/21] microvm: name qboot binary qboot.rom Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 02/21] seabios: add microvm config, update build rules Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 03/21] seabios: add bios-microvm.bin binary Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 04/21] acpi: ged: add control regs Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 05/21] acpi: ged: add x86 device variant Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 06/21] acpi: move acpi_dsdt_add_power_button() to ged Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 07/21] microvm: make virtio irq base runtime configurable Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 08/21] microvm/acpi: add minimal acpi support Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 09/21] microvm/acpi: add acpi_dsdt_add_virtio() for x86 Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 10/21] microvm/acpi: use GSI 16-23 for virtio Gerd Hoffmann
2020-09-15 12:08 ` [PATCH v8 11/21] microvm/acpi: use seabios with acpi=on Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 12/21] microvm/acpi: disable virtio-mmio cmdline hack Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 13/21] x86: constify x86_machine_is_*_enabled Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 14/21] x86: move acpi_dev from pc/microvm Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 15/21] x86: move cpu hotplug from pc to x86 Gerd Hoffmann
2020-09-15 12:55   ` Igor Mammedov
2020-09-15 12:09 ` Gerd Hoffmann [this message]
2020-09-15 12:09 ` [PATCH v8 17/21] tests/acpi: allow microvm test data updates Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 18/21] tests/acpi: allow override blkdev Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 19/21] tests/acpi: add microvm test Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 20/21] tests/acpi: update expected data files for microvm Gerd Hoffmann
2020-09-15 12:09 ` [PATCH v8 21/21] microvm: enable ramfb Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200915120909.20838-17-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=shannon.zhaosl@gmail.com \
    --cc=slp@redhat.com \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.