Re: [Libvir] [PATCH] Xen driver doesn't support autostart, so remove it from xen_unified
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).
I can't remember the details offhand, but there's a new field in the
SEXPR for domains that lets you mark them as autostart, so we could
probably figure out some form of hacky implementation for the new
scheme. The old symlink way is easy- we just re-use the code from
the QEMU driver which does the same symlink magic
Dan.
--
|=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=|
|=- Perl modules: http://search.cpan.org/~danberr/ -=|
|=- Projects: http://freshmeat.net/~danielpb/ -=|
|=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|