The disk isn't guaranteed to be the first device in the config sxp --

make sure we can restart domains when using a boot loader even if not.

Signed-off-by: Jeremy Katz <katzj@redhat.com>
This commit is contained in:
kaf24@firebug.cl.cam.ac.uk committed 2006-02-26 10:55:21 +01:00
1 parent b3a1978719
commit 91bc5c3347
1 file changed
+5 -6
+5 -6
View File
@@ -1502,15 +1502,14 @@ class XendDomainInfo:
if not self.info['bootloader']:
return
# if we're restarting with a bootloader, we need to run it
# FIXME: this assumes the disk is the first device and
# that we're booting from the first disk
blcfg = None
config = self.sxpr()
# FIXME: this assumes that we want to use the first disk
dev = sxp.child_value(config, "device")
if dev:
disk = sxp.child_value(dev, "uname")
fn = blkdev_uname_to_file(disk)
for dev in sxp.children(config, "device"):
disk = sxp.child(dev, "vbd")
if disk is None:
continue
fn = blkdev_uname_to_file(sxp.child_value(disk, "uname"))
blcfg = bootloader(self.info['bootloader'], fn, 1,
self.info['vcpus'])
if blcfg is None:
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.