Re: [PATCH] kvm tools: Use mmap for working with disk image V2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 11, 2011 at 9:41 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> How do you plan to handle I/O errors or ENOSPC conditions?  Note that
> shared writeable mappings are by far the feature in the VM/FS code
> that is most error prone, including the impossiblity of doing sensible
> error handling.

Good point. I reverted the commit. Thanks!

On Mon, Apr 11, 2011 at 9:41 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> The version that accidentally used MAP_PRIVATE actually makes a lot of
> sense for an equivalent of qemu's snapshot mode where the image is
> readonly and changes are kept private as long as the amount of modified
> blocks is small enough to not kill the host VM, but using shared
> writeable mappings just sems dangerous.

Yup, Sasha, mind submitting a MAP_PRIVATE version that's enabled with
'--snapshot' (or equivalent) command line option.

                        Pekka
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux