From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: ACPI shutdown unreliable with win7?
Date: Fri, 22 May 2015 09:59:04 +0100
Message-ID: <555EEFD8.8020306@citrix.com>
References: <1432284841.10746.136.camel@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1432284841.10746.136.camel@citrix.com>
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: Ian Campbell <ian.campbell@citrix.com>, xen-devel <xen-devel@lists.xen.org>
Cc: Paul Durrant <paul.durrant@citrix.com>, Ian Jackson <Ian.Jackson@eu.citrix.com>, Jan Beulich <JBeulich@suse.com>
List-Id: xen-devel@lists.xenproject.org

On 22/05/15 09:54, Ian Campbell wrote:
> In osstest self gate flight 56412 a change to make HVM guests
> configurably use "xl shutdown -F" (-F == use ACPI fallback if PV drivers
> absent) and apply the option to the win7 and winxpsp3 tests went into
> production.
>
> On winxpsp3 this appears to have been a success and things are working
> pretty well since that flight:
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemuu-winxpsp3.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemuu-winxpsp3-vcpus1.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-amd64-xl-qemut-winxpsp3.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemut-winxpsp3.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemut-winxpsp3-vcpus1.html
>
> However win7 seems to be in somewhat poorer shape:
>
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-amd64-xl-qemuu-win7-amd64.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemuu-win7-amd64.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-amd64-xl-qemut-win7-amd64.html
> http://logs.test-lab.xenproject.org/osstest/results/history.test-amd64-i386-xl-qemut-win7-amd64.html
>
> it passes less than one time in 10 (of the ones which don't get blocked
> earlier on).
>
> The problem seems to be independent of the qemu version and with qemuu
> of seabios vs rombios. dom0 kernel also seems irrelevant.

This is because there is no such thing as ACPI shutdown.

There is ACPI "Someone pressed the power button" and ACPI "Someone
closed the laptop lid".

I believe Win7 attempts to sleep by default, rather than to shut down,
although IIRC this does depend on which S states are offered in the ACPI
tables.  One thing to try would be to boot windows without any S3 or S4
SSDT tables, which should persuade windows into believing that "someone
pressed the power button" should mean S5 shutdown.

~Andrew