Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-30 Thread Olivier Matz
On Wed, 29 Mar 2017 18:25:31 +0530, Shreyansh Jain wrote: > Hello Olivier, > > On Wednesday 29 March 2017 01:48 PM, Olivier Matz wrote: > > On Tue, 28 Mar 2017 17:12:47 +0530, Shreyansh Jain > > wrote: > >> Hello Olivier, > >> > >> On Friday 24 March 2017 09:52 PM, Olivier Matz wrote: > >> [

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-29 Thread Shreyansh Jain
Hello Olivier, On Wednesday 29 March 2017 01:48 PM, Olivier Matz wrote: On Tue, 28 Mar 2017 17:12:47 +0530, Shreyansh Jain wrote: Hello Olivier, On Friday 24 March 2017 09:52 PM, Olivier Matz wrote: [..] I tried to pass the mempool autotest, and it issues a segfault. I think the libraries

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-29 Thread Olivier Matz
On Tue, 28 Mar 2017 17:12:47 +0530, Shreyansh Jain wrote: > Hello Olivier, > > On Friday 24 March 2017 09:52 PM, Olivier Matz wrote: > [..] > > > I tried to pass the mempool autotest, and it issues a segfault. > > I think the libraries are missing in rte.app.mk, so no handler is > > registered.

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-28 Thread Shreyansh Jain
Hello Olivier, On Friday 24 March 2017 09:52 PM, Olivier Matz wrote: [..] I tried to pass the mempool autotest, and it issues a segfault. I think the libraries are missing in rte.app.mk, so no handler is registered. I have been trying to simulate the segfault that you are referring to above.

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-27 Thread Olivier Matz
Hi Shreyansh, [...] > > Also, the MAINTAINERS file should be updated. > > Yes, that is something I thought of updating but left it out before > sending the patch. > One confirmation: I assume that maintainers need to be added with > "drivers/mempool/ring" and "drivers/mempool/stack" folders.

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-26 Thread Shreyansh Jain
Hello Olivier, On Friday 24 March 2017 09:52 PM, Olivier Matz wrote: Hi Shreyansh, On Mon, 20 Mar 2017 15:33:09 +0530, Shreyansh Jain wrote: CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. Stack mempool handler moved from lib/librte_mempool into drivers/mempool. With this patch

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-24 Thread Olivier Matz
Hi Shreyansh, On Mon, 20 Mar 2017 15:33:09 +0530, Shreyansh Jain wrote: > CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. > Stack mempool handler moved from lib/librte_mempool into drivers/mempool. > > With this patch, the Stack mempool handler registration is optional and > toggle

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-21 Thread Wiles, Keith
> On Mar 21, 2017, at 1:25 AM, Shreyansh Jain wrote: > > Hello Keith, > > On Tuesday 21 March 2017 11:32 AM, Wiles, Keith wrote: >> >>> On Mar 20, 2017, at 11:55 PM, Shreyansh Jain wrote: >>> >>> Hello David, >>> >>> On Monday 20 March 2017 08:20 PM, Hunt, David wrote: On 20/3/20

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Shreyansh Jain
On Tuesday 21 March 2017 11:55 AM, Shreyansh Jain wrote: Hello Keith, On Tuesday 21 March 2017 11:32 AM, Wiles, Keith wrote: On Mar 20, 2017, at 11:55 PM, Shreyansh Jain wrote: Hello David, On Monday 20 March 2017 08:20 PM, Hunt, David wrote: On 20/3/2017 10:03 AM, Shreyansh Jain wrote:

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Shreyansh Jain
Hello Keith, On Tuesday 21 March 2017 11:32 AM, Wiles, Keith wrote: On Mar 20, 2017, at 11:55 PM, Shreyansh Jain wrote: Hello David, On Monday 20 March 2017 08:20 PM, Hunt, David wrote: On 20/3/2017 10:03 AM, Shreyansh Jain wrote: CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_ba

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Wiles, Keith
> On Mar 20, 2017, at 11:55 PM, Shreyansh Jain wrote: > > Hello David, > > On Monday 20 March 2017 08:20 PM, Hunt, David wrote: >> >> On 20/3/2017 10:03 AM, Shreyansh Jain wrote: >>> CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. >>> Stack mempool handler moved from lib/librte_me

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Shreyansh Jain
Hello David, On Monday 20 March 2017 08:20 PM, Hunt, David wrote: On 20/3/2017 10:03 AM, Shreyansh Jain wrote: CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. Stack mempool handler moved from lib/librte_mempool into drivers/mempool. <...> -} - -static struct rte_mempool_ops o

Re: [dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Hunt, David
On 20/3/2017 10:03 AM, Shreyansh Jain wrote: CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. Stack mempool handler moved from lib/librte_mempool into drivers/mempool. With this patch, the Stack mempool handler registration is optional and toggled via the configuration file. In case

[dpdk-dev] [PATCH 1/2] drivers/mempool: add stack mempool handler as driver

2017-03-20 Thread Shreyansh Jain
CONFIG_RTE_DRIVER_MEMPOOL_STACK option added to common_base. Stack mempool handler moved from lib/librte_mempool into drivers/mempool. With this patch, the Stack mempool handler registration is optional and toggled via the configuration file. In case disabled (N), it would imply request for creati