On Thu, 21 Nov 2019 16:12:55 +0100
David Marchand <david.march...@redhat.com> wrote:

> -     uint16_t                nb_rx_desc[MAX_QUEUE_ID+1]; /**< per queue rx 
> desc number */
> -     uint16_t                nb_tx_desc[MAX_QUEUE_ID+1]; /**< per queue tx 
> desc number */
> -     struct rte_eth_rxconf   rx_conf[MAX_QUEUE_ID+1]; /**< per queue rx 
> configuration */
> -     struct rte_eth_txconf   tx_conf[MAX_QUEUE_ID+1]; /**< per queue tx 
> configuration */
> +     uint16_t                nb_rx_desc[RTE_MAX_QUEUES_PER_PORT+1]; /**< per 
> queue rx desc number */
> +     uint16_t                nb_tx_desc[RTE_MAX_QUEUES_PER_PORT+1]; /**< per 
> queue tx desc number */
> +     struct rte_eth_rxconf   rx_conf[RTE_MAX_QUEUES_PER_PORT+1]; /**< per 
> queue rx configuration */
> +     struct rte_eth_txconf   tx_conf[RTE_MAX_QUEUES_PER_PORT+1]; /**< per 
> queue tx configuration */

Why not  put all the per-queue stuff together in one structure
and put it at the end. Then dynamically size based on number of queues?

Reply via email to