From mboxrd@z Thu Jan  1 00:00:00 1970
From: Maxime Coquelin <maxime.coquelin@redhat.com>
Subject: Re: [PATCH v4 07/10] net/ifc: add devarg for LM mode
Date: Sun, 16 Dec 2018 10:21:00 +0100
Message-ID: <bd68f5c6-342e-841f-1351-4c79e6ef50d4@redhat.com>
References: <20181213100910.13087-2-xiao.w.wang@intel.com>
 <20181214211612.167681-1-xiao.w.wang@intel.com>
 <20181214211612.167681-8-xiao.w.wang@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Cc: alejandro.lucero@netronome.com, dev@dpdk.org, zhihong.wang@intel.com,
 xiaolong.ye@intel.com
To: Xiao Wang <xiao.w.wang@intel.com>, tiwei.bie@intel.com
Return-path: <dev-bounces@dpdk.org>
Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28])
 by dpdk.org (Postfix) with ESMTP id C005A1B9BD
 for <dev@dpdk.org>; Sun, 16 Dec 2018 10:21:05 +0100 (CET)
In-Reply-To: <20181214211612.167681-8-xiao.w.wang@intel.com>
Content-Language: en-US
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>



On 12/14/18 10:16 PM, Xiao Wang wrote:
> This patch series enables a new method for live migration, i.e. software
> assisted live migration. This patch provides a device argument for user
> to choose the methold.
> 
> When "swlm=1", driver/device will do live migration with a relay thread
> dealing with dirty page logging. Without this parameter, device will do
> dirty page logging and there's no relay thread consuming CPU resource.
> 
> Signed-off-by: Xiao Wang <xiao.w.wang@intel.com>
> ---
>   drivers/net/ifc/ifcvf_vdpa.c | 13 +++++++++++++
>   1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/net/ifc/ifcvf_vdpa.c b/drivers/net/ifc/ifcvf_vdpa.c
> index c0e50354a..395c5112f 100644
> --- a/drivers/net/ifc/ifcvf_vdpa.c
> +++ b/drivers/net/ifc/ifcvf_vdpa.c
> @@ -8,6 +8,7 @@
>   #include <sys/ioctl.h>
>   #include <sys/epoll.h>
>   #include <linux/virtio_net.h>
> +#include <stdbool.h>
>   
>   #include <rte_malloc.h>
>   #include <rte_memory.h>
> @@ -31,9 +32,11 @@
>   #endif
>   
>   #define IFCVF_VDPA_MODE		"vdpa"
> +#define IFCVF_SW_FALLBACK_LM	"swlm"


The patch looks good, except that I don't like the "swlm" name.
Maybe we could have something less obscure, even if a little bt longer?

What about "sw-live-migration"?