[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-25 Thread Mauricio Vásquez
Hello to everybody, I find this proposal very interesting as It could be used to solve an issue that has not been mentioned yet: using memory pools shared with ivshmem. Currently, due to the per lcore cache, it is not possible to allocate and deallocate packets from the guest as it will cause cach

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-25 Thread Venkatesan, Venky
> -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Friday, March 25, 2016 3:56 AM > To: Venkatesan, Venky ; Lazaros Koromilas > ; Wiles, Keith > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mempool: allow for user-owned mem

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-25 Thread Olivier Matz
Hi Venky, >> The main benefit of having an external cache is to allow mempool users >> (threads) to maintain a local cache even though they don't have a valid >> lcore_id (non-EAL threads). The fact that cache access is done by indexing >> with the lcore_id is what makes it difficult... > > Hi La

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-24 Thread Lazaros Koromilas
On Mon, Mar 21, 2016 at 3:49 PM, Wiles, Keith wrote: >>Hi Lazaros, >> >>Thanks for this patch. To me, this is a valuable enhancement. >>Please find some comments inline. >> >>On 03/10/2016 03:44 PM, Lazaros Koromilas wrote: >>> The mempool cache is only available to EAL threads as a per-lcore >>>

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-24 Thread Lazaros Koromilas
Hi Konstantin, On Mon, Mar 21, 2016 at 3:15 PM, Ananyev, Konstantin wrote: > > Hi lads, > >> >> Hi Lazaros, >> >> Thanks for this patch. To me, this is a valuable enhancement. >> Please find some comments inline. > > Yep, patch seems interesting. > One question - what would be the usage model for

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-24 Thread Wiles, Keith
>On Mon, Mar 21, 2016 at 3:49 PM, Wiles, Keith wrote: >>>Hi Lazaros, >>> >>>Thanks for this patch. To me, this is a valuable enhancement. >>>Please find some comments inline. >>> >>>On 03/10/2016 03:44 PM, Lazaros Koromilas wrote: The mempool cache is only available to EAL threads as a per-lc

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-24 Thread Venkatesan, Venky
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Lazaros Koromilas > Sent: Thursday, March 24, 2016 7:36 AM > To: Wiles, Keith > Cc: Olivier Matz ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mempool: allow for user-owned mempool &g

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-21 Thread Wiles, Keith
>Hi Lazaros, > >Thanks for this patch. To me, this is a valuable enhancement. >Please find some comments inline. > >On 03/10/2016 03:44 PM, Lazaros Koromilas wrote: >> The mempool cache is only available to EAL threads as a per-lcore >> resource. Change this so that the user can create and provide

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-21 Thread Olivier Matz
Hi Lazaros, Thanks for this patch. To me, this is a valuable enhancement. Please find some comments inline. On 03/10/2016 03:44 PM, Lazaros Koromilas wrote: > The mempool cache is only available to EAL threads as a per-lcore > resource. Change this so that the user can create and provide their ow

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-21 Thread Ananyev, Konstantin
Hi lads, > > Hi Lazaros, > > Thanks for this patch. To me, this is a valuable enhancement. > Please find some comments inline. Yep, patch seems interesting. One question - what would be the usage model for get/put_with_cache functions for non-EAL threads? I meant for each non-EAL thread user

[dpdk-dev] [PATCH] mempool: allow for user-owned mempool caches

2016-03-10 Thread Lazaros Koromilas
The mempool cache is only available to EAL threads as a per-lcore resource. Change this so that the user can create and provide their own cache on mempool get and put operations. This works with non-EAL threads too. This commit introduces new API calls with the 'with_cache' suffix, while the curren