Xen devel - Aug 2010 - libxl: missing disk device feature and bugs in error path

Hi!

In my guest config file I have this line:

disk = [ ''file:/hvm-guest/win7.img,ioemu:hda,w'' ]

While ''xm create'' parses this correctly,
''xl create'' fails in libxl.c line 1474 where it
calls device_disk_dev_number(). disk->virtpath
contains "ioemu".

device_disk_dev_number() returns -1
and this results in the error message:

libxl: error: libxl.c:1455:libxl_device_disk_add Invalid or unuspported 
virtual disk identifier ioemu

The follow-up error messages are:

libxl: error: libxl.c:851:libxl_domain_destroy pci shutdown failed for domid 1
libxl: error: libxl.c:809:libxl_destroy_device_model Couldn''t find
device
model''s pid: N
o such file or directory
libxl: error: libxl.c:863:libxl_domain_destroy libxl_destroy_device_model 
failed for 1

Looks like things are tried to get destroyed even when they are not 
initialized.

Christoph


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632


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

On Tue, 2010-08-03 at 13:15 +0100, Christoph Egger
wrote:
> Hi!
> 
> In my guest config file I have this line:
> 
> disk = [ ''file:/hvm-guest/win7.img,ioemu:hda,w'' ]
> 
> While ''xm create'' parses this correctly,
> ''xl create'' fails in libxl.c line 1474 where it
> calls device_disk_dev_number(). disk->virtpath
> contains "ioemu".
I have a patch fixing the disk device parsing in the pipeline but it
re-implements what was already in xl and doesn''t handle this case. I am
not sure how this flag ought to be handled so if Stefano or Ian have
suggestions I can fold that in to my patch.
> device_disk_dev_number() returns -1
> and this results in the error message:
> 
> libxl: error: libxl.c:1455:libxl_device_disk_add Invalid or unuspported 
> virtual disk identifier ioemu
> 
> The follow-up error messages are:
> 
> libxl: error: libxl.c:851:libxl_domain_destroy pci shutdown failed for
domid 1
> libxl: error: libxl.c:809:libxl_destroy_device_model Couldn''t find
device
> model''s pid: N
> o such file or directory
> libxl: error: libxl.c:863:libxl_domain_destroy libxl_destroy_device_model 
> failed for 1
> 
> Looks like things are tried to get destroyed even when they are not 
> initialized.
PCI one is known, introduced recently by me and fixed in my last
patchset.


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

On Tue, 3 Aug 2010, Gianni Tedesco (3P) wrote:
> On Tue, 2010-08-03 at 13:15 +0100, Christoph Egger wrote:
> > Hi!
> > 
> > In my guest config file I have this line:
> > 
> > disk = [ ''file:/hvm-guest/win7.img,ioemu:hda,w'' ]
> > 
> > While ''xm create'' parses this correctly,
> > ''xl create'' fails in libxl.c line 1474 where it
> > calls device_disk_dev_number(). disk->virtpath
> > contains "ioemu".
> 
> I have a patch fixing the disk device parsing in the pipeline but it
> re-implements what was already in xl and doesn''t handle this case.
I am
> not sure how this flag ought to be handled so if Stefano or Ian have
> suggestions I can fold that in to my patch.
ioemu should be ignored: by default we provide an emulated disk if the
device name is hd* and we don''t if the disk name is xvd*



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