On Thu, Feb 22, 2007 at 01:51:11AM -0500, Daniel Veillard wrote:
On Wed, Feb 21, 2007 at 04:31:16PM +0000, Daniel P. Berrange wrote:
> Because of the way Xen guest creation works, the virDomainCreateLinux
> implementation for Xen is really a three stage process - first we create
> the raw domain, then we have to wait for device hotplug to complete,
> finally unpausing the domain.
>
> Currently if the device hotplug fails we error out, leaving a 'braindead'
> VM lying around in the paused state. This is rather confusing for users
> leading to people thinking everything was OK, and thus unpausing the
> guest manually, then filing bug reports when they find they've no devices
> in the guest VM !
Makes sense. I'm just not clear why code need to be moved from one function
to another, but I'm lacking context ATM.
It isn't moved from one function to another - there are two fairly similar
functions - one deals with creating fresh domains (in xend_internal.c) and
the other deals with starting inactive domains (in xm_internal.c). Both needed
a fairly similar fix.
Regards,
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 -=|