From: Wenzhuo Lu Date: Thu, 29 Apr 2021 01:33:57 +0000 (+0800) Subject: net/iavf: fix Rx function selection X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=bb3ef9aaa478;p=dpdk.git net/iavf: fix Rx function selection A performance drop is caused by that the RX scalar path is selected when AVX512 is disabled and some HW offload is enabled. Actually, the HW offload is supported by AVX2 and SSE. In this scenario AVX2 path should be chosen. This patch removes the offload related check for SSE and AVX2 as SSE and AVX2 do support the offload features. No implementation change about the data path. Fixes: eff56a7b9f97 ("net/iavf: add offload path for Rx AVX512") Signed-off-by: Wenzhuo Lu Acked-by: Qi Zhang --- diff --git a/drivers/net/iavf/iavf_rxtx.c b/drivers/net/iavf/iavf_rxtx.c index 232516c414..87f7eebc65 100644 --- a/drivers/net/iavf/iavf_rxtx.c +++ b/drivers/net/iavf/iavf_rxtx.c @@ -2401,13 +2401,11 @@ iavf_set_rx_function(struct rte_eth_dev *dev) check_ret = iavf_rx_vec_dev_check(dev); if (check_ret >= 0 && rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_128) { - if (check_ret == IAVF_VECTOR_PATH) { - use_sse = true; - if ((rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX2) == 1 || - rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512F) == 1) && - rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_256) - use_avx2 = true; - } + use_sse = true; + if ((rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX2) == 1 || + rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512F) == 1) && + rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_256) + use_avx2 = true; #ifdef CC_AVX512_SUPPORT if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512F) == 1 &&