From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: system_powerdown not working for qemu-kvm 0.12.4? Date: Tue, 12 Oct 2010 08:49:58 +0200 Message-ID: <4CB40515.2030809@redhat.com> References: <4BEFDC8D.70007@redhat.com> <4BF41051.2010608@redhat.com> <4C67C8D2.5000400@redhat.com> <4CB341E4.3060400@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Teck Choon Giam , kvm@vger.kernel.org, Gleb Natapov , "'Kevin O'Connor'" , seabios@seabios.org To: Ruben Kerkhof Return-path: Received: from mx1.redhat.com ([209.132.183.28]:52312 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755373Ab0JLGul (ORCPT ); Tue, 12 Oct 2010 02:50:41 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On 10/11/2010 07:53 PM, Ruben Kerkhof wrote: > On Mon, Oct 11, 2010 at 18:57, Avi Kivity wrote: > > On 10/11/2010 06:29 PM, Ruben Kerkhof wrote: > >> > >> Hi Avi, > >> > >> On Sun, Aug 15, 2010 at 13:00, Avi Kivity wrote: > >> > >> > I'm betting 73b48d914f9 is the cause, but let's see the full bisect. > >> > >> system_powerdown with FreeBSD 8.1 works on 73b48d914f9. > >> > >> I've bisected the issue to commit 76e617c. > >> > >> Let me know if I should do some other tests. > > > > It works on 73b48d914f9 but fails on 76e617c due to the seabios change. > > > > Can you try 76e617c but with SeaBIOS 0.5.0? If that works, keep qemu-kvm on > > 76e617c but bisect SeaBIOS from 0.5.0 to 0.5.1 to see which commit broke > > freebsd. > > > > Sure, > > 5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 is the first bad commit > commit 5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 > Author: Kevin O'Connor > Date: Wed Dec 30 12:36:22 2009 -0500 > > Commit compiled dsdt file; misc comment updates. > > Commit new dsdt with recent changes. > Add some misc comments. > Also, fix uninitialized warning in mptable code. > > Regards, Ruben Gleb, Kevin, any ideas? (summary: qemu-kvm doesn't acpi shutdown freebsd 8.1 with this commit; qemu.git does. May be due to interrupt polarity which kvm implements but qemu does not) -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.