From: Igor Mammedov <imammedo@redhat.com>
To: qemu-devel@nongnu.org
Cc: mst@redhat.com
Subject: [Qemu-devel] [PATCH] pc: q35: disable CPU hotplug handler on machines less than 2.0
Date: Tue, 6 May 2014 15:09:04 +0200 [thread overview]
Message-ID: <1399381744-28873-1-git-send-email-imammedo@redhat.com> (raw)
CPU hotplug handling for Q35 machine types was added
only since 2.0 but commit wrongly c649983b58 added
CPU hotplug handler to Q35 1.5 machine without
adding ACPI support for it. As result user can
create VCPU but guest couldn't use it since it
hasn't any data on it (missing ACPI implementation).
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
hw/i386/pc_q35.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
index c844dc2..687a397 100644
--- a/hw/i386/pc_q35.c
+++ b/hw/i386/pc_q35.c
@@ -308,7 +308,9 @@ static QEMUMachine pc_q35_machine_v2_0 = {
.init = pc_q35_init,
};
-#define PC_Q35_1_7_MACHINE_OPTIONS PC_Q35_MACHINE_OPTIONS
+#define PC_Q35_1_7_MACHINE_OPTIONS \
+ PC_Q35_MACHINE_OPTIONS, \
+ .hot_add_cpu = NULL
static QEMUMachine pc_q35_machine_v1_7 = {
PC_Q35_1_7_MACHINE_OPTIONS,
@@ -342,9 +344,7 @@ static QEMUMachine pc_q35_machine_v1_5 = {
},
};
-#define PC_Q35_1_4_MACHINE_OPTIONS \
- PC_Q35_1_6_MACHINE_OPTIONS, \
- .hot_add_cpu = NULL
+#define PC_Q35_1_4_MACHINE_OPTIONS PC_Q35_1_6_MACHINE_OPTIONS
static QEMUMachine pc_q35_machine_v1_4 = {
PC_Q35_1_4_MACHINE_OPTIONS,
--
1.7.1
next reply other threads:[~2014-05-06 13:09 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-06 13:09 Igor Mammedov [this message]
2014-05-07 7:03 ` [Qemu-devel] [PATCH] pc: q35: disable CPU hotplug handler on machines less than 2.0 Hu Tao
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=1399381744-28873-1-git-send-email-imammedo@redhat.com \
--to=imammedo@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
/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).