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 -=|