WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH][LINUX] use machine_emergency_restart() from the

To: Jimi Xenidis <jimix@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH][LINUX] use machine_emergency_restart() from the generic header.
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Tue, 08 Aug 2006 13:26:40 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 08 Aug 2006 05:27:03 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <A1DBF1B2-384E-407C-86B7-DD46281625AC@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Aca65eZCJJ7bkCbZEduaRAAKle7CWA==
Thread-topic: [Xen-devel] [PATCH][LINUX] use machine_emergency_restart() from the generic header.
User-agent: Microsoft-Entourage/11.2.5.060620


On 8/8/06 12:49 pm, "Jimi Xenidis" <jimix@xxxxxxxxxxxxxx> wrote:

> I have a linker problem building this file in POWERPC as you may
> suspect.
> The ./driver/xen/ file is more like the other archs than x86, and
> should be more portable.
> 
> Not sure what header magik would be needed here, if you think its
> worthwhile I'll pursue it.

You could switch the call order in drivers/xen/core/reboot.c so that
machine_emergency_restart calls machine_restart, and then conditionally
compile the former only for x86. Not super nice but it's at least
straightforward. I'd take a patch that did that.

 -- Keir



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