Re: [dpdk-dev] i40e_rxtx_vec_avx2 code review

2020-08-30 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jeff Guo > Sent: Monday, August 31, 2020 8:30 AM > > hi, mb > > Seems that there are a referring issue here which you have found, i > think we need to fix it, if there are on objection i will add that fix > into the prior burst fixing patch,

Re: [dpdk-dev] i40e_rxtx_vec_avx2 code review

2020-08-30 Thread Jeff Guo
hi, mb Seems that there are a referring issue here which you have found, i think we need to fix it, if there are on objection i will add that fix into the prior burst fixing patch, thanks very much. On 8/29/2020 3:21 PM, Morten Brørup wrote: Jeff, Beilei, Looking closer at i40e_rxtx_vec_avx

[dpdk-dev] i40e_rxtx_vec_avx2 code review

2020-08-29 Thread Morten Brørup
Jeff, Beilei, Looking closer at i40e_rxtx_vec_avx2.c, I noticed: 1. i40e_rxq_rearm() uses RTE_I40E_DESCS_PER_LOOP, while all other functions in the file use RTE_I40E_DESCS_PER_LOOP_AVX. It may be correct, but please check. 2. The function descriptions of i40e_recv_pkts_vec_avx2(), i40e_recv_scat