[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer |
Date: |
Thu, 15 Jul 2010 19:10:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-2.fc12 Thunderbird/3.0.4 |
Am 15.07.2010 18:20, schrieb Anthony Liguori:
> I have another idea that I hope will solve the problem in a more
> complete way. The fundamental issue is that it's impossible to probe
> raw images reliably. We can probe qcow2, vmdk, etc but not raw.
>
> So, let's do the following: have raw_probe() always fail. Probing
> shouldn't be a heuristic, it should be an absolute. We can't prove it's
> a raw image, so we should always fail.
>
> To accomodate current use-cases with raw, let's introduce a new format
> called "probed_raw". probed_raw's semantics will be the following:
>
> The signature of a probed_raw will be ~{'QFI\xfb', 'VMDK', 'COWD',
> 'OOOM', ...}. If the signature is 'QRAW', then instead of reading the
> first sector at offset 0, we read the first sector at offset LENGTH. If
> the signature is 'QRAW', LENGTH is computed by calculating FILE_SIZE - 512.
>
> For probed_raw, write requests to sector 0 are checked. If the first
> four bytes is an invalid probed_raw signature or QRAW, we write a QRAW
> signature to file offset 0 and copy the first sector to the end of the
> file redirecting reads and writes to the end of file.
>
> An approach like this has the following properties:
>
> 1) We can make the bdrv_probe check 100% reliable and return a boolean.
> 2) In the cases where we known format=raw, none of this code is ever
> invoked.
> 3) probed_raw images usually look exactly like raw images in most cases
> 4) In the degenerate cases, probe_raw images are still mountable in the
> normal way.
> 5) Even after the QRAW signature is applied, if the guest writes a valid
> signature, we can truncate the file and make it appear as a normal raw
> image.
>
> Christoph/Markus/Stefan, does this seem like a more reasonable approach?
I hope I may answer even though you didn't ask me. ;-)
This qraw format you introduce is a weird thing, mainly because
depending on a signature the first sectors moves to somewhere completely
else.
If we want to introduce something, that's _almost_ raw and that we can
use by default, what about doing something similar to what statically
sized VHD images look like: take a raw image and attach a footer? With
this approach you should still be able to mount it in the host etc. We
could automatically append this footer when opening a probed qraw image r/w.
Kevin
- [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/14
- [Qemu-devel] Re: [PATCH] Make default invocation of block drivers safer, Kevin Wolf, 2010/07/14
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Christoph Hellwig, 2010/07/14
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/14
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Markus Armbruster, 2010/07/15
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/15
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/15
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Kevin Wolf, 2010/07/16
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Stefan Hajnoczi, 2010/07/16
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Stefan Hajnoczi, 2010/07/16
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Markus Armbruster, 2010/07/16
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/16
- Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Kevin Wolf, 2010/07/16
Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Anthony Liguori, 2010/07/14
Re: [Qemu-devel] [PATCH] Make default invocation of block drivers safer, Aurelien Jarno, 2010/07/14