From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43614) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VjUMD-0004i0-3r for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:31:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VjUM6-0004PX-9v for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:31:45 -0500 Received: from mx1.redhat.com ([209.132.183.28]:59419) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VjUM5-0004Ok-R1 for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:31:38 -0500 Date: Thu, 21 Nov 2013 14:31:21 +0100 From: Igor Mammedov Message-ID: <20131121143121.5e54b5d0@nial.usersys.redhat.com> In-Reply-To: <20131121082840.GH24886@G08FNSTD100614.fnst.cn.fujitsu.com> References: <1385001528-12003-1-git-send-email-imammedo@redhat.com> <1385001528-12003-17-git-send-email-imammedo@redhat.com> <20131121071418.GB19703@redhat.com> <20131121081222.GG24886@G08FNSTD100614.fnst.cn.fujitsu.com> <20131121082636.GA20073@redhat.com> <20131121082840.GH24886@G08FNSTD100614.fnst.cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 16/27] acpi: ich9: allow guest to clear SCI rised by GPE List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Hu Tao Cc: peter.maydell@linaro.org, mdroth@linux.vnet.ibm.com, "Michael S. Tsirkin" , qemu-devel@nongnu.org, mjt@tls.msk.ru, stefanb@linux.vnet.ibm.com, stefanha@redhat.com, armbru@redhat.com, lcapitulino@redhat.com, vasilis.liaskovitis@profitbricks.com, quintela@redhat.com, chegu_vinod@hp.com, kraxel@redhat.com, aliguori@amazon.com, pbonzini@redhat.com, marcel.a@redhat.com, afaerber@suse.de On Thu, 21 Nov 2013 16:28:40 +0800 Hu Tao wrote: > On Thu, Nov 21, 2013 at 10:26:36AM +0200, Michael S. Tsirkin wrote: > > On Thu, Nov 21, 2013 at 04:12:22PM +0800, Hu Tao wrote: > > > On Thu, Nov 21, 2013 at 09:14:18AM +0200, Michael S. Tsirkin wrote: > > > > On Thu, Nov 21, 2013 at 03:38:37AM +0100, Igor Mammedov wrote: > > > > > it fixes IRQ storm since guest isn't able to lower SCI IRQ > > > > > after it has been handled when it clears GPE event. > > > > > > > > > > Signed-off-by: Igor Mammedov > > > > > > > > The storm is only on memory hotplug right? > > > > > > IIRC, it happens on cpu hotplug, too. > > > > So this is a bugfix then? We should include this in 1.7? > > Yes. but cpu hotplug support is not added for q35, so the bug doesn't > affect anyone, yet. > I was thinking about CPU hotplug for q35 as well, thus patch 1/27 "acpi: factor out common pm_update_sci() into acpi core" moves common for CPU/Memory hotplug ACPI bits into generic code, so making CPU hotplug for q35 would require only unifying CPU hotplug bits and plumbing it in q35 its done for mem hotplug in: [PATCH 17/27] acpi: initialize memory hotplug ACPI ICH9 hardware