Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2018-01-08 Thread Pavan Nikhilesh
Hi Gage, On Mon, Jan 08, 2018 at 06:36:36PM +, Eads, Gage wrote: > Hi Pavan, > > Thanks for the report and the GDB output. We've reproduced this and traced it > down to how the PMD (mis)handles the re-configuration case. When the SW PMD > is reconfigured, it reallocates the IQ chunks and rei

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2018-01-08 Thread Eads, Gage
Hi Pavan, Thanks for the report and the GDB output. We've reproduced this and traced it down to how the PMD (mis)handles the re-configuration case. When the SW PMD is reconfigured, it reallocates the IQ chunks and reinitializes the chunk freelist, but it doesn't delete the stale pointers in sw-

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2018-01-08 Thread Pavan Nikhilesh
On Mon, Jan 08, 2018 at 03:50:24PM +, Van Haaren, Harry wrote: > > From: Pavan Nikhilesh [mailto:pbhagavat...@caviumnetworks.com] > > Sent: Monday, January 8, 2018 3:32 PM > > To: Eads, Gage ; Van Haaren, Harry > > ; jerin.ja...@caviumnetworks.com; > > santosh.shu...@caviumnetworks.com > > Cc:

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2018-01-08 Thread Van Haaren, Harry
> From: Pavan Nikhilesh [mailto:pbhagavat...@caviumnetworks.com] > Sent: Monday, January 8, 2018 3:32 PM > To: Eads, Gage ; Van Haaren, Harry > ; jerin.ja...@caviumnetworks.com; > santosh.shu...@caviumnetworks.com > Cc: dev@dpdk.org > Subject: Re: [PATCH 2/2] event/sw: use dynamically-sized IQs >

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2018-01-08 Thread Pavan Nikhilesh
On Wed, Nov 29, 2017 at 09:08:34PM -0600, Gage Eads wrote: > This commit introduces dynamically-sized IQs, by switching the underlying > data structure from a fixed-size ring to a linked list of queue 'chunks.' > This has a number of benefits: > - Certain corner cases were observed in which all of

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2017-12-09 Thread Jerin Jacob
-Original Message- > Date: Thu, 7 Dec 2017 17:15:35 + > From: "Van Haaren, Harry" > To: "Eads, Gage" , "dev@dpdk.org" > CC: "jerin.ja...@caviumnetworks.com" , > "Richardson, Bruce" , "hemant.agra...@nxp.com" > , "nipun.gu...@nxp.com" , > "santosh.shu...@caviumnetworks.com" , > "pb

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2017-12-07 Thread Van Haaren, Harry
> From: Eads, Gage > Sent: Thursday, November 30, 2017 3:09 AM > To: dev@dpdk.org > Cc: jerin.ja...@caviumnetworks.com; Van Haaren, Harry > ; Richardson, Bruce > ; hemant.agra...@nxp.com; nipun.gu...@nxp.com; > santosh.shu...@caviumnetworks.com; pbhagavat...@caviumnetworks.com > Subject: [PATCH 2/2

[dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2017-11-29 Thread Gage Eads
This commit introduces dynamically-sized IQs, by switching the underlying data structure from a fixed-size ring to a linked list of queue 'chunks.' This has a number of benefits: - Certain corner cases were observed in which all of a pipeline's flows could be pinned to one port for extended perio