From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:53134) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RYDLz-0005xD-BF for qemu-devel@nongnu.org; Wed, 07 Dec 2011 03:59:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RYDLy-0000Gf-AD for qemu-devel@nongnu.org; Wed, 07 Dec 2011 03:59:51 -0500 Received: from mx1.redhat.com ([209.132.183.28]:8745) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RYDLx-0000GO-Qm for qemu-devel@nongnu.org; Wed, 07 Dec 2011 03:59:50 -0500 Message-ID: <4EDF2BC0.10607@redhat.com> Date: Wed, 07 Dec 2011 10:02:56 +0100 From: Kevin Wolf MIME-Version: 1.0 References: <4EDC8D06.20308@msgid.tls.msk.ru> <4EDCC6FE.8040702@redhat.com> <4EDD2763.8010808@msgid.tls.msk.ru> <4EDDEF26.9030403@redhat.com> <4EDDF659.6040701@msgid.tls.msk.ru> <20111206122752.GA31385@redhat.com> <4EDE2AA0.7090001@msgid.tls.msk.ru> <4EDE42EB.2020101@msgid.tls.msk.ru> <4EDE450B.6090801@siemens.com> <4EDE498A.2040309@msgid.tls.msk.ru> <4EDE5B3E.7000003@msgid.tls.msk.ru> <4EDE5D2A.2020809@siemens.com> In-Reply-To: <4EDE5D2A.2020809@siemens.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] winXP "Standard PC" HAL and qemu-kvm >= 0.15 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: KVM list , "Michael S. Tsirkin" , "Justin M. Forbes" , Michael Tokarev , "qemu-devel@nongnu.org" , Isaku Yamahata , Avi Kivity Am 06.12.2011 19:21, schrieb Jan Kiszka: > On 2011-12-06 19:13, Michael Tokarev wrote: >> On 06.12.2011 20:57, Michael Tokarev wrote: >>> On 06.12.2011 20:38, Jan Kiszka wrote: >>>> On 2011-12-06 17:29, Michael Tokarev wrote: >>> [] >>>>> It appears there are two issues here, one is fixed by >>>>> 09de0f469c3c2a277c7874f6c60992c8b94719a9 and is 32bit-only, and >>>>> another bisect leads to this commit: >>> >>> Or 3... :) >>> >>>>> commit 59539c913383fdd3350681301b44f02fa7ee2757 >>>>> Author: Jan Kiszka >>>>> Date: Mon Jun 27 12:22:28 2011 +0200 >>>>> >>>>> qemu-kvm: Fix in-kernel PIC reset >>> >>> [] >>>>> Anything wrong with this patch? >>>> >>>> I tend to say "no". It may just reveals some issue elsewhere. >>>> >>>> To cross-check: Does this series [1] expose the same issue with vanilla >>>> QEMU when enabling that in-kernel irqchip version? >>> >>> I cross-checked it the other way, which really should have been done >>> at the very beginning instead of wasting so much time of so many >>> people at once. I just tried qemu-kvm-1.0, which I wasn't due to >>> an unrelated issue (1.0 does not work for me in my funky 32/64bit >>> environment). Also, since the bug has been reported especially >>> against 0.15 version (triggering by upgrading from 0.14 to 0.15), >>> I didn't insist on trying 1.0, which was my biggest mistake. >>> >>> And in qemu-kvm-1.0, the problem guest Just Works. So i t must be >>> something else which were fixed between 0.15 and 1.0. >> >> For the 0.15 .. 1.0 change, the first commit which restores the (broken >> in 0.15) functionality is this one: >> >> commit 86fbf97ceb4a9c46a609dd4ae053ba4262b68fe8 >> Author: Jan Kiszka >> Date: Fri Oct 7 09:19:45 2011 +0200 >> >> i8259: Clear ELCR on reset >> >> The ELCR is actually part of the chipset but we model it here for >> simplicity reasons. The PIIX3 clears the ELCR on reset, which was once >> broken by 4dbe19e181. Fix this by splitting up pic_init_reset from >> pic_reset and clearing the register in the latter. >> >> Signed-off-by: Jan Kiszka >> Signed-off-by: Blue Swirl >> >> Which is quite expected having in mind the commit which "broke" >> it for 0.15. > > Yep, makes a lot of sense. That patch should be applied to stable then > (who's in charge?). Justin, I guess (CCed). Not sure if we're planning to have another 0.15.x release, though. Kevin