Re: [dpdk-dev] incorrect vlan_tci in rte mbuf

[Olivier Matz <olivier.matz@6wind.com>]

Hi,

On Fri, May 15, 2020 at 11:12:27AM +0000, Yan, Xiaoping (NSB - CN/Hangzhou) wrote:
> Hi,
> 
> I'm using i40e vf, dpdk 18.11, x86_64 CPU (rx function in use is i40e_recv_scattered_pkts_vec_avx2)
> When enable hw vlan strip:
> 
>   *   If packet fit in one mbuf segment, the vlan_tci field is correct
>   *   If packets are stored in several mbuf segment, the vlan_tci of last segment is correct, vlan_tci of other segments are invalid
> 
> It seems i40e_recv_scattered_pkts has correctly set the vlan_tci, by calling
> i40e_rxd_to_vlan_tci(first_seg, &rxd);
> 
> Is this a bug in i40e_recv_scattered_pkts_vec_avx2?
> (I didn't find setting vlan_tci for first segment, but it's a bit difficult for me to understand codes in i40e_recv_scattered_pkts_vec_avx2, so I'm not quite sure)
> I checked the latest dpdk version 20.02 http://lxr.dpdk.org/dpdk/latest/source/drivers/net/i40e/i40e_rxtx_vec_avx2.c#L791
> But seems no change for this.
> 
> Any comment please?
> [...]

I don't know about the i40e driver, but I can confirm that the vlan tci
flag should be in the first segment, and not in the next ones.

I suggest to CC i40e driver maintainers.

Regards,
Olivier