[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster siz
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity |
Date: |
Mon, 21 Jan 2013 15:10:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 18/01/2013 17:22, Paolo Bonzini ha scritto:
>> > If it does, we mark the whole cluster dirty now, but in the cow_bitmap
>> > it's still marked at present on the target. When restarting the job,
>> > wouldn't it copy only the start of the cluster next time and corrupt the
>> > rest of it?
> Yes, very good catch.
Actually, it works. Because the whole destination cluster is marked
dirty, all of it is ultimately copied correctly. The special handling
of COW is required for all others sources of dirty data, where the dirty
bitmap can include a subset of a destination cluster, but not in this case.
I'll include the testcase, and test the patch I attached to my previous
message.
Paolo
- Re: [Qemu-devel] [PATCH v2 02/12] add hierarchical bitmap data type and test cases, (continued)
- [Qemu-devel] [PATCH v2 04/12] block: make round_to_clusters public, Paolo Bonzini, 2013/01/16
- [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Paolo Bonzini, 2013/01/16
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Kevin Wolf, 2013/01/18
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Paolo Bonzini, 2013/01/18
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Kevin Wolf, 2013/01/18
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Paolo Bonzini, 2013/01/18
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Kevin Wolf, 2013/01/21
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Paolo Bonzini, 2013/01/21
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity, Stefan Hajnoczi, 2013/01/22
- Re: [Qemu-devel] [PATCH v2 05/12] mirror: perform COW if the cluster size is bigger than the granularity,
Paolo Bonzini <=
[Qemu-devel] [PATCH v2 07/12] block: allow customizing the granularity of the dirty bitmap, Paolo Bonzini, 2013/01/16
[Qemu-devel] [PATCH v2 08/12] mirror: allow customizing the granularity, Paolo Bonzini, 2013/01/16
[Qemu-devel] [PATCH v2 09/12] mirror: switch mirror_iteration to AIO, Paolo Bonzini, 2013/01/16
[Qemu-devel] [PATCH v2 10/12] mirror: add buf-size argument to drive-mirror, Paolo Bonzini, 2013/01/16