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 14:29:47 +0000 (15:29 +0100)
commit9b705c104f2cde7106fdbe27d14bcf4be64df0ad
tree840ef13be54808180d3f1421c34396563d63f2e6
parenta5e110c163c134277b975f2c93c221e52554ec28
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