qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

             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).