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

[Xen-devel] Re: [RFC PATCH 26/35] Add Xen subarch reboot support

On Wed, 2006-03-22 at 10:22 +0000, Keir Fraser wrote:
> On 22 Mar 2006, at 08:40, Arjan van de Ven wrote:
> 
> >> +static int shutdown_process(void *__unused)
> >> +{
> >> +  static char *envp[] = { "HOME=/", "TERM=linux",
> >> +                          "PATH=/sbin:/usr/sbin:/bin:/usr/bin", NULL };
> >> +  static char *restart_argv[]  = { "/sbin/reboot", NULL };
> >> +  static char *poweroff_argv[] = { "/sbin/poweroff", NULL };
> >
> > how is this function different from the generic one? If not, why aren't
> > you using the generic one?
> 
> The intent is to allow remote management tools to trigger a clean 
> shutdown of the virtual machine. That requires us to notify to 
> userspace, and this function does that by exec'ing one of the standard 
> userspace programs. Given the trigger is received by the kernel in the 
> first instance I don't know a better way of doing this. And if this is 
> the best way, I don't think there is generic code in the kernel which 
> does the same thing.


well this isn't really different from the normal ctrl-alt-delete right?
I would strongly suggest to follow the normal ctrl-alt-del path.. that
follows the normal convention sysadmins are used to.
It's not "/sbin/poweroff" fwiw... at least not hardcoded. Following the
normal ctrl-alt-del codepath gets all the policy out of this kind of
thing as well..



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

<Prev in Thread] Current Thread [Next in Thread>