qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [Bug 1070762] [NEW] savevm fails with inserted CD, "Device '%s' is writable but does not support  snapshots."
@ 2012-10-24  9:58 Oliver Francke
  2016-08-31 18:43 ` [Qemu-devel] [Bug 1070762] " John Snow
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Oliver Francke @ 2012-10-24  9:58 UTC (permalink / raw)
  To: qemu-devel

Public bug reported:

Hi,

yesterday unfortunately a customer reported a failed snapshot of his VM.
Going through the logfile I discovered:

"Device 'ide1-cd0' is writable but does not support snapshots"

this is with qemu-1.2.0 and 1.0.1 at least...

Why writeable?
Even if I specify "-drive ...,readonly=on,snapshot=off" to qemu the monitor-command sees the CD-ROM-device as being writeable?!

Somewhere I saw a "hint" for blockdev.c:
=== snip ===

--- /tmp/blockdev.c	2012-10-24 11:37:10.000000000 +0200
+++ blockdev.c	2012-10-24 11:37:17.000000000 +0200
@@ -551,6 +551,7 @@
     case IF_XEN:
     case IF_NONE:
         dinfo->media_cd = media == MEDIA_CDROM;
+	dinfo->bdrv->read_only = 1;
         break;
     case IF_SD:
     case IF_FLOPPY:

=== snap ===

after installing with this small patch applied it works, so insert CD, savevm <somename> succeeds.
This should be fixed at all correct places, and the tags "readonly=on,snapshot=off" should do it, too. Or even just work after specifying a drive being a CD-rom should do the trick ;-)

Another "bad habit" is, that the ISO/DVD-file has to be writeable to be
changed?

Thnx for attention and regards,

Oliver.

** Affects: qemu
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1070762

Title:
  savevm fails with inserted CD, "Device '%s' is writable but does not
  support  snapshots."

Status in QEMU:
  New

Bug description:
  Hi,

  yesterday unfortunately a customer reported a failed snapshot of his
  VM. Going through the logfile I discovered:

  "Device 'ide1-cd0' is writable but does not support snapshots"

  this is with qemu-1.2.0 and 1.0.1 at least...

  Why writeable?
  Even if I specify "-drive ...,readonly=on,snapshot=off" to qemu the monitor-command sees the CD-ROM-device as being writeable?!

  Somewhere I saw a "hint" for blockdev.c:
  === snip ===

  --- /tmp/blockdev.c	2012-10-24 11:37:10.000000000 +0200
  +++ blockdev.c	2012-10-24 11:37:17.000000000 +0200
  @@ -551,6 +551,7 @@
       case IF_XEN:
       case IF_NONE:
           dinfo->media_cd = media == MEDIA_CDROM;
  +	dinfo->bdrv->read_only = 1;
           break;
       case IF_SD:
       case IF_FLOPPY:

  === snap ===

  after installing with this small patch applied it works, so insert CD, savevm <somename> succeeds.
  This should be fixed at all correct places, and the tags "readonly=on,snapshot=off" should do it, too. Or even just work after specifying a drive being a CD-rom should do the trick ;-)

  Another "bad habit" is, that the ISO/DVD-file has to be writeable to
  be changed?

  Thnx for attention and regards,

  Oliver.

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1070762/+subscriptions

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-09-26  4:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-24  9:58 [Qemu-devel] [Bug 1070762] [NEW] savevm fails with inserted CD, "Device '%s' is writable but does not support snapshots." Oliver Francke
2016-08-31 18:43 ` [Qemu-devel] [Bug 1070762] " John Snow
2020-07-28  1:34 ` John Snow
2020-09-26  4:17 ` Launchpad Bug Tracker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).