On Thu 10 Mar 2016 03:49:39 AM CET, Changlong Xie wrote: > From: Wen Congyang <we...@cn.fujitsu.com> > > In some cases, we want to take a quorum child offline, and take > another child online. > > Signed-off-by: Wen Congyang <we...@cn.fujitsu.com> > Signed-off-by: zhanghailiang <zhang.zhanghaili...@huawei.com> > Signed-off-by: Gonglei <arei.gong...@huawei.com> > Signed-off-by: Changlong Xie <xiecl.f...@cn.fujitsu.com> > Reviewed-by: Max Reitz <mre...@redhat.com>
> + if (!tmp) { > + error_setg(errp, "The node %s does not have child named %s", > + bdrv_get_device_or_node_name(parent_bs), > + bdrv_get_device_or_node_name(child->bs)); > + return; I think it should be "does not have a child" or "does not have any child". With that corrected, Reviewed-by: Alberto Garcia <be...@igalia.com> Berto