[dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-17 Thread Ajit Khaparde
From: Lance Richardson This commit enables the creation of a dedicated completion ring for asynchronous event handling instead of handling these events on a receive completion ring. For the stingray platform and other platforms needing tighter control of resource utilization, we retain the abili

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-22 Thread Ferruh Yigit
On 7/18/2019 4:36 AM, Ajit Khaparde wrote: > From: Lance Richardson > > This commit enables the creation of a dedicated completion > ring for asynchronous event handling instead of handling these > events on a receive completion ring. > > For the stingray platform and other platforms needing tig

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-22 Thread Thomas Monjalon
22/07/2019 16:57, Ferruh Yigit: > On 7/18/2019 4:36 AM, Ajit Khaparde wrote: > > From: Lance Richardson > > --- a/config/common_base > > +++ b/config/common_base > > @@ -212,6 +212,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_PERIODIC=n > > # Compile burst-oriented Broadcom BNXT PMD driver > > # > > CONF

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-22 Thread Lance Richardson
On Mon, Jul 22, 2019 at 11:06 AM Thomas Monjalon wrote: > > 22/07/2019 16:57, Ferruh Yigit: > > On 7/18/2019 4:36 AM, Ajit Khaparde wrote: > > > From: Lance Richardson > > > --- a/config/common_base > > > +++ b/config/common_base > > > @@ -212,6 +212,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_PERIODIC=n

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-22 Thread Ferruh Yigit
On 7/22/2019 6:57 PM, Lance Richardson wrote: > On Mon, Jul 22, 2019 at 11:06 AM Thomas Monjalon wrote: >> >> 22/07/2019 16:57, Ferruh Yigit: >>> On 7/18/2019 4:36 AM, Ajit Khaparde wrote: From: Lance Richardson --- a/config/common_base +++ b/config/common_base @@ -212,6 +212,

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-23 Thread Thomas Monjalon
22/07/2019 20:34, Ferruh Yigit: > On 7/22/2019 6:57 PM, Lance Richardson wrote: > > On Mon, Jul 22, 2019 at 11:06 AM Thomas Monjalon > > wrote: > >> 22/07/2019 16:57, Ferruh Yigit: > >>> On 7/18/2019 4:36 AM, Ajit Khaparde wrote: > From: Lance Richardson > --- a/config/common_base > >>

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-23 Thread Lance Richardson
Yes, the platform can be detected at runtime from the PCI device ID. This would be a better approach than adding a configuration parameter, I think. I'll take a look. Thanks, Lance On Tue, Jul 23, 2019 at 4:04 AM Thomas Monjalon wrote: > > 22/07/2019 20:34, Ferruh Yigit: > > On 7/22/2019 6:5

Re: [dpdk-dev] [PATCH 09/22] net/bnxt: use dedicated cpr for async events

2019-07-23 Thread Lance Richardson
On Mon, Jul 22, 2019 at 2:34 PM Ferruh Yigit wrote: > I dropped this patch from next-net since discussion is going on, I did my best > to resolve the conflicts but please confirm the final result in next-net. > I did some basic testing of the current head of dpdk-next-net and diffed against the