From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:44685) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RkAtg-0001jS-3Q for qemu-devel@nongnu.org; Mon, 09 Jan 2012 03:48:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RkAta-0006p2-E1 for qemu-devel@nongnu.org; Mon, 09 Jan 2012 03:48:04 -0500 Received: from mx1.redhat.com ([209.132.183.28]:8208) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RkAta-0006om-3X for qemu-devel@nongnu.org; Mon, 09 Jan 2012 03:47:58 -0500 Date: Mon, 9 Jan 2012 10:47:52 +0200 From: Gleb Natapov Message-ID: <20120109084752.GG2167@redhat.com> References: <1326058804-561-1-git-send-email-imammedo@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1326058804-561-1-git-send-email-imammedo@redhat.com> Subject: Re: [Qemu-devel] [PATCH qemu-kvm] Fix: SCI isn't sent on cpu hotplug List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: jan.kiszka@siemens.com, qemu-devel@nongnu.org, avi@redhat.com On Sun, Jan 08, 2012 at 10:40:04PM +0100, Igor Mammedov wrote: > Change introduced by e71f08bb4a > "Fix cpu/pci hotplug to generate level triggered interrupt." > was lost somewhre along the way. And as result SCI is not sent in > case of cpu hotplug event. > Restoring hunk 1 of e71f08bb4a fixes issue. > Hmm, I sent similar patch [1] last time someone complained about cpu hotplug here. Which remind me that in that thread more problem were found in cpu hotplug. IIRC Jan collected all the patches. Jan, what happened to them? [1] http://permalink.gmane.org/gmane.comp.emulators.kvm.devel/76399 > Signed-off-by: Igor Mammedov > --- > hw/acpi_piix4.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c > index 1b35707..741a8cc 100644 > --- a/hw/acpi_piix4.c > +++ b/hw/acpi_piix4.c > @@ -94,7 +94,8 @@ static void pm_update_sci(PIIX4PMState *s) > ACPI_BITMASK_POWER_BUTTON_ENABLE | > ACPI_BITMASK_GLOBAL_LOCK_ENABLE | > ACPI_BITMASK_TIMER_ENABLE)) != 0) || > - (((s->gpe.sts[0] & s->gpe.en[0]) & PIIX4_PCI_HOTPLUG_STATUS) != 0); > + (((s->gpe.sts[0] & s->gpe.en[0]) & > + (PIIX4_CPU_HOTPLUG_STATUS | PIIX4_PCI_HOTPLUG_STATUS)) != 0); > > qemu_set_irq(s->irq, sci_level); > /* schedule a timer interruption if needed */ > -- > 1.7.7.5 -- Gleb.