[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH 44/54] mirror: Add filter-node-name to blockdev-

From: Max Reitz
Subject: Re: [Qemu-block] [PATCH 44/54] mirror: Add filter-node-name to blockdev-mirror
Date: Sat, 25 Feb 2017 15:19:42 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0

On 21.02.2017 15:58, Kevin Wolf wrote:
> Management tools need to be able to know about every node in the graph
> and need a way to address them. This new option to blockdev-mirror
> allows the client to set a node-name for the automatically inserted
> filter driver, and at the same time serves as a witness that this
> version of qemu does automatically insert a filter driver.
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  block/mirror.c            | 14 ++++++++------
>  blockdev.c                | 12 +++++++++++-
>  include/block/block_int.h |  5 ++++-
>  qapi/block-core.json      |  8 +++++++-
>  4 files changed, 30 insertions(+), 9 deletions(-)

Reviewed-by: Max Reitz <address@hidden>

I'm not so happy about "...serves as a witness..." because that doesn't
make it less of an incompatible change.

However, since (as you know) I had plans to introduce such a filter node
myself anyway, I'll gladly let you take all the git-blame if something

(And it should be easy for me to add the mirror target as another child
to the filter node.)


Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]