[Fwd: Re: [RFC][PATCH] allow dom0 to power off machine using ACPI on shutdown.]

[Fwd: Re: [RFC][PATCH] allow dom0 to power off machine using ACPI on shutdown.]

[Harry Butterworth]

[-- Attachment #1: Type: text/plain, Size: 292 bytes --]

Ewan,

    Keir seems to think this patch is OK.  I tested it on an
Intellistation A Pro for x86_64 and it works for me.  I think it is low
risk because it's executed at the end of the shutdown process.  Please
apply.

http://lists.xensource.com/archives/html/xen-devel/2006-04/msg00223.html

[-- Attachment #2: Forwarded message - Re: [Xen-devel] [RFC][PATCH] allow dom0 to power off machine using ACPI on shutdown. --]
[-- Type: message/rfc822, Size: 4316 bytes --]

From: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
To: Harry Butterworth <harry@hebutterworth.freeserve.co.uk>
Cc: xen-devel@lists.xensource.com
Subject: Re: [Xen-devel] [RFC][PATCH] allow dom0 to power off machine using ACPI on shutdown.
Date: Fri, 7 Apr 2006 08:27:50 +0100
Message-ID: <461bea0b36f0398882dec8610f691f54@cl.cam.ac.uk>


On 6 Apr 2006, at 22:22, Harry Butterworth wrote:

> It might be better to install the hypervisor call in pm_power_off and
> let it get replaced by the acpi_power_off call if ACPI is detected.
> That would seem to fit in better with the way the x86 arch handles
> multiple power off strategies.
>
> I haven't looked to see how the other architectures do power off yet.
>
> Was hoping for some comment about whether to do power off in xen or in
> domU and whether the hypervisor needed notification.

I think your patch looked okay for now, actually. Alternatively our 
machine_restart() could be moved into arch-specific code files. Doing 
the power off in dom0 is fine -- as you say, Xen doesn't do anyhting 
special before shutdown.

  -- Keir


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel


[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel