[Libvir] [PATCH] Xen driver doesn't support autostart, so remove it from xen_unified

Daniel Veillard veillard at redhat.com
Fri Jun 22 13:43:21 UTC 2007


On Fri, Jun 22, 2007 at 02:08:10PM +0100, Richard W.M. Jones wrote:
> 
> Our implementation of the Xen driver doesn't support autostart[*]. 
> However xen_unified.c has a loop for get/setAutostart which causes it to 
> return an error without setting virterror first.  This patch removes 
> that loop.
> 
> Rich.
> 
> [*] Although Xen supports it, by putting files in /etc/xen/autostart/ 
> (at least that was the "old Xen" way before the current insanity of 
> getting rid of config files was invented).

  Hum, if we are desesperate of ever adding the support to Xen,
then okay, but wouldn't it be better to emit the error before returning
in case no Xen sub-driver supports it ?

  just wondering, +1 otherwise.

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard at redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/




More information about the libvir-list mailing list