[PATCH] Allow blktap to be able to be booted as system volume for PV-on-HVM

["Kasai Takanori" <kasai.takanori@jp.fujitsu.com>]

[-- Attachment #1: Type: text/plain, Size: 1259 bytes --]

Hi All,

We were testing the PV driver on the HVM domain.
When blktap was booting system volume, PV-on-HVM domain was not able to be 
started.

The configuration file is specified as follows.
disk = [ 'tap:aio:/xen/test/rhel5ga_full.img,hda,w' ]

The error occurred by the initialization of system volume in qemu-dm.

qemu: could not open hard disk image 'aio:/xen/test/rhel5ga_full.img'

It is because "aio:" is added to the head of params in xenstore.
However, qemu-dm open device by params.

This patch corrected the problem of params on the qemu-dm.

Signed-off-by: Takanori Kasai <kasai.takanori@jp.fujitsu.com>
Signed-off-by: Tomonari Horikoshi <t.horikoshi@jp.fujitsu.com>
Signed-off-by: Tsunehisa Doi <Doi.Tsunehisa@jp.fujitsu.com>

The execution result is as follows.

# xm create -f rhel5ga_vti.conf
Using config file "./rhel5ga_vti.conf".
Started domain RHEL5GA-test2
# xm block-list RHEL5GA-test2
Vdev  BE handle state evt-ch ring-ref BE-path
768    0    0     1      -1     -1    /local/domain/0/backend/tap/75/768
...      <<= /* Waiting for booting process */
# xm block-list RHEL5GA-test2
Vdev  BE handle state evt-ch ring-ref BE-path
768    0    0     4      7      8     /local/domain/0/backend/tap/75/768

Best Regards,

--
Takanori Kasai 

[-- Attachment #2: allow-blktap-system-volume.patch --]
[-- Type: application/octet-stream, Size: 1327 bytes --]

# HG changeset patch
# User kasai.takanori@jp.fujitsu.com
# Date 1183713661 -32400
# Node ID 9d9d64fae3cd39cc7dbd398dd94ead6e413980bd
# Parent  d54d47fc8c6cdea23437476407bec05d85742760
Allow blktap to be able to be booted as system volume for PV-on-HVM

Signed-off-by: Takanori Kasai <kasai.takanori@jp.fujitsu.com>
Signed-off-by: Tomonari Horikoshi <t.horikoshi@jp.fujitsu.com>
Signed-off-by: Tsunehisa Doi <Doi.Tsunehisa@jp.fujitsu.com>

diff -r d54d47fc8c6c -r 9d9d64fae3cd tools/ioemu/xenstore.c
--- a/tools/ioemu/xenstore.c	Thu Jul 05 10:19:25 2007 +0100
+++ b/tools/ioemu/xenstore.c	Fri Jul 06 18:21:01 2007 +0900
@@ -173,10 +173,17 @@ void xenstore_parse_domain_config(int do
         }
         /* open device now if media present */
         if (params[0]) {
+            char *p = NULL;
+            if ((p = strchr(params, ':'))) {
+                p++;
+            }
+            else {
+                p = params;
+            }
             if (bdrv_open(bs_table[hd_index + (is_scsi ? MAX_DISKS : 0)],
-                          params, 0 /* snapshot */) < 0)
+                          p, 0 /* snapshot */) < 0)
                 fprintf(stderr, "qemu: could not open hard disk image '%s'\n",
-                        params);
+                        p);
         }
     }
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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