On 02/06/2016 05:28 PM, zhanghailiang wrote:
> Enable all buffer filters that added by COLO while
> go into COLO process, and disable them while exit COLO.
>
> Signed-off-by: zhanghailiang <zhang.zhanghaili...@huawei.com>
> Cc: Jason Wang <jasow...@redhat.com>
> Cc: Yang Hongyang <hongyang.y...@easystack.cn>
> ---
> v14:
> - New patch
> ---
> migration/colo.c | 32 ++++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
>
> diff --git a/migration/colo.c b/migration/colo.c
> index 235578b..86a7638 100644
> --- a/migration/colo.c
> +++ b/migration/colo.c
> @@ -104,10 +104,26 @@ static void secondary_vm_do_failover(void)
> }
> }
>
> +static void colo_set_filter_status(NetFilterState *nf, void *opaque,
> + Error **errp)
> +{
> + char colo_filter[128];
> + char *name = object_get_canonical_path_component(OBJECT(nf));
> + char *status = opaque;
> +
> + snprintf(colo_filter, sizeof(colo_filter), "%scolo", nf->netdev_id);
> + if (strcmp(colo_filter, name)) {
> + return;
> + }
Checking by name is not elegant. As we've discussed last time, why not
let filter-buffer track all filters with zero interval in a linked list
and just export a helper to disable and enable them all? Things will be
greatly simplified with this, and there's even no need for patch 36.
