Re: [PATCH] net/iavf: revert fix VLAN insertion

[Kevin Traynor <ktraynor@redhat.com>]

On 18/10/2022 11:26, Yiding Zhou wrote:
> The patch to be reverted forces to select normal Tx path when kernel driver
> tells that L2TAG2 is required, it results in a lot of performance loss.
> 
> We should support Tx context descriptor on vector path to handle the L2TAG2
> case.
> 

Hi. Was the original fix in 0d58caa7d6d1 not needed? or is it just being 
reverted because of performance issues and the original functional issue 
still remains? It is not very clear in the commit message. It would be 
good to write what the status of the original issue with this patch 
applied is in the commit message.

> This commit reverts
> commit 0d58caa7d6d1 ("net/iavf: fix VLAN insertion")
> 

Better to add DPDK style tags here, so they can be picked up by scripts, 
thanks. i.e.

Fixes: 0d58caa7d6d1 ("net/iavf: fix VLAN insertion")

> Signed-off-by: Yiding Zhou <yidingx.zhou@intel.com>
> ---
>   drivers/net/iavf/iavf_rxtx_vec_common.h | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/net/iavf/iavf_rxtx_vec_common.h b/drivers/net/iavf/iavf_rxtx_vec_common.h
> index 4ab22c6b2b..a59cb2ceee 100644
> --- a/drivers/net/iavf/iavf_rxtx_vec_common.h
> +++ b/drivers/net/iavf/iavf_rxtx_vec_common.h
> @@ -253,9 +253,6 @@ iavf_tx_vec_queue_default(struct iavf_tx_queue *txq)
>   	if (txq->offloads & IAVF_TX_NO_VECTOR_FLAGS)
>   		return -1;
>   
> -	if (txq->vlan_flag == IAVF_TX_FLAGS_VLAN_TAG_LOC_L2TAG2)
> -		return -1;
> -
>   	if (txq->offloads & IAVF_TX_VECTOR_OFFLOAD)
>   		return IAVF_VECTOR_OFFLOAD_PATH;
>