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

["Daniel P. Berrange" <berrange@xxxxxxxxxx>]

On Fri, Jul 06, 2007 at 07:50:06PM +0900, Kasai Takanori wrote:
> 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.

That's not safe enough since it always looks for a ':' regardless of
whether the driver is 'tap' or not. It also doesn't take account of
fact that the device may be named 'xvda' rather than 'hda'. 

I posted patches to address this more completely here:

http://lists.xensource.com/archives/html/xen-devel/2007-06/msg01021.html

I'm attaching them again for convenience

Regards,
Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=|

Attachment: xen-revert-phantom.patch
Description: Text document

Attachment: qemu-blktap.patch
Description: Text document

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