Re: [dpdk-dev] [PATCH v4 1/7] ethdev: allocate max space for internal queue array

2021-10-05 Thread Thomas Monjalon
05/10/2021 18:45, Ananyev, Konstantin: > > > At queue configure stage always allocate space for maximum possible > > > number (RTE_MAX_QUEUES_PER_PORT) of queue pointers. > > > That will allow 'fast' inline functions (eth_rx_burst, etc.) to refer > > > pointer to internal queue data without extra c

Re: [dpdk-dev] [PATCH v4 1/7] ethdev: allocate max space for internal queue array

2021-10-05 Thread Ananyev, Konstantin
> > At queue configure stage always allocate space for maximum possible > > number (RTE_MAX_QUEUES_PER_PORT) of queue pointers. > > That will allow 'fast' inline functions (eth_rx_burst, etc.) to refer > > pointer to internal queue data without extra checking of current number > > of configured que

Re: [dpdk-dev] [PATCH v4 1/7] ethdev: allocate max space for internal queue array

2021-10-05 Thread Thomas Monjalon
04/10/2021 15:55, Konstantin Ananyev: > At queue configure stage always allocate space for maximum possible > number (RTE_MAX_QUEUES_PER_PORT) of queue pointers. The problem with this max is that it cannot be changed dynamically. That could be another patch, but I would like to see this number a d

Re: [dpdk-dev] [PATCH v4 1/7] ethdev: allocate max space for internal queue array

2021-10-05 Thread Thomas Monjalon
04/10/2021 15:55, Konstantin Ananyev: > At queue configure stage always allocate space for maximum possible > number (RTE_MAX_QUEUES_PER_PORT) of queue pointers. > That will allow 'fast' inline functions (eth_rx_burst, etc.) to refer > pointer to internal queue data without extra checking of curren

[dpdk-dev] [PATCH v4 1/7] ethdev: allocate max space for internal queue array

2021-10-04 Thread Konstantin Ananyev
At queue configure stage always allocate space for maximum possible number (RTE_MAX_QUEUES_PER_PORT) of queue pointers. That will allow 'fast' inline functions (eth_rx_burst, etc.) to refer pointer to internal queue data without extra checking of current number of configured queues. That would help