From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:50374) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UEhno-0004eY-CD for qemu-devel@nongnu.org; Sun, 10 Mar 2013 11:04:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UEhnn-0000mq-2H for qemu-devel@nongnu.org; Sun, 10 Mar 2013 11:04:44 -0400 Received: from mail-we0-x229.google.com ([2a00:1450:400c:c03::229]:52080) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UEhnm-0000ma-Sp for qemu-devel@nongnu.org; Sun, 10 Mar 2013 11:04:42 -0400 Received: by mail-we0-f169.google.com with SMTP id t11so2796216wey.28 for ; Sun, 10 Mar 2013 08:04:42 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <513CA107.5030602@redhat.com> Date: Sun, 10 Mar 2013 16:04:39 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1362811730-6716-1-git-send-email-pbonzini@redhat.com> <1362811730-6716-3-git-send-email-pbonzini@redhat.com> <20130310115439.GN11223@redhat.com> <513C9876.6090708@redhat.com> <20130310145524.GJ24444@redhat.com> In-Reply-To: <20130310145524.GJ24444@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH uq/master 2/2] kvm: forward INIT signals coming from the chipset List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gleb Natapov Cc: jan.kiszka@siemens.com, mtosatti@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org Il 10/03/2013 15:55, Gleb Natapov ha scritto: > > > Why not move INIT case from below as is? Vcpu is reset to correct sate > > > by QEMU just like during system_reset. > > > > APs would not be able to receive SIPIs after executing do_cpu_init, > > because they would stay in KVM_MP_STATE_RUNNABLE state. > > If APs are in runnable state after reset with in kernel irq chip we > have a bug somewhere. Here is where we are resetting the processor. After clearing CPU_INTERRUPT_INIT, no matter what else we do (such as resetting the APIC and CPU), we need to set the mp_state to KVM_MP_STATE_INIT_RECEIVED. Or if we go with your simpler hypervisor patch, we need to go to either KVM_MP_STATE_INIT_RECEIVED for APs (wait for SIPI) or KVM_MP_STATE_SIPI_RECEIVED for the BSP (restart running from the reset vector). > Should AP be able to get SIPI without INIT after trigger of INIT# line? Yes, the effect is the same for an INIT interrupt and the triggering of INIT#. Paolo