[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/5] block/mirror: Fix (?) target backing BDS
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/5] block/mirror: Fix (?) target backing BDS |
Date: |
Mon, 13 Jun 2016 17:16:04 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 10.06.2016 um 20:57 hat Max Reitz geschrieben:
> Issue #1: If the target image does not have a backing BDS before mirror
> completion, qemu tries really hard to give it a backing BDS. If the
> source has a backing BDS, it will actually always "succeed".
> In some cases, the target is not supposed to have a backing BDS, though
> (absolute-paths: because of sync=full; existing: because the target
> image does not have a backing file; blockdev-mirror: because of an
> explicit "backing": ""). Then, this is pretty bad behavior.
>
> This should generally not change the target's visible data, but it still
> is ugly.
>
> Issue #2: Currently the backing chain of the target is basically opened
> using bdrv_open_backing_file() (except for sometimes™). This results in
> multiple BDSs for a single physical file, which is bad. In most use
> cases, this is only temporary, but it still is bad.
>
> If we can reuse the existing backing chain of the source (which is with
> drive-mirror in "absolute-paths" mode), we should just do so.
Reviewed-by: Kevin Wolf <address@hidden>
I'll still wait to apply the series so that you have a chance to answer
Fam's question before it is in.
Kevin
- Re: [Qemu-devel] [PATCH v3 1/5] block: Allow replacement of a BDS by its overlay, (continued)
- [Qemu-devel] [PATCH v3 4/5] iotests: Add test for post-mirror backing chains, Max Reitz, 2016/06/10
- [Qemu-devel] [PATCH v3 3/5] block/null: Implement bdrv_refresh_filename(), Max Reitz, 2016/06/10
- [Qemu-devel] [PATCH v3 5/5] iotests: Add test for oVirt-like storage migration, Max Reitz, 2016/06/10
- [Qemu-devel] [PATCH v3 2/5] block/mirror: Fix target backing BDS, Max Reitz, 2016/06/10
- Re: [Qemu-devel] [PATCH v3 0/5] block/mirror: Fix (?) target backing BDS, Fam Zheng, 2016/06/12
- Re: [Qemu-devel] [PATCH v3 0/5] block/mirror: Fix (?) target backing BDS,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH v3 0/5] block/mirror: Fix (?) target backing BDS, Max Reitz, 2016/06/14