[dpdk-dev] event/dpaa2: return correct number of event ports

Message ID 1514997859-3902-1-git-send-email-nipun.gupta@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: Jerin Jacob
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Nipun Gupta Jan. 3, 2018, 4:44 p.m. UTC
  In the info get API we were returning number of event
ports as MAX number of LCORE's. After this change actual
number of event ports (i.e. number of DPIO's) is provided
in the info get API.

Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
---
This patch is dependent on https://dpdk.org/dev/patchwork/patch/32834/
as it uses the API exposed in the mentioned patch

 drivers/event/dpaa2/dpaa2_eventdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Hemant Agrawal Jan. 12, 2018, 2:42 p.m. UTC | #1
On 1/3/2018 10:14 PM, Nipun Gupta wrote:
> In the info get API we were returning number of event
> ports as MAX number of LCORE's. After this change actual
> number of event ports (i.e. number of DPIO's) is provided
> in the info get API.
>
> Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> ---
> This patch is dependent on https://dpdk.org/dev/patchwork/patch/32834/
> as it uses the API exposed in the mentioned patch
>
>  drivers/event/dpaa2/dpaa2_eventdev.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c
> index 13e7122..5cbe1cf 100644
> --- a/drivers/event/dpaa2/dpaa2_eventdev.c
> +++ b/drivers/event/dpaa2/dpaa2_eventdev.c
> @@ -326,7 +326,7 @@ static void dpaa2_eventdev_process_atomic(struct qbman_swp *swp,
>  		DPAA2_EVENT_MAX_QUEUE_PRIORITY_LEVELS;
>  	dev_info->max_event_priority_levels =
>  		DPAA2_EVENT_MAX_EVENT_PRIORITY_LEVELS;
> -	dev_info->max_event_ports = RTE_MAX_LCORE;
> +	dev_info->max_event_ports = rte_fslmc_get_device_count(DPAA2_IO);
>  	dev_info->max_event_port_dequeue_depth =
>  		DPAA2_EVENT_MAX_PORT_DEQUEUE_DEPTH;
>  	dev_info->max_event_port_enqueue_depth =
>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
  
Jerin Jacob Jan. 18, 2018, 7:04 a.m. UTC | #2
-----Original Message-----
> Date: Fri, 12 Jan 2018 20:12:27 +0530
> From: Hemant Agrawal <hemant.agrawal@nxp.com>
> To: Nipun Gupta <nipun.gupta@nxp.com>, dev@dpdk.org
> CC: jerin.jacob@caviumnetworks.com
> Subject: Re: [PATCH] event/dpaa2: return correct number of event ports
> User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101
>  Thunderbird/45.8.0
> 
> On 1/3/2018 10:14 PM, Nipun Gupta wrote:
> > In the info get API we were returning number of event
> > ports as MAX number of LCORE's. After this change actual
> > number of event ports (i.e. number of DPIO's) is provided
> > in the info get API.
> > 
> > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> > ---
> > This patch is dependent on https://dpdk.org/dev/patchwork/patch/32834/
> > as it uses the API exposed in the mentioned patch
> > 
> >  drivers/event/dpaa2/dpaa2_eventdev.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c
> > index 13e7122..5cbe1cf 100644
> > --- a/drivers/event/dpaa2/dpaa2_eventdev.c
> > +++ b/drivers/event/dpaa2/dpaa2_eventdev.c
> > @@ -326,7 +326,7 @@ static void dpaa2_eventdev_process_atomic(struct qbman_swp *swp,
> >  		DPAA2_EVENT_MAX_QUEUE_PRIORITY_LEVELS;
> >  	dev_info->max_event_priority_levels =
> >  		DPAA2_EVENT_MAX_EVENT_PRIORITY_LEVELS;
> > -	dev_info->max_event_ports = RTE_MAX_LCORE;
> > +	dev_info->max_event_ports = rte_fslmc_get_device_count(DPAA2_IO);
> >  	dev_info->max_event_port_dequeue_depth =
> >  		DPAA2_EVENT_MAX_PORT_DEQUEUE_DEPTH;
> >  	dev_info->max_event_port_enqueue_depth =
> > 
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Applied to dpdk-next-eventdev/master. Thanks.
  

Patch

diff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c
index 13e7122..5cbe1cf 100644
--- a/drivers/event/dpaa2/dpaa2_eventdev.c
+++ b/drivers/event/dpaa2/dpaa2_eventdev.c
@@ -326,7 +326,7 @@  static void dpaa2_eventdev_process_atomic(struct qbman_swp *swp,
 		DPAA2_EVENT_MAX_QUEUE_PRIORITY_LEVELS;
 	dev_info->max_event_priority_levels =
 		DPAA2_EVENT_MAX_EVENT_PRIORITY_LEVELS;
-	dev_info->max_event_ports = RTE_MAX_LCORE;
+	dev_info->max_event_ports = rte_fslmc_get_device_count(DPAA2_IO);
 	dev_info->max_event_port_dequeue_depth =
 		DPAA2_EVENT_MAX_PORT_DEQUEUE_DEPTH;
 	dev_info->max_event_port_enqueue_depth =