xm: Do not check path of kernel if bootloader is specified

When create DomU, if bootloader is specified, 'kernel/ramdisk' will be
used by bootloader when boots DomU. So it is needless to check the
path is existent or not.

Signed-off-by: Yu Zhiguo <yuzg@cn.fujitsu.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
This commit is contained in:
Ian Jackson committed 2010-07-15 16:32:50 +01:00
1 parent fb1d9a4ad1
commit 8582d429a1
1 file changed
+10 -2
+10 -2
View File
@@ -708,7 +708,12 @@ def configure_image(vals):
return None
config_image = [ vals.builder ]
if vals.kernel:
if os.path.dirname(vals.kernel) != "" and os.path.exists(vals.kernel):
if vals.bootloader:
# If bootloader is specified, vals.kernel will be used
# by bootloader when boots DomU. So it is needless to
# check the path is existent or not.
config_image.append([ 'kernel', vals.kernel ])
elif os.path.dirname(vals.kernel) != "" and os.path.exists(vals.kernel):
config_image.append([ 'kernel', vals.kernel ])
elif vals.kernel == 'hvmloader':
# Keep hvmloader w/o a path and let xend find it.
@@ -721,7 +726,10 @@ def configure_image(vals):
else:
raise ValueError('Cannot find kernel "%s"' % vals.kernel)
if vals.ramdisk:
if os.path.dirname(vals.ramdisk) != "" and os.path.exists(vals.ramdisk):
if vals.bootloader:
# Same with 'kernel' above
config_image.append([ 'ramdisk', vals.ramdisk ])
elif os.path.dirname(vals.ramdisk) != "" and os.path.exists(vals.ramdisk):
config_image.append([ 'ramdisk', vals.ramdisk ])
elif os.path.exists(os.path.abspath(vals.ramdisk)):
# Keep old behaviour, if path is valid.
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.