[dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Thomas Monjalon
Performance of access in a fixed-size array is very good because of cache locality and because there is a single pointer to dereference. The only drawback is the lack of flexibility: the size of such an array cannot be increase at runtime. An approach to this problem is to allocate the array at ru

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, 14 June 2021 12.59 > > Performance of access in a fixed-size array is very good > because of cache locality > and because there is a single pointer to dereference. > The only drawback is the lack of flexibility:

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Bruce Richardson
On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > Sent: Monday, 14 June 2021 12.59 > > > > Performance of access in a fixed-size array is very good > > because of cache locality > > and because there is a sin

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Thomas Monjalon
14/06/2021 14:22, Morten Brørup: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > Sent: Monday, 14 June 2021 12.59 > > > > Performance of access in a fixed-size array is very good > > because of cache locality > > and because there is a single pointer to dereference. >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Thomas Monjalon
14/06/2021 15:15, Bruce Richardson: > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > Sent: Monday, 14 June 2021 12.59 > > > > > > Performance of access in a fixed-size array is very good > > > because

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Ananyev, Konstantin
> > 14/06/2021 15:15, Bruce Richardson: > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > > Sent: Monday, 14 June 2021 12.59 > > > > > > > > Performance of access in a fixed-size array is very goo

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, 14 June 2021 15.32 > > 14/06/2021 15:15, Bruce Richardson: > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas > Monjalon > > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Jerin Jacob
On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin wrote: > > > > > > 14/06/2021 15:15, Bruce Richardson: > > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > > > Sent: Monday, 14 June 2021 12.59

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Ananyev, Konstantin
> > > > 14/06/2021 15:15, Bruce Richardson: > > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > > > Sent: Monday, 14 June 2021 12.59 > > > > > > > > > > Performance of access in a fixed-size arr

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Thomas Monjalon
14/06/2021 17:48, Morten Brørup: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > It would be much simpler to just increase RTE_MAX_ETHPORTS to something big > enough to hold a sufficiently large array. And possibly add an > rte_max_ethports variable to indicate the numb

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-14 Thread Thomas Monjalon
14/06/2021 17:48, Jerin Jacob: > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > wrote: > > > 14/06/2021 15:15, Bruce Richardson: > > > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Tuesday, 15 June 2021 08.48 > > 14/06/2021 17:48, Morten Brørup: > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas > Monjalon > > It would be much simpler to just increase RTE_MAX_ETHPORTS to > something

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Jerin Jacob
On Tue, Jun 15, 2021 at 12:22 PM Thomas Monjalon wrote: > > 14/06/2021 17:48, Jerin Jacob: > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > wrote: > > > > 14/06/2021 15:15, Bruce Richardson: > > > > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > > > > From: de

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Bruce Richardson
On Tue, Jun 15, 2021 at 09:53:33AM +0200, Morten Brørup wrote: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > Sent: Tuesday, 15 June 2021 08.48 > > > > 14/06/2021 17:48, Morten Brørup: > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas > > Monjalon >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Thomas Monjalon
15/06/2021 10:00, Jerin Jacob: > On Tue, Jun 15, 2021 at 12:22 PM Thomas Monjalon wrote: > > 14/06/2021 17:48, Jerin Jacob: > > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > > wrote: > > > > > 14/06/2021 15:15, Bruce Richardson: > > > > > > While I dislike the hard-coded limits in DPD

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Thomas Monjalon
15/06/2021 10:44, Bruce Richardson: > On Tue, Jun 15, 2021 at 09:53:33AM +0200, Morten Brørup wrote: > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > > Sent: Tuesday, 15 June 2021 08.48 > > > > > > 14/06/2021 17:48, Morten Brørup: > > > > > From: dev [mailto:dev-boun.

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Ananyev, Konstantin
> 14/06/2021 17:48, Jerin Jacob: > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > wrote: > > > > 14/06/2021 15:15, Bruce Richardson: > > > > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Thomas Monjalon
15/06/2021 11:33, Ananyev, Konstantin: > > 14/06/2021 17:48, Jerin Jacob: > > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > > wrote: > > > > I had only a quick look at your approach so far. > > > > But from what I can read, in MT environment your suggestion will require > > > > extra s

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Ananyev, Konstantin
> > 15/06/2021 11:33, Ananyev, Konstantin: > > > 14/06/2021 17:48, Jerin Jacob: > > > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > > > wrote: > > > > > I had only a quick look at your approach so far. > > > > > But from what I can read, in MT environment your suggestion will > > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Thomas Monjalon
15/06/2021 12:08, Ananyev, Konstantin: > > 15/06/2021 11:33, Ananyev, Konstantin: > > > > 14/06/2021 17:48, Jerin Jacob: > > > > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > > > > wrote: > > > > > > I had only a quick look at your approach so far. > > > > > > But from what I can read,

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-15 Thread Honnappa Nagarahalli
> > 15/06/2021 12:08, Ananyev, Konstantin: > > > 15/06/2021 11:33, Ananyev, Konstantin: > > > > > 14/06/2021 17:48, Jerin Jacob: > > > > > > On Mon, Jun 14, 2021 at 8:29 PM Ananyev, Konstantin > > > > > > wrote: > > > > > > > I had only a quick look at your approach so far. > > > > > > > But fr

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Jerin Jacob
On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon wrote: > > 14/06/2021 17:48, Morten Brørup: > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > > It would be much simpler to just increase RTE_MAX_ETHPORTS to something big > > enough to hold a sufficiently large array. An

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Burakov, Anatoly
On 14-Jun-21 11:58 AM, Thomas Monjalon wrote: Performance of access in a fixed-size array is very good because of cache locality and because there is a single pointer to dereference. The only drawback is the lack of flexibility: the size of such an array cannot be increase at runtime. An approac

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Morten Brørup
> From: Jerin Jacob [mailto:jerinjac...@gmail.com] > Sent: Wednesday, 16 June 2021 11.42 > > On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon > wrote: > > > > 14/06/2021 17:48, Morten Brørup: > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas > Monjalon > > > It would be much sim

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Jerin Jacob
On Wed, Jun 16, 2021 at 4:57 PM Morten Brørup wrote: > > > From: Jerin Jacob [mailto:jerinjac...@gmail.com] > > Sent: Wednesday, 16 June 2021 11.42 > > > > On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon > > wrote: > > > > > > 14/06/2021 17:48, Morten Brørup: > > > > > From: dev [mailto:dev-bou

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Burakov, Anatoly
On 16-Jun-21 10:42 AM, Jerin Jacob wrote: On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon wrote: 14/06/2021 17:48, Morten Brørup: From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon It would be much simpler to just increase RTE_MAX_ETHPORTS to something big enough to hold

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Jerin Jacob
On Wed, Jun 16, 2021 at 5:52 PM Burakov, Anatoly wrote: > > On 16-Jun-21 10:42 AM, Jerin Jacob wrote: > > On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon > > wrote: > >> > >> 14/06/2021 17:48, Morten Brørup: > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > >>> It wo

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Bruce Richardson
On Wed, Jun 16, 2021 at 01:27:17PM +0200, Morten Brørup wrote: > > From: Jerin Jacob [mailto:jerinjac...@gmail.com] > > Sent: Wednesday, 16 June 2021 11.42 > > > > On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon > > wrote: > > > > > > 14/06/2021 17:48, Morten Brørup: > > > > > From: dev [mailto:

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > Sent: Wednesday, 16 June 2021 15.03 > > On Wed, Jun 16, 2021 at 01:27:17PM +0200, Morten Brørup wrote: > > > From: Jerin Jacob [mailto:jerinjac...@gmail.com] > > > Sent: Wednesday, 16 June 2021 11.42 > > > > > > On Tue, Jun

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Bruce Richardson
On Wed, Jun 16, 2021 at 05:01:46PM +0200, Morten Brørup wrote: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > > Sent: Wednesday, 16 June 2021 15.03 > > > > On Wed, Jun 16, 2021 at 01:27:17PM +0200, Morten Brørup wrote: > > > > From: Jerin Jacob [mailto:jerinjac...@gmai

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-16 Thread Dmitry Kozlyuk
2021-06-16 18:29 (UTC+0530), Jerin Jacob: > On Wed, Jun 16, 2021 at 5:52 PM Burakov, Anatoly > wrote: > > > > On 16-Jun-21 10:42 AM, Jerin Jacob wrote: > > > On Tue, Jun 15, 2021 at 12:18 PM Thomas Monjalon > > > wrote: > > >> > > >> 14/06/2021 17:48, Morten Brørup: > > From: dev [mai

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Ferruh Yigit
On 6/14/2021 4:54 PM, Ananyev, Konstantin wrote: > > >>> >>> 14/06/2021 15:15, Bruce Richardson: On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon >> Sent: Monday, 14 June 2021 12.59 >> >> Pe

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Ananyev, Konstantin
> >>> > >>> 14/06/2021 15:15, Bruce Richardson: > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > >> Sent: Monday, 14 June 2021 12.59 > >> > >> Performance of access in a fixed-size arr

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Morten Brørup
> From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] > Sent: Thursday, 17 June 2021 16.59 > > > >>> > > >>> 14/06/2021 15:15, Bruce Richardson: > > On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > > >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thoma

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Ferruh Yigit
On 6/17/2021 3:58 PM, Ananyev, Konstantin wrote: > > > > 14/06/2021 15:15, Bruce Richardson: >> On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon Sent: Monday, 14 June 2021 12.59 >>

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Ferruh Yigit
On 6/17/2021 4:17 PM, Morten Brørup wrote: >> From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] >> Sent: Thursday, 17 June 2021 16.59 >> >> >> 14/06/2021 15:15, Bruce Richardson: >>> On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > From: dev [mailto

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit > Sent: Thursday, 17 June 2021 18.13 > > On 6/17/2021 4:17 PM, Morten Brørup wrote: > >> From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] > >> Sent: Thursday, 17 June 2021 16.59 > >> > >> > >> 14/06/2021 15:

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-17 Thread Ananyev, Konstantin
> On 6/17/2021 4:17 PM, Morten Brørup wrote: > >> From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] > >> Sent: Thursday, 17 June 2021 16.59 > >> > >> > >> 14/06/2021 15:15, Bruce Richardson: > >>> On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > Sent: Thursday, 17 June 2021 19.06 > > I think it would be better to change rx_pkt_burst() to accept port_id > and queue_id, instead of void *. Current: typedef uint16_t (*eth_rx_burst_t)(void *rxq,

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Ferruh Yigit
On 6/17/2021 5:55 PM, Morten Brørup wrote: >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit >> Sent: Thursday, 17 June 2021 18.13 >> >> On 6/17/2021 4:17 PM, Morten Brørup wrote: From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] Sent: Thursday, 17 June 20

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Ferruh Yigit
On 6/17/2021 6:05 PM, Ananyev, Konstantin wrote: > > >> On 6/17/2021 4:17 PM, Morten Brørup wrote: From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] Sent: Thursday, 17 June 2021 16.59 14/06/2021 15:15, Bruce Richardson: > On Mon, Jun 14, 2021

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Ananyev, Konstantin
> > > > 14/06/2021 15:15, Bruce Richardson: > >> On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, 14 June 2021 12.59 > > Performance of access in a f

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Ferruh Yigit
On 6/18/2021 10:14 AM, Morten Brørup wrote: >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, >> Konstantin >> Sent: Thursday, 17 June 2021 19.06 >> >> I think it would be better to change rx_pkt_burst() to accept port_id >> and queue_id, instead of void *. > > Current: > > typedef

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Ferruh Yigit
On 6/18/2021 11:41 AM, Ananyev, Konstantin wrote: > >>> >>> 14/06/2021 15:15, Bruce Richardson: On Mon, Jun 14, 2021 at 02:22:42PM +0200, Morten Brørup wrote: >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon >> Sent: Monday, 14 June 2021 12.5

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-18 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit > Sent: Friday, 18 June 2021 12.47 > > On 6/18/2021 10:14 AM, Morten Brørup wrote: > > Another thing: > > > > I just noticed that struct rte_eth_dev_data has "void **rx_queues;" > (and similarly for tx_queues). > > > > That should

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
Hi everyone, > > >>> One more thought here - if we are talking about rte_ethdev[] in > > >>> particular, I think we can: > > >>> 1. move public function pointers (rx_pkt_burst(), etc.) from rte_ethdev > > >>> into a separate flat array. > > >>> We can keep it public to still use inline functi

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > > > > How can we hide the callbacks since they are used by inline burst > functions. > > > > I probably I owe a better explanation to what I meant in first mail. > > Otherwise it sounds confusing. > > I'll try to write

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
> > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > Konstantin > > > > > > How can we hide the callbacks since they are used by inline burst > > functions. > > > > > > I probably I owe a better explanation to what I meant in first mail. > > > Otherwise it sounds confusing. > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > Konstantin > > > > > > > > How can we hide the callbacks since they are used by inline > burst > > > functions. > > > > > > > > I probably I

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ferruh Yigit
On 6/21/2021 12:06 PM, Ananyev, Konstantin wrote: > > Hi everyone, > >> One more thought here - if we are talking about rte_ethdev[] in >> particular, I think we can: >> 1. move public function pointers (rx_pkt_burst(), etc.) from rte_ethdev >> into a separate flat array. >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ferruh Yigit
On 6/21/2021 1:30 PM, Ananyev, Konstantin wrote: > >> >>> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, >>> Konstantin >>> > How can we hide the callbacks since they are used by inline burst >>> functions. I probably I owe a better explanation to what I meant in first

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
> > On 6/21/2021 1:30 PM, Ananyev, Konstantin wrote: > > > >> > >>> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > >>> Konstantin > >>> > > How can we hide the callbacks since they are used by inline burst > >>> functions. > > I probably I owe a better explanation t

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
> >> One more thought here - if we are talking about rte_ethdev[] in > >> particular, I think we can: > >> 1. move public function pointers (rx_pkt_burst(), etc.) from > >> rte_ethdev into a separate flat array. > >> We can keep it public to still use inline functions for 'f

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ferruh Yigit
On 6/21/2021 3:42 PM, Ananyev, Konstantin wrote: > One more thought here - if we are talking about rte_ethdev[] in particular, I think we can: 1. move public function pointers (rx_pkt_burst(), etc.) from rte_ethdev into a separate flat array. We can

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
> > On 6/21/2021 3:42 PM, Ananyev, Konstantin wrote: > > > One more thought here - if we are talking about rte_ethdev[] in > particular, I think we can: > 1. move public function pointers (rx_pkt_burst(), etc.) from > rte_ethdev into a separate flat array. >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ferruh Yigit
On 6/21/2021 3:38 PM, Ananyev, Konstantin wrote: > >> >> On 6/21/2021 1:30 PM, Ananyev, Konstantin wrote: >>> > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > >>> How can we hide the callbacks since they are used by inline burst > functions. >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-21 Thread Ananyev, Konstantin
> > On 6/21/2021 3:38 PM, Ananyev, Konstantin wrote: > > > >> > >> On 6/21/2021 1:30 PM, Ananyev, Konstantin wrote: > >>> > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > Konstantin > > > >>> How can we hide the callbacks since they are used by inline

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-22 Thread Ananyev, Konstantin
> > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > Konstantin > > > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > > Konstantin > > > > > > > > > > How can we hide the callbacks since they are used by inline > > burst > > > > functions. > > > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-22 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > Konstantin > > > > > > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > > > Konstantin > > > > > > > > > > > >

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-22 Thread Ananyev, Konstantin
> > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > Konstantin > > > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > > Konstantin > > > > > > > > > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > > > > Konstantin

Re: [dpdk-dev] [PATCH] parray: introduce internal API for dynamic arrays

2021-06-22 Thread Morten Brørup
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > Konstantin > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > Konstantin > > > > > > > > > > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ananyev, > > > > > Konstantin > > > > > > > > > > > >