[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [RFC] libxl: relax readonly check introduced by XSA-142 fix
Ian Campbell wrote: > On Thu, 2015-11-12 at 08:45 -0700, Jim Fehlig wrote: >> >>> The commit message doesn't say anything about AHCI. Are AHCI disks >>> actually emulated correctly by QEMU with readonly=on? >> I just double checked, and good thing since AHCI + readonly is another >> rejected >> combination >> >> /usr/lib/xen/bin/qemu-system-i386 -device ahci,id=ahci0 \ >> -drive file=/tmp/disk.raw,if=none,id=ahcidisk-0,format=raw,readonly=on \ >> -device ide-hd,bus=ahci0.0,unit=0,drive=ahcidisk-0 >> qemu-system-i386: -device ide-hd,bus=ahci0.0,unit=0,drive=ahcidisk-0: >> Can't use >> a read-only drive >> >> So IDE/SATA/AHCI are all incompatible with readonly=on. I'll fix this and >> ammend >> the commit message in V2. > > Just to clarify when you say "rejected" and "incompatible" do you mean that > qemu will fail to start if you try, or that it will ignore you and give a > writeable disk? qemu will fail to start. > > If, as I think, it will fail, why don't we just always ask and rely on qemu > to reject, instead of trying to whitelist the ones we know work in the > libxl code? That would be possible, but makes it more difficult to track down why the domain failed to start. With the check in libxl: # xl create sles12-hvm.xl Parsing config from sles12-hvm.xl libxl: error: libxl_dm.c:1201:libxl__build_device_model_args_new: qemu-xen doesn't support read-only IDE disk drivers libxl: error: libxl_dm.c:1891:device_model_spawn_outcome: (null): spawn failed (rc=-6) libxl: error: libxl_create.c:1340:domcreate_devmodel_started: device model did not start: -6 Allowing qemu to fail: # xl create sles12-hvm.xl Parsing config from sles12-hvm.xl libxl: error: libxl_dm.c:1887:device_model_spawn_outcome: domain 14 device model: spawn failed (rc=-3) libxl: error: libxl_create.c:1340:domcreate_devmodel_started: device model did not start: -3 libxl: error: libxl_dm.c:1997:kill_device_model: Device Model already exited Ok, not so obvious why qemu failed to start. One would need to peek at /var/log/xen/qemu-dm-sles12-hvm.log: char device redirected to /dev/pts/3 (label serial0) qemu-system-i386: Can't use a read-only drive qemu-system-i386: Device initialization failed. Regards, Jim _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |