DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
@ 2020-10-26 16:01 Timothy McDaniel
  2020-10-28  9:35 ` Jerin Jacob
  2020-10-30 20:03 ` Eads, Gage
  0 siblings, 2 replies; 5+ messages in thread
From: Timothy McDaniel @ 2020-10-26 16:01 UTC (permalink / raw)
  To: Bruce Richardson
  Cc: dev, erik.g.carrillo, gage.eads, harry.van.haaren, jerinj

DLB supports a total of 256 queues, 128 load balanced queues
and 128 directed queues.

Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
---
 config/rte_config.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/rte_config.h b/config/rte_config.h
index 8aa46a1..b78c6aa 100644
--- a/config/rte_config.h
+++ b/config/rte_config.h
@@ -71,7 +71,7 @@
 
 /* eventdev defines */
 #define RTE_EVENT_MAX_DEVS 16
-#define RTE_EVENT_MAX_QUEUES_PER_DEV 64
+#define RTE_EVENT_MAX_QUEUES_PER_DEV 255
 #define RTE_EVENT_TIMER_ADAPTER_NUM_MAX 32
 #define RTE_EVENT_ETH_INTR_RING_SIZE 1024
 #define RTE_EVENT_CRYPTO_ADAPTER_MAX_INSTANCE 32
-- 
2.6.4


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
  2020-10-26 16:01 [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255 Timothy McDaniel
@ 2020-10-28  9:35 ` Jerin Jacob
  2020-10-28 14:52   ` McDaniel, Timothy
  2020-10-30 20:03 ` Eads, Gage
  1 sibling, 1 reply; 5+ messages in thread
From: Jerin Jacob @ 2020-10-28  9:35 UTC (permalink / raw)
  To: Timothy McDaniel
  Cc: Bruce Richardson, dpdk-dev, Erik Gabriel Carrillo, Gage Eads,
	Van Haaren, Harry, Jerin Jacob

On Mon, Oct 26, 2020 at 9:29 PM Timothy McDaniel
<timothy.mcdaniel@intel.com> wrote:
>
> DLB supports a total of 256 queues, 128 load balanced queues
> and 128 directed queues.
>
> Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
> ---
>  config/rte_config.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/config/rte_config.h b/config/rte_config.h
> index 8aa46a1..b78c6aa 100644
> --- a/config/rte_config.h
> +++ b/config/rte_config.h
> @@ -71,7 +71,7 @@
>
>  /* eventdev defines */
>  #define RTE_EVENT_MAX_DEVS 16
> -#define RTE_EVENT_MAX_QUEUES_PER_DEV 64
> +#define RTE_EVENT_MAX_QUEUES_PER_DEV 255

If it is 0..255 then this value should be 256. Isn't it?


>  #define RTE_EVENT_TIMER_ADAPTER_NUM_MAX 32
>  #define RTE_EVENT_ETH_INTR_RING_SIZE 1024
>  #define RTE_EVENT_CRYPTO_ADAPTER_MAX_INSTANCE 32
> --
> 2.6.4
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
  2020-10-28  9:35 ` Jerin Jacob
@ 2020-10-28 14:52   ` McDaniel, Timothy
  0 siblings, 0 replies; 5+ messages in thread
From: McDaniel, Timothy @ 2020-10-28 14:52 UTC (permalink / raw)
  To: Jerin Jacob
  Cc: Richardson, Bruce, dpdk-dev, Carrillo, Erik G, Eads, Gage,
	Van Haaren, Harry, Jerin Jacob



> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Wednesday, October 28, 2020 4:36 AM
> To: McDaniel, Timothy <timothy.mcdaniel@intel.com>
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; dpdk-dev
> <dev@dpdk.org>; Carrillo, Erik G <erik.g.carrillo@intel.com>; Eads, Gage
> <gage.eads@intel.com>; Van Haaren, Harry <harry.van.haaren@intel.com>;
> Jerin Jacob <jerinj@marvell.com>
> Subject: Re: [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV
> to 255
> 
> On Mon, Oct 26, 2020 at 9:29 PM Timothy McDaniel
> <timothy.mcdaniel@intel.com> wrote:
> >
> > DLB supports a total of 256 queues, 128 load balanced queues
> > and 128 directed queues.
> >
> > Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
> > ---
> >  config/rte_config.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/config/rte_config.h b/config/rte_config.h
> > index 8aa46a1..b78c6aa 100644
> > --- a/config/rte_config.h
> > +++ b/config/rte_config.h
> > @@ -71,7 +71,7 @@
> >
> >  /* eventdev defines */
> >  #define RTE_EVENT_MAX_DEVS 16
> > -#define RTE_EVENT_MAX_QUEUES_PER_DEV 64
> > +#define RTE_EVENT_MAX_QUEUES_PER_DEV 255
> 
> If it is 0..255 then this value should be 256. Isn't it?
> 

.max_event_queues is a uint8_t, so we could not go to 256 without changing the
data structure.

> 
> >  #define RTE_EVENT_TIMER_ADAPTER_NUM_MAX 32
> >  #define RTE_EVENT_ETH_INTR_RING_SIZE 1024
> >  #define RTE_EVENT_CRYPTO_ADAPTER_MAX_INSTANCE 32
> > --
> > 2.6.4
> >

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
  2020-10-26 16:01 [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255 Timothy McDaniel
  2020-10-28  9:35 ` Jerin Jacob
@ 2020-10-30 20:03 ` Eads, Gage
  2020-10-31 10:58   ` Jerin Jacob
  1 sibling, 1 reply; 5+ messages in thread
From: Eads, Gage @ 2020-10-30 20:03 UTC (permalink / raw)
  To: McDaniel, Timothy, Richardson, Bruce
  Cc: dev, Carrillo, Erik G, Van Haaren, Harry, jerinj



> -----Original Message-----
> From: McDaniel, Timothy <timothy.mcdaniel@intel.com>
> Sent: Monday, October 26, 2020 11:01 AM
> To: Richardson, Bruce <bruce.richardson@intel.com>
> Cc: dev@dpdk.org; Carrillo, Erik G <erik.g.carrillo@intel.com>; Eads, Gage
> <gage.eads@intel.com>; Van Haaren, Harry <harry.van.haaren@intel.com>;
> jerinj@marvell.com
> Subject: [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
> 
> DLB supports a total of 256 queues, 128 load balanced queues
> and 128 directed queues.
> 
> Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>

Perhaps you can increase the queues-related fields to uint16_t in the future
to allow for the full 256, but in the meantime:

Reviewed-by: Gage Eads <gage.eads@intel.com>

Thanks,
Gage

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
  2020-10-30 20:03 ` Eads, Gage
@ 2020-10-31 10:58   ` Jerin Jacob
  0 siblings, 0 replies; 5+ messages in thread
From: Jerin Jacob @ 2020-10-31 10:58 UTC (permalink / raw)
  To: Eads, Gage
  Cc: McDaniel, Timothy, Richardson, Bruce, dev, Carrillo, Erik G,
	Van Haaren, Harry, jerinj

On Sat, Oct 31, 2020 at 1:34 AM Eads, Gage <gage.eads@intel.com> wrote:
>
>
>
> > -----Original Message-----
> > From: McDaniel, Timothy <timothy.mcdaniel@intel.com>
> > Sent: Monday, October 26, 2020 11:01 AM
> > To: Richardson, Bruce <bruce.richardson@intel.com>
> > Cc: dev@dpdk.org; Carrillo, Erik G <erik.g.carrillo@intel.com>; Eads, Gage
> > <gage.eads@intel.com>; Van Haaren, Harry <harry.van.haaren@intel.com>;
> > jerinj@marvell.com
> > Subject: [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255
> >
> > DLB supports a total of 256 queues, 128 load balanced queues
> > and 128 directed queues.
> >
> > Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
>
> Perhaps you can increase the queues-related fields to uint16_t in the future
> to allow for the full 256, but in the meantime:
>
> Reviewed-by: Gage Eads <gage.eads@intel.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>

Applied to dpdk-next-eventdev/for-main with following git commit change, Thanks.

commit e79fdca1baaff30cbdd423b1fec82087cafc0b83 (HEAD -> for-main,
origin/for-main, origin/HEAD)
Author: Timothy McDaniel <timothy.mcdaniel@intel.com>
Date:   Mon Oct 26 11:01:08 2020 -0500

    eventdev: increase max queues configuration

    DLB supports a total of 256 queues, 128 load balanced queues
    and 128 directed queues. Increase RTE_EVENT_MAX_QUEUES_PER_DEV
    to max possible uint_8_t max value.

    Signed-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
    Reviewed-by: Gage Eads <gage.eads@intel.com>
    Acked-by: Jerin Jacob <jerinj@marvell.com>


>
> Thanks,
> Gage

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-10-31 10:58 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-26 16:01 [dpdk-dev] [PATCH 1/1] eventdev: increase MAX QUEUES PER DEV to 255 Timothy McDaniel
2020-10-28  9:35 ` Jerin Jacob
2020-10-28 14:52   ` McDaniel, Timothy
2020-10-30 20:03 ` Eads, Gage
2020-10-31 10:58   ` Jerin Jacob

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).