RE: [PATCH 0/1] mempool: implement index-based per core cache

2023-07-31 Thread Dharmik Jayesh Thakkar
> ; olivier.m...@6wind.com; > andrew.rybche...@oktetlabs.ru > Subject: RE: [PATCH 0/1] mempool: implement index-based per core cache > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > Sent: Monday, 31 July 2023 14.24 > > > > The v2 was not sent, and Steph

RE: [PATCH 0/1] mempool: implement index-based per core cache

2023-07-31 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, 31 July 2023 14.24 > > The v2 was not sent, and Stephen dropped the patch from patchwork. > > Do we abandon this feature? +1, because I think that the zero-copy mempool cache access functions make this patch irrelevant. > Sho

Re: [PATCH 0/1] mempool: implement index-based per core cache

2023-07-31 Thread Thomas Monjalon
The v2 was not sent, and Stephen dropped the patch from patchwork. Do we abandon this feature? Should I remove it from the roadmap? 06/07/2023 19:43, Stephen Hemminger: > On Thu, 13 Jan 2022 05:31:18 + > Dharmik Thakkar wrote: > > > Hi, > > > > Thank you for your valuable review comments

Re: [PATCH 0/1] mempool: implement index-based per core cache

2023-07-06 Thread Stephen Hemminger
On Thu, 13 Jan 2022 05:31:18 + Dharmik Thakkar wrote: > Hi, > > Thank you for your valuable review comments and suggestions! > > I will be sending out a v2 in which I have increased the size of the mempool > to 32GB by using division by sizeof(uintptr_t). > However, I am seeing ~5% perform

Re: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-12 Thread Dharmik Thakkar
Hi, Thank you for your valuable review comments and suggestions! I will be sending out a v2 in which I have increased the size of the mempool to 32GB by using division by sizeof(uintptr_t). However, I am seeing ~5% performance degradation with mempool_perf_autotest (for bulk size of 32) with th

Re: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-09 Thread Jerin Jacob
On Sat, Jan 8, 2022 at 3:07 PM Morten Brørup wrote: > > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > > Sent: Friday, 7 January 2022 14.51 > > > > On Fri, Jan 07, 2022 at 12:29:23PM +0100, Morten Brørup wrote: > > > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > >

RE: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-08 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Friday, 7 January 2022 14.51 > > On Fri, Jan 07, 2022 at 12:29:23PM +0100, Morten Brørup wrote: > > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > > > Sent: Friday, 7 January 2022 12.16 > > > > > > On Sat, Dec 25,

Re: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-07 Thread Bruce Richardson
On Fri, Jan 07, 2022 at 12:29:23PM +0100, Morten Brørup wrote: > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > > Sent: Friday, 7 January 2022 12.16 > > > > On Sat, Dec 25, 2021 at 01:16:03AM +0100, Morten Brørup wrote: > > > > From: Dharmik Thakkar [mailto:dharmik.thak...@arm.com]

RE: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-07 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Friday, 7 January 2022 12.16 > > On Sat, Dec 25, 2021 at 01:16:03AM +0100, Morten Brørup wrote: > > > From: Dharmik Thakkar [mailto:dharmik.thak...@arm.com] Sent: > Friday, 24 > > > December 2021 23.59 > > > > > > Current mempool

Re: [PATCH 0/1] mempool: implement index-based per core cache

2022-01-07 Thread Bruce Richardson
On Sat, Dec 25, 2021 at 01:16:03AM +0100, Morten Brørup wrote: > > From: Dharmik Thakkar [mailto:dharmik.thak...@arm.com] Sent: Friday, 24 > > December 2021 23.59 > > > > Current mempool per core cache implementation stores pointers to mbufs > > On 64b architectures, each pointer consumes 8B This

RE: [PATCH 0/1] mempool: implement index-based per core cache

2021-12-24 Thread Morten Brørup
> From: Dharmik Thakkar [mailto:dharmik.thak...@arm.com] > Sent: Friday, 24 December 2021 23.59 > > Current mempool per core cache implementation stores pointers to mbufs > On 64b architectures, each pointer consumes 8B > This patch replaces it with index-based implementation, > where in each buff