# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID c9bbfb42ba84e7576516021cd33d6fec55f52270
# Parent a5994e258f777703b46a8433238bd72c8bf8c1cb
Fix config file parsing for VMX domains.
If we define "vif" in the config file, image.py will raise error "vmx:
missing vbd configuration". The reason is "vif" is dealt with as a "vbd"
device.
This patch fixes this issue by dealing with "vbd" and "vif " separately,
removing "macaddr" arg and parsing mac address from "vif" instead.
Also, the vbd doesn't have to be a file anymore, but can be a physical
disk partition.
Signed-off-by: Yunfeng Zhao <yunfeng.zhao@xxxxxxxxx>
Signed-off-by: Arun Sharma <arun.sharma@xxxxxxxxx>
diff -r a5994e258f77 -r c9bbfb42ba84 tools/examples/xmexample.vmx
--- a/tools/examples/xmexample.vmx Thu Aug 11 20:57:09 2005
+++ b/tools/examples/xmexample.vmx Thu Aug 11 20:58:08 2005
@@ -132,8 +132,3 @@
#-----------------------------------------------------------------------------
# start in full screen
#full-screen=1
-
-#-----------------------------------------------------------------------------
-# set the mac address of the first interface
-#macaddr=
-
diff -r a5994e258f77 -r c9bbfb42ba84 tools/python/xen/xend/image.py
--- a/tools/python/xen/xend/image.py Thu Aug 11 20:57:09 2005
+++ b/tools/python/xen/xend/image.py Thu Aug 11 20:58:08 2005
@@ -305,7 +305,7 @@
# xm config file
def parseDeviceModelArgs(self):
dmargs = [ 'cdrom', 'boot', 'fda', 'fdb',
- 'localtime', 'serial', 'macaddr', 'stdvga', 'isa' ]
+ 'localtime', 'serial', 'stdvga', 'isa' ]
ret = []
for a in dmargs:
v = sxp.child_value(self.vm.config, a)
@@ -322,20 +322,25 @@
ret.append("-%s" % a)
ret.append("%s" % v)
- # Handle hd img related options
+ # Handle disk/network related options
devices = sxp.children(self.vm.config, 'device')
for device in devices:
- vbdinfo = sxp.child(device, 'vbd')
- if not vbdinfo:
- raise VmError("vmx: missing vbd configuration")
- uname = sxp.child_value(vbdinfo, 'uname')
- vbddev = sxp.child_value(vbdinfo, 'dev')
- (vbdtype, vbdparam) = string.split(uname, ':', 1)
- vbddev_list = ['hda', 'hdb', 'hdc', 'hdd']
- if vbdtype != 'file' or vbddev not in vbddev_list:
- raise VmError("vmx: for qemu vbd type=file&dev=hda~hdd")
- ret.append("-%s" % vbddev)
- ret.append("%s" % vbdparam)
+ name = sxp.name(sxp.child0(device))
+ if name == 'vbd':
+ vbdinfo = sxp.child(device, 'vbd')
+ uname = sxp.child_value(vbdinfo, 'uname')
+ vbddev = sxp.child_value(vbdinfo, 'dev')
+ (vbdtype, vbdparam) = string.split(uname, ':', 1)
+ vbddev_list = ['hda', 'hdb', 'hdc', 'hdd']
+ if vbddev not in vbddev_list:
+ raise VmError("vmx: for qemu vbd type=file&dev=hda~hdd")
+ ret.append("-%s" % vbddev)
+ ret.append("%s" % vbdparam)
+ if name == 'vif':
+ vifinfo = sxp.child(device, 'vif')
+ mac = sxp.child_value(vifinfo, 'mac')
+ ret.append("-macaddr")
+ ret.append("%s" % mac)
# Handle graphics library related options
vnc = sxp.child_value(self.vm.config, 'vnc')
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
Home