After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 679646 - Failed to boot from CD-ROM
Failed to boot from CD-ROM
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: general
3.4.x (unsupported)
Other Linux
: Normal normal
: --
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-07-09 18:31 UTC by Balló György
Modified: 2016-03-31 14:02 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Generated libvirt xml config file (6.53 KB, text/xml)
2012-07-09 18:31 UTC, Balló György 		  Details
Qemu log file (6.46 KB, application/octet-stream)
2012-07-09 18:34 UTC, Balló György 		  Details
Mark CDROM device as block-based/file-based early (1.53 KB, patch)
2012-07-13 10:01 UTC, Christophe Fergeau 		committed Details | Review

Description Balló György 2012-07-09 18:31:42 UTC 
Created attachment 218358 [details]
Generated libvirt xml config file

When I try to run a Live CD, qemu can't boot from it:
"Boot failed: could not read from CDROM (code 0003)"

It works if I boot from an iso file, but not works if I boot from a phisical CD.

It seems that the following line is missing from the generated libvirt config file (see the attached config):
<source dev='/dev/sr0'/>

System: Arch Linux

Package versions:
spice 0.10.1
qemu-kvm 1.1.0
spice-gtk 0.12
libosinfo 0.1.2
libvirt 0.9.13
libvirt-glib 0.0.9
gnome-boxes 3.4.3
Comment 1 Balló György 2012-07-09 18:34:59 UTC 
Created attachment 218359 [details]
Qemu log file
Comment 2 Zeeshan Ali 2012-07-13 02:25:32 UTC 
I am able to reproduce this on 3.5.x as well.
Comment 3 Christophe Fergeau 2012-07-13 10:01:39 UTC 
Created attachment 218696 [details] [review]
Mark CDROM device as block-based/file-based early

The type of the libvirt disk config object is needed to generate
proper XML for the 'source' element, so we need to set it as early
as possible.
Comment 4 Christophe Fergeau 2012-07-13 10:02:42 UTC 
We need at least this change. If this is not enough, knowing what 'device_file' is in this function would be useful in debugging this further.
Comment 5 Zeeshan Ali 2012-07-13 13:01:26 UTC 
Review of attachment 218696 [details] [review]:

Yeah, it fixes the issue here. I think we should add a warning/g_return_if_fail to libvirt-gconfig since I imagine other apps easily falling for the same.
Comment 6 Christophe Fergeau 2012-07-13 13:26:36 UTC 
Attachment 218696 [details] pushed as b84e779 - Mark CDROM device as block-based/file-based early