qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	Richard Henderson <rth@twiddle.net>,
	qemu-devel@nongnu.org, Igor Mammedov <imammedo@redhat.com>
Subject: [Qemu-devel] [PULL 18/28] pc: Forbid BSP removal
Date: Wed, 20 Jul 2016 12:08:24 -0300	[thread overview]
Message-ID: <1469027314-31655-19-git-send-email-ehabkost@redhat.com> (raw)
In-Reply-To: <1469027314-31655-1-git-send-email-ehabkost@redhat.com>

From: Igor Mammedov <imammedo@redhat.com>

Boot CPU is assumed to always present in QEMU code, so
untile that assumptions are gone, deny removal request,
In another words QEMU won't support BSP hot-unplug.

Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---
 hw/i386/pc.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index f4c6a27..dd5661a 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -1803,10 +1803,18 @@ out:
 static void pc_cpu_unplug_request_cb(HotplugHandler *hotplug_dev,
                                      DeviceState *dev, Error **errp)
 {
+    int idx = -1;
     HotplugHandlerClass *hhc;
     Error *local_err = NULL;
     PCMachineState *pcms = PC_MACHINE(hotplug_dev);
 
+    pc_find_cpu_slot(pcms, CPU(dev), &idx);
+    assert(idx != -1);
+    if (idx == 0) {
+        error_setg(&local_err, "Boot CPU is unpluggable");
+        goto out;
+    }
+
     hhc = HOTPLUG_HANDLER_GET_CLASS(pcms->acpi_dev);
     hhc->unplug_request(HOTPLUG_HANDLER(pcms->acpi_dev), dev, &local_err);
 
-- 
2.5.5

  parent reply	other threads:[~2016-07-20 15:09 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-20 15:08 [Qemu-devel] [PULL v2 00/28] x86 queue, 2016-07-20 Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 01/28] target-i386: Provide TCG_PHYS_ADDR_BITS Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 02/28] target-i386: Allow physical address bits to be set Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 03/28] target-i386: Mask mtrr mask based on CPU physical address limits Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 04/28] target-i386: Fill high bits of mtrr mask Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 05/28] target-i386: Use uint32_t for X86CPU.apic_id Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 06/28] pc: Add x86_topo_ids_from_apicid() Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 07/28] target-i386: Set physical address bits based on host Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 08/28] pc: Extract CPU lookup into a separate function Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 09/28] pc: cpu: Consolidate apic-id validity checks in pc_cpu_pre_plug() Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 10/28] target-i386: Replace custom apic-id setter/getter with static property Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 11/28] target-i386: Add socket/core/thread properties to X86CPU Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 12/28] target-i386: Add support for UMIP and RDPID CPUID bits Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 13/28] target-i386: cpu: Do not ignore error and fix apic parent Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 14/28] target-i386: Fix apic object leak when CPU is deleted Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 15/28] pc: Set APIC ID based on socket/core/thread ids if it's not been set yet Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 16/28] pc: Delay setting number of boot CPUs to machine_done time Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 17/28] pc: Register created initial and hotpluged CPUs in one place pc_cpu_plug() Eduardo Habkost
2016-07-20 15:08 ` Eduardo Habkost [this message]
2016-07-20 15:08 ` [Qemu-devel] [PULL 19/28] pc: Enforce adding CPUs contiguously and removing them in opposite order Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 20/28] pc: cpu: Allow device_add to be used with x86 cpu Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 21/28] pc: Implement query-hotpluggable-cpus callback Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 22/28] apic: move MAX_APICS check to 'apic' class Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 23/28] apic: Drop APICCommonState.idx and use APIC ID as index in local_apics[] Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 24/28] apic: kvm-apic: Fix crash due to access to freed memory region Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 25/28] (kvm)apic: Add unrealize callbacks Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 26/28] apic: Use apic_id as apic's migration instance_id Eduardo Habkost
2016-07-26  5:11   ` Amit Shah
2016-07-26  8:00     ` Igor Mammedov
2016-07-26 11:47       ` Amit Shah
2016-07-26 12:58         ` Igor Mammedov
2016-07-26 13:11           ` Amit Shah
2016-07-26 13:41             ` Igor Mammedov
2016-07-26 14:16             ` Igor Mammedov
2016-07-26 19:19               ` Eduardo Habkost
2016-07-27  7:41                 ` Igor Mammedov
2016-07-26  9:41     ` Igor Mammedov
2016-07-26 12:26       ` Amit Shah
2016-07-20 15:08 ` [Qemu-devel] [PULL 27/28] target-i386: Add x86_cpu_unrealizefn() Eduardo Habkost
2016-07-20 15:08 ` [Qemu-devel] [PULL 28/28] pc: Make device_del CPU work for x86 CPUs Eduardo Habkost
2016-07-20 20:59 ` [Qemu-devel] [PULL v2 00/28] x86 queue, 2016-07-20 Peter Maydell
  -- strict thread matches above, loose matches on Subject: below --
2016-07-19 17:22 [Qemu-devel] [PULL 00/28] x86 queue for -rc0 Eduardo Habkost
2016-07-19 17:22 ` [Qemu-devel] [PULL 18/28] pc: Forbid BSP removal Eduardo Habkost

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=1469027314-31655-19-git-send-email-ehabkost@redhat.com \
    --to=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).