Revert "add_drive_ro adds readonly=on option if available." (RHBZ#617200).
authorRichard Jones <rjones@redhat.com>
Thu, 22 Jul 2010 13:39:36 +0000 (14:39 +0100)
committerRichard Jones <rjones@redhat.com>
Thu, 22 Jul 2010 15:51:55 +0000 (16:51 +0100)
commit799d52be4f08f6c70c0e8ba1aa7367ba4cdd78c4
tree0efef0a96ea2e23b5598296d845e6f2d7253774d
parent2a286f16215ebfac88a32d259f2b68191eb8d27e
Revert "add_drive_ro adds readonly=on option if available." (RHBZ#617200).

Adding the readonly=on option is not so clever.  This causes
qemu to present the disk as read-only to the guest.  (The
expected behaviour of snapshots=on,readonly=on was that it
would open the disk O_RDONLY but present a writable disk to
the guest).

Since the guest sees a read-only disk, we are unable to do any
recovery if a filesystem on the disk is inconsistent.  This basically
prevents most accesses to live disk images.

What we really want is a qemu option which presents a writable
disk to the guest, but only opens the disk on the host side with
O_RDONLY, to alleviate the udev bug RHBZ#571714.

This reverts commit 676462684e05dd8341dd695762dd99a87d8ec022.
src/generator.ml
src/guestfs.c