From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=48594 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Pkw0c-0001ip-Ea for qemu-devel@nongnu.org; Thu, 03 Feb 2011 05:01:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Pkw0b-00005T-GB for qemu-devel@nongnu.org; Thu, 03 Feb 2011 05:01:50 -0500 Received: from mx1.redhat.com ([209.132.183.28]:46388) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Pkw0b-000055-6n for qemu-devel@nongnu.org; Thu, 03 Feb 2011 05:01:49 -0500 Message-ID: <4D4A7D08.6070900@redhat.com> Date: Thu, 03 Feb 2011 12:01:44 +0200 From: Avi Kivity MIME-Version: 1.0 Subject: Re: [Qemu-devel] KVM: Windows 64-bit troubles with user space irqchip References: <4D4946F7.1070702@siemens.com> <20110202123532.GF14984@redhat.com> <4D4952FA.8020300@siemens.com> <4D49569F.6060207@redhat.com> <4D496A8D.90000@siemens.com> <4D496BC5.10807@redhat.com> <4D496D77.2010405@siemens.com> <4D496FA6.8070301@siemens.com> <4D49738D.7080404@redhat.com> <4D4979BD.6080900@siemens.com> <20110202154611.GR14984@redhat.com> <4D497DAB.7010901@siemens.com> <4D4A64F2.8010309@redhat.com> <4D4A7629.1010506@siemens.com> In-Reply-To: <4D4A7629.1010506@siemens.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: kvm , Gleb Natapov , qemu-devel On 02/03/2011 11:32 AM, Jan Kiszka wrote: > On 2011-02-03 09:18, Avi Kivity wrote: > > On 02/02/2011 05:52 PM, Jan Kiszka wrote: > >>>> > >>> If there is no problem in the logic of this commit (and I do not see > >>> one yet) then we somewhere miss kicking vcpu when interrupt, that should be > >>> handled, arrives? > >> > >> I'm not yet confident about the logic of the kernel patch: mov to cr8 is > >> serializing. If the guest raises the tpr and then signals this with a > >> succeeding, non vm-exiting instruction to the other vcpus, one of those > >> could inject an interrupt with a higher priority than the previous tpr, > >> but a lower one than current tpr. QEMU user space would accept this > >> interrupt - and would likely surprise the guest. Do I miss something? > > > > apic_get_interrupt() is only called from the vcpu thread, so it should > > see a correct tpr. > > > > The only difference I can see with the patch is that we may issue a > > spurious cpu_interrupt(). But that shouldn't do anything bad, should it? > > I tested this yesterday, and it doesn't confuse Windows. It actually > receives quite a few spurious IRQs in normal operation, w/ or w/o the > kernel's tpr optimization. I don't see why there should be any spurious interrupts in normal operation. From the docs, these happen due to an INTA cycle racing with raising the TPR, but in ioapic mode, there shouldn't be any INTA cycles. -- error compiling committee.c: too many arguments to function