Re: [dpdk-dev] [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e

["Zhang, Qi Z" <qi.z.zhang@intel.com>]

> -----Original Message-----
> From: Richardson, Bruce
> Sent: Tuesday, January 9, 2018 10:33 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Cc: dev@dpdk.org; Zhang, Helin <helin.zhang@intel.com>; Yigit, Ferruh
> <ferruh.yigit@intel.com>; Richardson, Bruce <bruce.richardson@intel.com>
> Subject: [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e
> 
> This patch adds an AVX2 vectorized path to the i40e driver, based on the
> existing SSE4.2 version. Using AVX2 instructions gives better performance than
> the SSE version, though the percentage increase depends on the exact settings
> used. For example:
> 
> * Using 16B rather than 32B descriptors gives the biggest benefit since
>   2 descriptors at a time can be read, rather than just 1 when 32B ones
>   are used.
> * Bigger burst sizes for RX gives improved performance - while we see an
>   improvement with testpmd with the default burst size of 32, burst sizes
>   of up to 128 give further improvements
> * In my testing, most of the improvement comes from faster processing on
>   the RX path, though the improved TX also gives benefit.
> 
> This has been tested on a system with CPU: "Intel(R) Xeon(R) Gold 6154 CPU @
> 3.00GHz", and I've focused on testing with Rx ring sizes of approx 1k - generally
> --rxd=1024 and --txd=512, rather than the defaults which tend to give poorer
> zero-loss performance due to the smaller amount of buffering.
> 
> V2:
> * Fixed incorrect config variable reference in makefile
> * Added missing stub function for when vector drivers are disabled
> * Added missing references to the new functions when checking for vector
>   code paths, e.g. for ring tear-down
> 
> Bruce Richardson (2):
>   net/i40e: add AVX2 Tx function
>   net/i40e: add AVX2 Rx function
> 
>  drivers/net/i40e/Makefile             |  19 +
>  drivers/net/i40e/i40e_rxtx.c          |  66 ++-
>  drivers/net/i40e/i40e_rxtx.h          |   6 +
>  drivers/net/i40e/i40e_rxtx_vec_avx2.c | 792
> ++++++++++++++++++++++++++++++++++
>  4 files changed, 880 insertions(+), 3 deletions(-)  create mode 100644
> drivers/net/i40e/i40e_rxtx_vec_avx2.c
> 
> --
> 2.14.3

Acked-by: Qi Zhang <qi.z.zhang@intel.com>