RE: [PATCH v7 1/2] net/iavf: fix Rx/Tx burst in multi-process

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

> -----Original Message-----
> From: Mingjin Ye <mingjinx.ye@intel.com>
> Sent: Tuesday, January 2, 2024 6:52 PM
> To: dev@dpdk.org
> Cc: Yang, Qiming <qiming.yang@intel.com>; Ye, MingjinX
> <mingjinx.ye@intel.com>; stable@dpdk.org; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: [PATCH v7 1/2] net/iavf: fix Rx/Tx burst in multi-process
> 
> In a multi-process environment, a secondary process operates on shared
> memory and changes the function pointer of the primary process, resulting
> in a crash when the primary process cannot find the function address during
> an Rx/Tx burst.
> 
> Fixes: 5b3124a0a6ef ("net/iavf: support no polling when link down")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> ---
> v2: Add fix for Rx burst.
> ---
> v3: fix Rx/Tx routing.
> ---
>  drivers/net/iavf/iavf.h      |  42 ++++++++-
>  drivers/net/iavf/iavf_rxtx.c | 173 +++++++++++++++++++++++------------
>  2 files changed, 157 insertions(+), 58 deletions(-)
> 
> diff --git a/drivers/net/iavf/iavf.h b/drivers/net/iavf/iavf.h index
> 10868f2c30..73a089c199 100644
> --- a/drivers/net/iavf/iavf.h
> +++ b/drivers/net/iavf/iavf.h
> @@ -313,6 +313,44 @@ struct iavf_devargs {
> 
>  struct iavf_security_ctx;
> 
> +enum iavf_rx_burst_type {
> +	IAVF_RX_DEFAULT,
> +	IAVF_RX_FLEX_RXD,
> +	IAVF_RX_BULK_ALLOC,
> +	IAVF_RX_SCATTERED,
> +	IAVF_RX_SCATTERED_FLEX_RXD,
> +	IAVF_RX_SSE,
> +	IAVF_RX_AVX2,
> +	IAVF_RX_AVX2_OFFLOAD,
> +	IAVF_RX_SSE_FLEX_RXD,
> +	IAVF_RX_AVX2_FLEX_RXD,
> +	IAVF_RX_AVX2_FLEX_RXD_OFFLOAD,
> +	IAVF_RX_SSE_SCATTERED,
> +	IAVF_RX_AVX2_SCATTERED,
> +	IAVF_RX_AVX2_SCATTERED_OFFLOAD,
> +	IAVF_RX_SSE_SCATTERED_FLEX_RXD,
> +	IAVF_RX_AVX2_SCATTERED_FLEX_RXD,
> +	IAVF_RX_AVX2_SCATTERED_FLEX_RXD_OFFLOAD,
> +	IAVF_RX_AVX512,
> +	IAVF_RX_AVX512_OFFLOAD,
> +	IAVF_RX_AVX512_FLEX_RXD,
> +	IAVF_RX_AVX512_FLEX_RXD_OFFLOAD,
> +	IAVF_RX_AVX512_SCATTERED,
> +	IAVF_RX_AVX512_SCATTERED_OFFLOAD,
> +	IAVF_RX_AVX512_SCATTERED_FLEX_RXD,
> +	IAVF_RX_AVX512_SCATTERED_FLEX_RXD_OFFLOAD,
> +};
> +
> +enum iavf_tx_burst_type {
> +	IAVF_TX_DEFAULT,
> +	IAVF_TX_SSE,
> +	IAVF_TX_AVX2,
> +	IAVF_TX_AVX2_OFFLOAD,
> +	IAVF_TX_AVX512,
> +	IAVF_TX_AVX512_OFFLOAD,
> +	IAVF_TX_AVX512_CTX_OFFLOAD,
> +};
> +
>  /* Structure to store private data for each VF instance. */  struct iavf_adapter
> {
>  	struct iavf_hw hw;
> @@ -328,8 +366,8 @@ struct iavf_adapter {
>  	bool stopped;
>  	bool closed;
>  	bool no_poll;
> -	eth_rx_burst_t rx_pkt_burst;
> -	eth_tx_burst_t tx_pkt_burst;
> +	enum iavf_rx_burst_type rx_burst_type;
> +	enum iavf_tx_burst_type tx_burst_type;
>  	uint16_t fdir_ref_cnt;
>  	struct iavf_devargs devargs;
>  };
> diff --git a/drivers/net/iavf/iavf_rxtx.c b/drivers/net/iavf/iavf_rxtx.c index
> f19aa14646..13b932ad85 100644
> --- a/drivers/net/iavf/iavf_rxtx.c
> +++ b/drivers/net/iavf/iavf_rxtx.c
> @@ -3707,15 +3707,68 @@ iavf_prep_pkts(__rte_unused void *tx_queue,
> struct rte_mbuf **tx_pkts,
>  	return i;
>  }
> 
> +static
> +const eth_rx_burst_t iavf_rx_pkt_burst_ops[] = {

Its will broke if we insert a new type without change the ops array.

It's better to explicitly initial an array with index

[IAVF_RX_DEFAULT] = iavf_recv_pkts,

....