[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 03/10] block: Remove child references from bs
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v2 03/10] block: Remove child references from bs->{options, explicit_options} |
Date: |
Wed, 5 Sep 2018 14:37:12 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-09-03 16:34, Alberto Garcia wrote:
> Block drivers allow opening their children using a reference to an
> existing BlockDriverState. These references remain stored in the
> 'options' and 'explicit_options' QDicts, but we don't need to keep
> them once everything is open.
>
> What is more important, these values can become wrong if the children
> change:
>
> $ qemu-img create -f qcow2 hd0.qcow2 10M
> $ qemu-img create -f qcow2 hd1.qcow2 10M
> $ qemu-img create -f qcow2 hd2.qcow2 10M
> $ $QEMU -drive if=none,file=hd0.qcow2,node-name=hd0 \
> -drive if=none,file=hd1.qcow2,node-name=hd1,backing=hd0 \
> -drive file=hd2.qcow2,node-name=hd2,backing=hd1
>
> After this hd2 has hd1 as its backing file. Now let's remove it using
> block_stream:
>
> (qemu) block_stream hd2 0 hd0.qcow2
>
> Now hd0 is the backing file of hd2, but hd2's options QDicts still
> contain backing=hd1.
>
> Signed-off-by: Alberto Garcia <address@hidden>
> ---
> block.c | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v2 04/10] block: Don't look for child references in append_open_options(), (continued)
- [Qemu-block] [PATCH v2 04/10] block: Don't look for child references in append_open_options(), Alberto Garcia, 2018/09/03
- [Qemu-block] [PATCH v2 02/10] file-posix: x-check-cache-dropped should default to false on reopen, Alberto Garcia, 2018/09/03
- [Qemu-block] [PATCH v2 08/10] block: Allow changing 'discard' on reopen, Alberto Garcia, 2018/09/03
- [Qemu-block] [PATCH v2 05/10] block: Allow child references on reopen, Alberto Garcia, 2018/09/03
- [Qemu-block] [PATCH v2 10/10] block: Allow changing 'force-share' on reopen, Alberto Garcia, 2018/09/03
- [Qemu-block] [PATCH v2 03/10] block: Remove child references from bs->{options, explicit_options}, Alberto Garcia, 2018/09/03
- Re: [Qemu-block] [PATCH v2 03/10] block: Remove child references from bs->{options, explicit_options},
Max Reitz <=