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 09:25:59 +0200 Message-ID: <4CB40D87.6000405@redhat.com> References: <4BEFDC8D.70007@redhat.com> <4BF41051.2010608@redhat.com> <4C67C8D2.5000400@redhat.com> <4CB341E4.3060400@redhat.com> <4CB40515.2030809@redhat.com> <20101012071124.GA10190@morn.localdomain> <20101012071404.GA18974@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "Kevin O'Connor" , Ruben Kerkhof , Teck Choon Giam , kvm@vger.kernel.org, seabios@seabios.org To: Gleb Natapov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:27613 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754044Ab0JLH0W (ORCPT ); Tue, 12 Oct 2010 03:26:22 -0400 In-Reply-To: <20101012071404.GA18974@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 10/12/2010 09:14 AM, Gleb Natapov wrote: > On Tue, Oct 12, 2010 at 03:11:24AM -0400, Kevin O'Connor wrote: > > On Tue, Oct 12, 2010 at 08:49:58AM +0200, Avi Kivity wrote: > > > On 10/11/2010 07:53 PM, Ruben Kerkhof wrote: > > > >5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 is the first bad commit > > > >commit 5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 > > > >Author: Kevin O'Connor > > > 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) > > > > The only thing in commit 5c99b6c9 that could cause an issue is that it > > has the compiled acpi changes actually made in commit 29f4b912, but I > > don't see how that would be a problem to reboots: > > > It makes line 0 of device 1 to be active low instead of active high. Correctly, according to the documentation (the line is marked IRQ9OUT#, which indicates it's active low, though that isn't set down elsewhere). Qemu however emulates it as active high. Other guests work, which is wierd. -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.