From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ye Xiaolong <xiaolong.ye@intel.com>
Subject: Re: [PATCH] net/ifc: do not notify before HW ready
Date: Thu, 13 Sep 2018 20:55:19 +0800
Message-ID: <20180913125519.GA13400@intel.com>
References: <20180910110531.138449-1-xiao.w.wang@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: tiwei.bie@intel.com, dev@dpdk.org
To: Xiao Wang <xiao.w.wang@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mga06.intel.com (mga06.intel.com [134.134.136.31])
 by dpdk.org (Postfix) with ESMTP id 919CF4C91
 for <dev@dpdk.org>; Thu, 13 Sep 2018 08:07:30 +0200 (CEST)
Content-Disposition: inline
In-Reply-To: <20180910110531.138449-1-xiao.w.wang@intel.com>
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>

Hi, Xiao

On 09/10, Xiao Wang wrote:
>Fixes: a3f8150eac6d ("net/ifcvf: add ifcvf vDPA driver")

Could you help describe what problem is without this fix in commit log?

Thanks,
Xiaolong
>
>Signed-off-by: Xiao Wang <xiao.w.wang@intel.com>
>---
> drivers/net/ifc/ifcvf_vdpa.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/net/ifc/ifcvf_vdpa.c b/drivers/net/ifc/ifcvf_vdpa.c
>index 3c5430dc0..7d3085d8d 100644
>--- a/drivers/net/ifc/ifcvf_vdpa.c
>+++ b/drivers/net/ifc/ifcvf_vdpa.c
>@@ -503,11 +503,11 @@ update_datapath(struct ifcvf_internal *internal)
> 		if (ret)
> 			goto err;
> 
>-		ret = setup_notify_relay(internal);
>+		ret = vdpa_ifcvf_start(internal);
> 		if (ret)
> 			goto err;
> 
>-		ret = vdpa_ifcvf_start(internal);
>+		ret = setup_notify_relay(internal);
> 		if (ret)
> 			goto err;
> 
>@@ -515,12 +515,12 @@ update_datapath(struct ifcvf_internal *internal)
> 	} else if (rte_atomic32_read(&internal->running) &&
> 		   (!rte_atomic32_read(&internal->started) ||
> 		    !rte_atomic32_read(&internal->dev_attached))) {
>-		vdpa_ifcvf_stop(internal);
>-
> 		ret = unset_notify_relay(internal);
> 		if (ret)
> 			goto err;
> 
>+		vdpa_ifcvf_stop(internal);
>+
> 		ret = vdpa_disable_vfio_intr(internal);
> 		if (ret)
> 			goto err;
>-- 
>2.15.1
>