On Fri, Oct 16, 2020 at 1:43 PM Ciara Power <ciara.po...@intel.com> wrote:
>
> When choosing a vector path to take, an extra condition must be
> satisfied to ensure the max SIMD bitwidth allows for the CPU enabled
> path.
>
> Cc: Ajit Khaparde <ajit.khapa...@broadcom.com>
> Cc: Somnath Kotur <somnath.ko...@broadcom.com>
>
> Signed-off-by: Ciara Power <ciara.po...@intel.com>
>
> ---
> v4: Updated enum name.
> ---
> drivers/net/bnxt/bnxt_ethdev.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
> index 8b63134c39..07d1a1a6ab 100644
> --- a/drivers/net/bnxt/bnxt_ethdev.c
> +++ b/drivers/net/bnxt/bnxt_ethdev.c
> @@ -1169,7 +1169,8 @@ bnxt_receive_function(struct rte_eth_dev *eth_dev)
> DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM |
> DEV_RX_OFFLOAD_RSS_HASH |
> DEV_RX_OFFLOAD_VLAN_FILTER)) &&
> - !BNXT_TRUFLOW_EN(bp) && BNXT_NUM_ASYNC_CPR(bp)) {
> + !BNXT_TRUFLOW_EN(bp) && BNXT_NUM_ASYNC_CPR(bp) &&
> + rte_get_max_simd_bitwidth() >= RTE_SIMD_128) {
> PMD_DRV_LOG(INFO, "Using vector mode receive for port %d\n",
> eth_dev->data->port_id);
> bp->flags |= BNXT_FLAG_RX_VECTOR_PKT_MODE;
> @@ -1202,7 +1203,8 @@ bnxt_transmit_function(__rte_unused struct rte_eth_dev
> *eth_dev)
> */
> if (!eth_dev->data->scattered_rx &&
> !(offloads & ~DEV_TX_OFFLOAD_MBUF_FAST_FREE) &&
> - !BNXT_TRUFLOW_EN(bp)) {
> + !BNXT_TRUFLOW_EN(bp) &&
> + rte_get_max_simd_bitwidth() >= RTE_SIMD_128) {
> PMD_DRV_LOG(INFO, "Using vector mode transmit for port %d\n",
> eth_dev->data->port_id);
> return bnxt_xmit_pkts_vec;
> --
> 2.22.0
>
Acked-by Somnath kotur <somnath.ko...@broadcom.com>
