From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kasai Takanori" Subject: [PATCH] Allow blktap to be able to be booted as system volume for PV-on-HVM Date: Fri, 6 Jul 2007 19:50:06 +0900 Message-ID: <4e4701c7bfbb$c7a8b360$dab2220a@VF03007L> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_NextPart_000_4E44_01C7C006.D9D83330" Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel List-Id: xen-devel@lists.xenproject.org This is a multi-part message in MIME format. ------=_NextPart_000_4E44_01C7C006.D9D83330 Content-Type: text/plain; format=flowed; charset="iso-2022-jp"; reply-type=original Content-Transfer-Encoding: 7bit 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 Signed-off-by: Tomonari Horikoshi Signed-off-by: Tsunehisa Doi 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 ------=_NextPart_000_4E44_01C7C006.D9D83330 Content-Type: application/octet-stream; name="allow-blktap-system-volume.patch" Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="allow-blktap-system-volume.patch" # HG changeset patch=0A= # User kasai.takanori@jp.fujitsu.com=0A= # Date 1183713661 -32400=0A= # Node ID 9d9d64fae3cd39cc7dbd398dd94ead6e413980bd=0A= # Parent d54d47fc8c6cdea23437476407bec05d85742760=0A= Allow blktap to be able to be booted as system volume for PV-on-HVM=0A= =0A= Signed-off-by: Takanori Kasai =0A= Signed-off-by: Tomonari Horikoshi =0A= Signed-off-by: Tsunehisa Doi =0A= =0A= diff -r d54d47fc8c6c -r 9d9d64fae3cd tools/ioemu/xenstore.c=0A= --- a/tools/ioemu/xenstore.c Thu Jul 05 10:19:25 2007 +0100=0A= +++ b/tools/ioemu/xenstore.c Fri Jul 06 18:21:01 2007 +0900=0A= @@ -173,10 +173,17 @@ void xenstore_parse_domain_config(int do=0A= }=0A= /* open device now if media present */=0A= if (params[0]) {=0A= + char *p =3D NULL;=0A= + if ((p =3D strchr(params, ':'))) {=0A= + p++;=0A= + }=0A= + else {=0A= + p =3D params;=0A= + }=0A= if (bdrv_open(bs_table[hd_index + (is_scsi ? MAX_DISKS : = 0)],=0A= - params, 0 /* snapshot */) < 0)=0A= + p, 0 /* snapshot */) < 0)=0A= fprintf(stderr, "qemu: could not open hard disk image = '%s'\n",=0A= - params);=0A= + p);=0A= }=0A= }=0A= =0A= ------=_NextPart_000_4E44_01C7C006.D9D83330 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel ------=_NextPart_000_4E44_01C7C006.D9D83330--