[Xen-devel] QEMU PV guests and xenstore_parse_domain_config()

["Pat Campbell" <plc@xxxxxxxxxx>]

New 3.2 QEMU PV guests are calling xenstore-parse_domain_config() opening 
block devices and setting watches that might not be necessary.

Seems like each machine type should have it's own specific xenstore parse
routine if it needs one.  

I have attached a patch that adds a xenstore parse pointer into the QEMUMachine 
struct.  main()  will call that machine specific function if set otherwise 
calls 
generic xenstore-parse_domain_config.  Attached patch is just something as a 
starting point for discussion not something I propose be applied.

----------------------------------
Some dump log traces
----------------------------------
qemu.log of PV guest running current xen-unstable tip
 xenstore_parse_domain_config()
 Change xvda to look like hda
 bdrv_open filename:/var/lib/xen/images/sles10-sp2-pv/disk0()
 bdrv_open2 filename:/var/lib/xen/images/sles10-sp2-pv/disk0()
 bdrv_open2 filename:/var/lib/xen/images/sles10-sp2-pv/disk0()
 Change xvdb to look like hdb
 bdrv_open filename:/home/tests/iso/sles/SLES-10-SP2-DVD-i386-Alpha2-DVD1.iso()
 bdrv_open2 filename:/home/tests/iso/sles/SLES-10-SP2-DVD-i386-Alpha2-DVD1.iso()
 bdrv_open2 filename:/home/tests/iso/sles/SLES-10-SP2-DVD-i386-Alpha2-DVD1.iso()
 Change xvdc to look like hdc
 bdrv_open filename:/dev/cdrom()
 bdrv_open2 filename:/dev/cdrom()
 Watching /local/domain/0/device-model/8/logdirty/next-active
 Watching /local/domain/0/device-model/8/command
 shift keysym 003e keycode 86

-----------------------------------
qemu.log of PV guest PV guest running current xen-unstable tip + patch
   xenstore_open()
   xen_parse_xenstore()
   shift keysym 003e keycode 86

------------------------------------
qemu.log of FV guest with proposed patch  ( no change )
 xenstore_open()
 xenstore_parse_domain_config()
 bdrv_open filename:/var/lib/xen/images/sles10-sp2/disk0()
 bdrv_open2 filename:/var/lib/xen/images/sles10-sp2/disk0()
 bdrv_open2 filename:/var/lib/xen/images/sles10-sp2/disk0()
 bdrv_open filename:/home/tests/iso/sles/SLES-10-SP1-DVD-i386-RC5-DVD1.iso()
 bdrv_open2 filename:/home/tests/iso/sles/SLES-10-SP1-DVD-i386-RC5-DVD1.iso()
 bdrv_open2 filename:/home/tests/iso/sles/SLES-10-SP1-DVD-i386-RC5-DVD1.iso()
 bdrv_open filename:/dev/cdrom()
 bdrv_open2 filename:/dev/cdrom()
 Watching /local/domain/0/device-model/7/logdirty/next-active
 Watching /local/domain/0/device-model/7/command
 config qemu network with xen bridge for  tap0 xenbr0
 shift keysym 003e keycode 86

Attachment: xen-qemupv.patch
Description: Text document

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel