From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ross Philipson <ross.philipson@gmail.com>
Subject: Re: ACPI shutdown unreliable with win7?
Date: Fri, 29 May 2015 12:11:53 -0400
Message-ID: <55688FC9.1030803@gmail.com>
References: <1432284841.10746.136.camel@citrix.com>	<555EEFD8.8020306@citrix.com>
	<1432285699.10746.147.camel@citrix.com>	<1432904091.15036.27.camel@citrix.com>	<55688018020000780007F06C@mail.emea.novell.com>	<1432905214.15036.31.camel@citrix.com>	<9AAE0902D5BC7E449B7C8E4E778ABCD02591CA56@AMSPEX01CL01.citrite.net>	<1432910144.15036.53.camel@citrix.com>	<9AAE0902D5BC7E449B7C8E4E778ABCD02591CD3C@AMSPEX01CL01.citrite.net>
	<1432912292.15036.64.camel@citrix.com> <55688716.7040304@gmail.com>
	<9AAE0902D5BC7E449B7C8E4E778ABCD02591CEEF@AMSPEX01CL01.citrite.net>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <9AAE0902D5BC7E449B7C8E4E778ABCD02591CEEF@AMSPEX01CL01.citrite.net>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Paul Durrant <Paul.Durrant@citrix.com>, Ian Campbell <Ian.Campbell@citrix.com>
Cc: Andrew Cooper <Andrew.Cooper3@citrix.com>, Ian Jackson <Ian.Jackson@citrix.com>, Jan Beulich <JBeulich@suse.com>, xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 05/29/2015 12:00 PM, Paul Durrant wrote:
>> -----Original Message-----
>> From: Ross Philipson [mailto:ross.philipson@gmail.com]
>> Sent: 29 May 2015 16:35
>> To: Ian Campbell; Paul Durrant
>> Cc: Andrew Cooper; xen-devel; Jan Beulich; Ian Jackson
>> Subject: Re: [Xen-devel] ACPI shutdown unreliable with win7?
>>
>> On 05/29/2015 11:11 AM, Ian Campbell wrote:
>>> On Fri, 2015-05-29 at 16:06 +0100, Paul Durrant wrote:
>>>>> FWIW We appear to wait 200s, if we were seeing failures due to
>> windows
>>>>> update then I'd be inclined to extend that, but I think right now that
>>>>> would be premature, unless WU happens with no status on the screen.
>>>>>
>>>>
>>>> No, you'd see something. Perhaps our ACPI lid/power switch code is just
>> buggy then?
>>>
>>> It seems to work reliably for the WinXP tests, FWIW...
>>>
>>> Ian.
>>>
>>
>> One thing I find confusing is that the firmware code does not even have
>> a power button device (PNP0C0C) or the fixed feature power button that
>> is enabled in the FADT (flag == PWR_BUTTON bit 4). So I don't see how
>> the shutdown is purely an ACPI function. Is there something else to the
>> story? Is it relying on PV tools to do it?
>>
>
> Xen (and QEMU seemingly) implement the 'Fixed Power Button' (section 4.8.2.2.1.1 on my spec) and this requires the PWR_BUTTON flag to be clear (according table 4-13). It also does not require a power button device to be implemented (which is presumably why this way of doing it was chosen).

Ah got it. I read the spec backwards - that the bit was set not clear 
for the fixed feature power button :)


>
>    Paul
>
>> Ross
>>
>>>
>>>
>>> _______________________________________________
>>> Xen-devel mailing list
>>> Xen-devel@lists.xen.org
>>> http://lists.xen.org/xen-devel
>>>
>>
>>
>> --
>> Ross Philipson


-- 
Ross Philipson