Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-07-01 Thread Martin Kletzander
On Tue, Jul 01, 2014 at 09:40:00AM +0200, Giuseppe Scrivano wrote: Martin Kletzander writes: If the user updates from QEMU without DRIVE_READONLY to newer one, that supports that flag, than XML with readonly IDE HDD will stop working even though it worked before the update *as requested*. Tha

Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-07-01 Thread Giuseppe Scrivano
Martin Kletzander writes: > If the user updates from QEMU without DRIVE_READONLY to newer one, > that supports that flag, than XML with readonly IDE HDD will stop > working even though it worked before the update *as requested*. That > readonly flag does not reflect how the disk is exposed in th

Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-06-30 Thread Martin Kletzander
On Mon, Jun 30, 2014 at 02:28:54PM +0100, Daniel P. Berrange wrote: On Mon, Jun 30, 2014 at 03:23:40PM +0200, Giuseppe Scrivano wrote: Martin Kletzander writes: Q> On Mon, Jun 30, 2014 at 12:05:06PM +0200, Giuseppe Scrivano wrote: >>The IDE bus doesn't support readonly disks, so inform the use

Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-06-30 Thread Giuseppe Scrivano
Martin Kletzander writes: Q> On Mon, Jun 30, 2014 at 12:05:06PM +0200, Giuseppe Scrivano wrote: >>The IDE bus doesn't support readonly disks, so inform the user with an >>error message instead of let qemu fail with a more obscure "Device >>'ide-hd' could not be initialized" error message. >> >>Cl

Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-06-30 Thread Daniel P. Berrange
On Mon, Jun 30, 2014 at 03:23:40PM +0200, Giuseppe Scrivano wrote: > Martin Kletzander writes: > > Q> On Mon, Jun 30, 2014 at 12:05:06PM +0200, Giuseppe Scrivano wrote: > >>The IDE bus doesn't support readonly disks, so inform the user with an > >>error message instead of let qemu fail with a mor

Re: [libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-06-30 Thread Martin Kletzander
On Mon, Jun 30, 2014 at 12:05:06PM +0200, Giuseppe Scrivano wrote: The IDE bus doesn't support readonly disks, so inform the user with an error message instead of let qemu fail with a more obscure "Device 'ide-hd' could not be initialized" error message. Closes: https://bugzilla.redhat.com/show_

[libvirt] [PATCH] qemu: raise an error when trying to use readonly ide disks

2014-06-30 Thread Giuseppe Scrivano
The IDE bus doesn't support readonly disks, so inform the user with an error message instead of let qemu fail with a more obscure "Device 'ide-hd' could not be initialized" error message. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1112939 Signed-off-by: Giuseppe Scrivano --- src/qemu/q