Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-09 Thread S.Sakamoto
> > > The attached patch implements the domain autostart commands for > > > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > > > on_xend_start field which can be used to autostart a domain. > > > > > It is a good patch for me. > > It moved without a problem when I tested it. > > > >

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-09 Thread Daniel Veillard
On Thu, May 08, 2008 at 01:49:15PM -0400, Cole Robinson wrote: > Cole Robinson wrote: > > The attached patch implements the domain autostart commands for > > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > > on_xend_start field which can be used to autostart a domain. > > > > Updat

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread Cole Robinson
Cole Robinson wrote: > The attached patch implements the domain autostart commands for > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > on_xend_start field which can be used to autostart a domain. > Updated patch with fixes Dan recommended. Thanks, Cole diff --git a/src/xen_unif

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread Daniel P. Berrange
On Wed, May 07, 2008 at 04:21:25PM -0400, Cole Robinson wrote: > +int > +xenDaemonDomainGetAutostart(virDomainPtr domain, > +int *autostart) > +{ > +struct sexpr *root; > +const char *tmp; > + > +if ((domain == NULL) || (domain->conn == NULL) || (domain->nam

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread Daniel P. Berrange
On Wed, May 07, 2008 at 04:21:25PM -0400, Cole Robinson wrote: > The attached patch implements the domain autostart commands for > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > on_xend_start field which can be used to autostart a domain. > > A couple things: > > 1) This works on

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread Daniel P. Berrange
On Thu, May 08, 2008 at 06:42:13PM +0900, S.Sakamoto wrote: > > The attached patch implements the domain autostart commands for > > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > > on_xend_start field which can be used to autostart a domain. > > > It is a good patch for me. > It mo

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread Atsushi SAKAI
Hi, Cole Cole Robinson <[EMAIL PROTECTED]> wrote: > 1) This works on a running guest, but will only show the >sexpr changes after the guest is restarted. Just curious >if there is any better way to do this? This is current Xen behavior. If you need to work as you hope, you should change

Re: [libvirt] [PATCH] Implement autostart commands for xen

2008-05-08 Thread S.Sakamoto
> The attached patch implements the domain autostart commands for > xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a > on_xend_start field which can be used to autostart a domain. > It is a good patch for me. It moved without a problem when I tested it. By the way, I have a question.

[libvirt] [PATCH] Implement autostart commands for xen

2008-05-07 Thread Cole Robinson
The attached patch implements the domain autostart commands for xen. The xen sexpr (since at least 3.0.4 = 1.5 years) has a on_xend_start field which can be used to autostart a domain. A couple things: 1) This works on a running guest, but will only show the sexpr changes after the guest is r