> -----Original Message-----
> From: Coyle, David
> Sent: Tuesday, July 31, 2018 10:25 PM
> To: Van Haaren, Harry <[email protected]>; [email protected]
> Cc: Richardson, Bruce <[email protected]>; [email protected];
> [email protected]; Xing, Beilei <[email protected]>; Zhang, Qi Z
> <[email protected]>
> Subject: RE: [PATCH v2] net/i40e: fix avx2 driver check for rx rearm
>
>
>
> -----Original Message-----
> From: Van Haaren, Harry
> Sent: Monday, July 30, 2018 6:34 PM
> To: [email protected]
> Cc: Van Haaren, Harry <[email protected]>; Richardson, Bruce
> <[email protected]>; [email protected]; [email protected];
> Coyle, David <[email protected]>; Xing, Beilei <[email protected]>;
> Zhang, Qi Z <[email protected]>
> Subject: [PATCH v2] net/i40e: fix avx2 driver check for rx rearm
>
> This commit fixes an infinite loop bug that could occur if the i40e AVX2
> driver
> is used, and high traffic rates cause the mempool from which the rxq pulls
> mbufs to become empty.
>
> The result would be an infinite loop of checking if we should perform an rx
> rearm, calling the function and an error return due the the mempool being
> emtpy.
>
> The fix is to align the code in the AVX2 driver with the SSE driver, where an
> if()
> is used instead of a while(), allowing the thread to return from i40e rx
> function even if the mempool is empty.
>
> Fixes: dafadd73762e ("net/i40e: add AVX2 Rx function")
> Cc: [email protected]
> Cc: [email protected]
>
> Reported-by: David Coyle <[email protected]>
> Signed-off-by: Harry van Haaren <[email protected]>
> Acked-by: Brendan Ryan <[email protected]>
> Tested-by: David Coyle <[email protected]>
Applied to dpdk-next-net-intel.
Thanks!
Qi
