[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks |
Date: |
Fri, 13 Sep 2013 08:34:28 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8 |
On 09/13/2013 08:29 AM, Max Reitz wrote:
> Another idea: Instead of providing an integer for "shorthand"
> manipulations, what do you think of a string parameter (such as -o
> cache=foo right now, although I do know -o cache isn't even document
> anymore and provided only for compatibility reasons, it seems to me)
> which will automatically be translated to the right settings? I'm
> thinking of:
> - overlap-check=none (no checks at all)
> - overlap-check=constant (only checks who can be performed in constant
> time, i.e., main header, active L1, refcount table and snapshot table)
> - overlap-check=cached (only checks which don't require disk access,
> i.e. the current (and as I'd propose, future) default)
> - overlap-check=all (all checks, including those requiring disk access
> (i.e., overlaps on inactive L2 tables))
Definitely a nice idea - saves the user from having to figure out which
checks have which impact, by categorizing the types of checks and only
exposing the categories.
>
> These would then provide templates which can be further refined through
> the booleans (as is the case with -o cache right now).
Probably don't need that much fine-tuning; your categories look pretty
nice (although we may think of more categories if we do add additional
checks with different levels of execution time).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v5 8/8] qemu-iotests: Overlapping cluster allocations, (continued)
- [Qemu-devel] [PATCH v5 8/8] qemu-iotests: Overlapping cluster allocations, Max Reitz, 2013/09/02
- [Qemu-devel] [PATCH v5 7/8] qcow2_check: Mark image consistent, Max Reitz, 2013/09/02
- [Qemu-devel] [PATCH v5 6/8] qcow2-refcount: Repair shared refcount blocks, Max Reitz, 2013/09/02
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Eric Blake, 2013/09/12
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Max Reitz, 2013/09/12
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Kevin Wolf, 2013/09/13
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Max Reitz, 2013/09/13
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Eric Blake, 2013/09/13
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Max Reitz, 2013/09/13
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Max Reitz, 2013/09/13
- Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks,
Eric Blake <=
Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks, Max Reitz, 2013/09/19