From mboxrd@z Thu Jan  1 00:00:00 1970
From: George Dunlap <george.dunlap@eu.citrix.com>
Subject: Re: Early ACPI events prevent subsequent ACPI
 functionality on xen 4.3 + HVM domU
Date: Tue, 21 May 2013 15:34:34 +0100
Message-ID: <519B85FA.6060809@eu.citrix.com>
References: <752956577.8634488.1368537190555.JavaMail.root@zimbra002>
	<1368812151.24012.66.camel@hastur.hellion.org.uk>
	<71194FD00ABB7415DD6106AA@nimrod.local>
	<CAFLBxZYpXV5noq3pM=O-6BtbfOD765c6k_wP_x720Wz7d1jabg@mail.gmail.com>
	<CAFLBxZYCQdPU3_iTj_j-OAcXy2q0e=3-V32=xb-sXdEC4-T=6Q@mail.gmail.com>
	<1369146037.21246.89.camel@zakaz.uk.xensource.com>
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: <1369146037.21246.89.camel@zakaz.uk.xensource.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>
Cc: Dave Scott <Dave.Scott@eu.citrix.com>, Konrad Wilk <konrad.wilk@oracle.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Paul Durrant <paul.durrant@citrix.com>, Stefano Stabellini <stefano.stabellini@citrix.com>, Alex Bligh <alex@alex.org.uk>, Anthony Perard <anthony.perard@citrix.com>, Diana Crisan <dcrisan@flexiant.com>
List-Id: xen-devel@lists.xenproject.org

On 05/21/2013 03:20 PM, Ian Campbell wrote:
> On Tue, 2013-05-21 at 15:16 +0100, George Dunlap wrote:
>> And will the Citrix PV drivers check for the existence of
>> shutdown commands before they start a watch?
>
> Watches fire immediately when you register them, to allow this race to
> be easily avoided without really having to think about it.

Well then that watch mechanism is broken, at least for values written 
before some point in time:

# xenstore-ls -f /local/domain/10
[...]
/local/domain/10/control/shutdown = "poweroff"
[...]

But the domain doesn't shut down.

Konrad, sorry to keep sending stuff your way, but I'm not sure who else 
to bring in for this one.  The original thread is here:

http://www.gossamer-threads.com/lists/xen/devel/282467

But the problem actually has nothing to do with ACPI -- it's the PV 
shutdown command that is getting lost after the driver registers 
HVM_PARAM_CALLBACK_IRQ but before something else comes up that starts 
watching ../control/shutdown.

  -George