Re: [Qemu-devel] [PATCH v3] more check for replaced node

[Stefan Hajnoczi <stefanha@redhat.com>]

On Fri, Jul 17, 2015 at 10:12:22AM +0800, Wen Congyang wrote:
> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
> is quorum, and to_replace is the broken child. The new child is target_bs.
> Without this patch, the replace node can be any node, and it can be
> top BDS with BB, or another quorum's child. We just check if the broken
> child is part of the quorum BDS in this patch.
> 
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> ---
>  block.c               | 5 +++--
>  block/mirror.c        | 3 ++-
>  blockdev.c            | 2 +-
>  include/block/block.h | 3 ++-
>  4 files changed, 8 insertions(+), 5 deletions(-)

There has been no further discussion.  I think the patch is a good idea,
so I'm applying it.

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan