[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Hunt, David
On 16/6/2016 9:58 AM, Olivier MATZ wrote: >>> >>> So I don't think we should have more cache misses whether it's >>> placed at the beginning or at the end. Maybe I'm missing something... >>> >>> I still believe it's better to group the 2 fields as they are >>> tightly linked together. It could be

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Olivier MATZ
>> >> So I don't think we should have more cache misses whether it's >> placed at the beginning or at the end. Maybe I'm missing something... >> >> I still believe it's better to group the 2 fields as they are >> tightly linked together. It could be at the end if you see better >> performance. >> >

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Olivier MATZ
On 06/16/2016 09:47 AM, Hunt, David wrote: > > > On 15/6/2016 5:40 PM, Olivier MATZ wrote: >> >> >> On 06/15/2016 06:34 PM, Hunt, David wrote: >>> >>> >>> On 15/6/2016 1:03 PM, Olivier MATZ wrote: [...] The opaque pointer would be saved in mempool structure, and used when the

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Hunt, David
On 16/6/2016 9:47 AM, Olivier MATZ wrote: > > > On 06/16/2016 09:47 AM, Hunt, David wrote: >> >> >> On 15/6/2016 5:40 PM, Olivier MATZ wrote: >>> >>> >>> On 06/15/2016 06:34 PM, Hunt, David wrote: On 15/6/2016 1:03 PM, Olivier MATZ wrote: > [...] > > The opaque pointer

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Hunt, David
On 15/6/2016 5:40 PM, Olivier MATZ wrote: > > > On 06/15/2016 06:34 PM, Hunt, David wrote: >> >> >> On 15/6/2016 1:03 PM, Olivier MATZ wrote: >>> [...] >>> >>> The opaque pointer would be saved in mempool structure, and used >>> when the mempool is populated (calling mempool_ops_alloc). >>> The t

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Hunt, David
Hi Shreyansh, On 16/6/2016 5:35 AM, Shreyansh Jain wrote: > Though I am late to the discussion... > >> -Original Message- >> From: Olivier MATZ [mailto:olivier.matz at 6wind.com] >> Sent: Wednesday, June 15, 2016 10:10 PM >> To: Hunt, David ; Jan Viktorin >> >> Cc: dev at dpdk.org; jerin.

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-16 Thread Shreyansh Jain
Though I am late to the discussion... > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Wednesday, June 15, 2016 10:10 PM > To: Hunt, David ; Jan Viktorin > > Cc: dev at dpdk.org; jerin.jacob at caviumnetworks.com; Shreyansh Jain > > Subject: Re: [PATCH

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Olivier MATZ
On 06/15/2016 06:34 PM, Hunt, David wrote: > > > On 15/6/2016 1:03 PM, Olivier MATZ wrote: >> [...] >> >> The opaque pointer would be saved in mempool structure, and used >> when the mempool is populated (calling mempool_ops_alloc). >> The type of the structure pointed by the opaque has to be def

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Hunt, David
On 15/6/2016 1:03 PM, Olivier MATZ wrote: > [...] > > The opaque pointer would be saved in mempool structure, and used > when the mempool is populated (calling mempool_ops_alloc). > The type of the structure pointed by the opaque has to be defined > (and documented) into each mempool_ops manager.

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Hunt, David
On 15/6/2016 3:47 PM, Jan Viktorin wrote: > On Wed, 15 Jun 2016 16:10:13 +0200 > Olivier MATZ wrote: > >> On 06/15/2016 04:02 PM, Hunt, David wrote: >>> >>> On 15/6/2016 2:50 PM, Olivier MATZ wrote: Hi David, On 06/15/2016 02:38 PM, Hunt, David wrote: > > On 15/6/2016 1:03

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Jan Viktorin
On Wed, 15 Jun 2016 16:10:13 +0200 Olivier MATZ wrote: > On 06/15/2016 04:02 PM, Hunt, David wrote: > > > > > > On 15/6/2016 2:50 PM, Olivier MATZ wrote: > >> Hi David, > >> > >> On 06/15/2016 02:38 PM, Hunt, David wrote: > >>> > >>> > >>> On 15/6/2016 1:03 PM, Olivier MATZ wrote: > Hi

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Olivier MATZ
On 06/15/2016 04:02 PM, Hunt, David wrote: > > > On 15/6/2016 2:50 PM, Olivier MATZ wrote: >> Hi David, >> >> On 06/15/2016 02:38 PM, Hunt, David wrote: >>> >>> >>> On 15/6/2016 1:03 PM, Olivier MATZ wrote: Hi, On 06/15/2016 01:47 PM, Hunt, David wrote: > > > On 15/6/20

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Olivier MATZ
Hi David, On 06/15/2016 02:38 PM, Hunt, David wrote: > > > On 15/6/2016 1:03 PM, Olivier MATZ wrote: >> Hi, >> >> On 06/15/2016 01:47 PM, Hunt, David wrote: >>> >>> >>> On 15/6/2016 11:13 AM, Jan Viktorin wrote: Hi, I've got one last question. Initially, I was interested in creating

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Hunt, David
On 15/6/2016 2:50 PM, Olivier MATZ wrote: > Hi David, > > On 06/15/2016 02:38 PM, Hunt, David wrote: >> >> >> On 15/6/2016 1:03 PM, Olivier MATZ wrote: >>> Hi, >>> >>> On 06/15/2016 01:47 PM, Hunt, David wrote: On 15/6/2016 11:13 AM, Jan Viktorin wrote: > Hi, > > I've g

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Olivier MATZ
Hi, On 06/15/2016 01:47 PM, Hunt, David wrote: > > > On 15/6/2016 11:13 AM, Jan Viktorin wrote: >> Hi, >> >> I've got one last question. Initially, I was interested in creating >> my own external memory provider based on a Linux Kernel driver. >> So, I've got an opened file descriptor that points

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Hunt, David
On 15/6/2016 1:03 PM, Olivier MATZ wrote: > Hi, > > On 06/15/2016 01:47 PM, Hunt, David wrote: >> >> >> On 15/6/2016 11:13 AM, Jan Viktorin wrote: >>> Hi, >>> >>> I've got one last question. Initially, I was interested in creating >>> my own external memory provider based on a Linux Kernel driver

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Hunt, David
On 15/6/2016 11:13 AM, Jan Viktorin wrote: > Hi, > > I've got one last question. Initially, I was interested in creating > my own external memory provider based on a Linux Kernel driver. > So, I've got an opened file descriptor that points to a device which > can mmap a memory regions for me. > >

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread Jan Viktorin
Hi, I've got one last question. Initially, I was interested in creating my own external memory provider based on a Linux Kernel driver. So, I've got an opened file descriptor that points to a device which can mmap a memory regions for me. ... int fd = open("/dev/uio0" ...); ... rte_mempool *pool

[dpdk-dev] [PATCH v12 0/3] mempool: add external mempool manager

2016-06-15 Thread David Hunt
Here's the latest version of the External Mempool Manager patchset. It's re-based on top of the latest head as of 14/6/2016, including Olivier's 35-part patch series on mempool re-org [1] [1] http://dpdk.org/ml/archives/dev/2016-May/039229.html v12 changes: * Fixed a comment (function pram h ->