[dpdk-dev] [PATCH] net/iavf: fix mismatch command

[Yang, Qiming]

> -----Original Message-----
> From: Jiang, JunyuX <junyux.jiang at intel.com>
> Sent: 2020年9月1日 16:15
> To: dev at dpdk.org
> Cc: Xing, Beilei <beilei.xing at intel.com>; Wu, Jingjing <jingjing.wu at intel.com>;
> Yang, Qiming <qiming.yang at intel.com>; Jiang, JunyuX
> <junyux.jiang at intel.com>; stable at dpdk.org
> Subject: [PATCH] net/iavf: fix mismatch command
> 
> The "command mismatch" warning shouldn't be triggered by
> VIRTCHNL_OP_EVENT opcode, because the VIRTCHNL_OP_EVENT opcode is
> used by PF notifies status change events to VF.
> This patch fixed the issue.
> 
> Fixes: 837c2ed86e4c ("net/iavf: return error if opcode is mismatched")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Junyu Jiang <junyux.jiang at intel.com>
> ---
>  drivers/net/iavf/iavf_vchnl.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index
> 33acea54a..331018f14 100644
> --- a/drivers/net/iavf/iavf_vchnl.c
> +++ b/drivers/net/iavf/iavf_vchnl.c
> @@ -53,8 +53,11 @@ iavf_read_msg_from_pf(struct iavf_adapter *adapter,
> uint16_t buf_len,
>  		    opcode, vf->cmd_retval);
> 
>  	if (opcode != vf->pend_cmd) {
> -		PMD_DRV_LOG(WARNING, "command mismatch, expect %u,
> get %u",
> -			    vf->pend_cmd, opcode);
> +		if (opcode != VIRTCHNL_OP_EVENT) {
> +			PMD_DRV_LOG(WARNING,
> +				    "command mismatch, expect %u, get %u",
> +				    vf->pend_cmd, opcode);
> +		}
>  		return IAVF_ERR_OPCODE_MISMATCH;
>  	}
> 
> --
> 2.17.1
Reviewed-by: Qiming Yang <qiming.yang at intel.com>