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>
Tue, 17 Aug 2010 15:01:40 +0000 (16:01 +0100)
commit8f549bd20a193bc61c6e78019c55aeb2efeb7bda
tree3380d0d6132c4cf434c07e18ab69c874a8152ec3
parent17de7c785c921aef542681ee66538508d270fdf3
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.
(cherry picked from commit 799d52be4f08f6c70c0e8ba1aa7367ba4cdd78c4)
src/generator.ml
src/guestfs.c