xend: Add syntax check for block devices

If we define wrong values to a disk parameter in domain configuration
files, we get an error message or a guest OS panic.

 1. If we define a wrong disk type, xm create command error occurs
    about 100 seconds later.
      e.g.  disk=['xyz:/xen/root-vm1.img,hda1,w']

 2. If we forget a disk type, a guest OS panic occurs.
      e.g.  disk=['/xen/root-vm1.img,hda1,w']

Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com>
This commit is contained in:
Keir Fraser committed 2008-05-12 11:19:09 +01:00
1 parent 543cb6d751
commit 499681128c
1 file changed
+5 -1
+5 -1
View File
@@ -56,8 +56,12 @@ class BlkifController(DevController):
else:
try:
(typ, params) = string.split(uname, ':', 1)
if typ not in ('phy', 'file'):
raise VmError(
'Block device must have "phy" or "file" specified to type')
except ValueError:
(typ, params) = ("", "")
raise VmError(
'Block device must have physical details specified')
mode = config.get('mode', 'r')
if mode not in ('r', 'w', 'w!'):
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.