DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
@ 2018-01-02 13:08 Shreyansh Jain
  2018-01-11  8:56 ` Hemant Agrawal
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Shreyansh Jain @ 2018-01-02 13:08 UTC (permalink / raw)
  To: ferruh.yigit; +Cc: dev, Shreyansh Jain

FSLMC bus detects a multiple type of logical objects representing
components of the datapath.

Using the type of device, a newly introduced API
rte_fslmc_get_device_count can return the count of devices
scanned of that device type.

Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
---
:: This patch is based on *net-next* tree.

 drivers/bus/fslmc/fslmc_bus.c               | 12 ++++++++++++
 drivers/bus/fslmc/rte_bus_fslmc_version.map |  1 +
 drivers/bus/fslmc/rte_fslmc.h               | 18 +++++++++++++++---
 3 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/drivers/bus/fslmc/fslmc_bus.c b/drivers/bus/fslmc/fslmc_bus.c
index 63c333a59..39478f7f3 100644
--- a/drivers/bus/fslmc/fslmc_bus.c
+++ b/drivers/bus/fslmc/fslmc_bus.c
@@ -53,6 +53,14 @@
 struct rte_fslmc_bus rte_fslmc_bus;
 uint8_t dpaa2_virt_mode;
 
+uint32_t
+rte_fslmc_get_device_count(enum rte_dpaa2_dev_type device_type)
+{
+	if (device_type > DPAA2_DEVTYPE_MAX)
+		return 0;
+	return rte_fslmc_bus.device_count[device_type];
+}
+
 static void
 cleanup_fslmc_device_list(void)
 {
@@ -164,6 +172,9 @@ scan_one_fslmc_device(char *dev_name)
 	else
 		dev->dev_type = DPAA2_UNKNOWN;
 
+	/* Update the device found into the device_count table */
+	rte_fslmc_bus.device_count[dev->dev_type]++;
+
 	t_ptr = strtok(NULL, ".");
 	if (!t_ptr) {
 		FSLMC_BUS_LOG(ERR, "Incorrect device string observed (%s).",
@@ -408,6 +419,7 @@ struct rte_fslmc_bus rte_fslmc_bus = {
 	},
 	.device_list = TAILQ_HEAD_INITIALIZER(rte_fslmc_bus.device_list),
 	.driver_list = TAILQ_HEAD_INITIALIZER(rte_fslmc_bus.driver_list),
+	.device_count = {0},
 };
 
 RTE_REGISTER_BUS(fslmc, rte_fslmc_bus.bus);
diff --git a/drivers/bus/fslmc/rte_bus_fslmc_version.map b/drivers/bus/fslmc/rte_bus_fslmc_version.map
index f59fc671f..16b759d8b 100644
--- a/drivers/bus/fslmc/rte_bus_fslmc_version.map
+++ b/drivers/bus/fslmc/rte_bus_fslmc_version.map
@@ -97,5 +97,6 @@ DPDK_18.02 {
 	dpaa2_virt_mode;
 	qbman_fq_query_state;
 	qbman_fq_state_frame_count;
+	rte_fslmc_get_device_count;
 
 } DPDK_17.11;
diff --git a/drivers/bus/fslmc/rte_fslmc.h b/drivers/bus/fslmc/rte_fslmc.h
index fd52e2b84..e6314b5cb 100644
--- a/drivers/bus/fslmc/rte_fslmc.h
+++ b/drivers/bus/fslmc/rte_fslmc.h
@@ -88,7 +88,8 @@ enum rte_dpaa2_dev_type {
 	DPAA2_CI,	/**< DPCI type device */
 	DPAA2_MPORTAL,  /**< DPMCP type device */
 	/* Unknown device placeholder */
-	DPAA2_UNKNOWN
+	DPAA2_UNKNOWN,
+	DPAA2_DEVTYPE_MAX,
 };
 
 TAILQ_HEAD(rte_dpaa2_object_list, rte_dpaa2_object);
@@ -150,8 +151,8 @@ struct rte_fslmc_bus {
 				/**< FSLMC DPAA2 Device list */
 	struct rte_fslmc_driver_list driver_list;
 				/**< FSLMC DPAA2 Driver list */
-	int device_count;
-				/**< Optional: Count of devices on bus */
+	int device_count[DPAA2_DEVTYPE_MAX];
+				/**< Count of all devices scanned */
 };
 
 /**
@@ -191,6 +192,17 @@ RTE_PMD_EXPORT_NAME(nm, __COUNTER__)
  */
 void rte_fslmc_object_register(struct rte_dpaa2_object *object);
 
+/**
+ * Count of a particular type of DPAA2 device scanned on the bus.
+ *
+ * @param dev_type
+ *   Type of device as rte_dpaa2_dev_type enumerator
+ * @return
+ *   >=0 for count; 0 indicates either no device of the said type scanned or
+ *   invalid device type.
+ */
+uint32_t rte_fslmc_get_device_count(enum rte_dpaa2_dev_type device_type);
+
 /** Helper for DPAA2 object registration */
 #define RTE_PMD_REGISTER_DPAA2_OBJECT(nm, dpaa2_obj) \
 RTE_INIT(dpaa2objinitfn_ ##nm); \
-- 
2.14.1

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

* Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
  2018-01-02 13:08 [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count Shreyansh Jain
@ 2018-01-11  8:56 ` Hemant Agrawal
  2018-01-12 11:45 ` Thomas Monjalon
  2018-01-12 18:01 ` Ferruh Yigit
  2 siblings, 0 replies; 6+ messages in thread
From: Hemant Agrawal @ 2018-01-11  8:56 UTC (permalink / raw)
  To: Shreyansh Jain, ferruh.yigit; +Cc: dev

On 1/2/2018 6:38 PM, Shreyansh Jain wrote:
> FSLMC bus detects a multiple type of logical objects representing
> components of the datapath.
>
> Using the type of device, a newly introduced API
> rte_fslmc_get_device_count can return the count of devices
> scanned of that device type.
>
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> ---
> :: This patch is based on *net-next* tree.

<snip>..
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

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

* Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
  2018-01-02 13:08 [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count Shreyansh Jain
  2018-01-11  8:56 ` Hemant Agrawal
@ 2018-01-12 11:45 ` Thomas Monjalon
  2018-01-12 12:59   ` Shreyansh Jain
  2018-01-12 18:01 ` Ferruh Yigit
  2 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2018-01-12 11:45 UTC (permalink / raw)
  To: Shreyansh Jain; +Cc: dev, ferruh.yigit

02/01/2018 14:08, Shreyansh Jain:
> FSLMC bus detects a multiple type of logical objects representing
> components of the datapath.
> 
> Using the type of device, a newly introduced API
> rte_fslmc_get_device_count can return the count of devices
> scanned of that device type.
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> ---
> :: This patch is based on *net-next* tree.

Why is it based on next-net tree?

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

* Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
  2018-01-12 11:45 ` Thomas Monjalon
@ 2018-01-12 12:59   ` Shreyansh Jain
  2018-01-12 14:09     ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Shreyansh Jain @ 2018-01-12 12:59 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, ferruh.yigit

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Friday, January 12, 2018 5:15 PM
> To: Shreyansh Jain <shreyansh.jain@nxp.com>
> Cc: dev@dpdk.org; ferruh.yigit@intel.com
> Subject: Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device
> count
> 
> 02/01/2018 14:08, Shreyansh Jain:
> > FSLMC bus detects a multiple type of logical objects representing
> > components of the datapath.
> >
> > Using the type of device, a newly introduced API
> > rte_fslmc_get_device_count can return the count of devices
> > scanned of that device type.
> >
> > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > ---
> > :: This patch is based on *net-next* tree.
> 
> Why is it based on next-net tree?

Because there is a dependency of a lot of patches for DPAA2 which were only on net-next when I sent this patch out. (dependency for lines, not for logic). Else, those would have conflicted when you would have merged net-next into master.

If those have been merged into master, this should be applicable on master as well.

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

* Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
  2018-01-12 12:59   ` Shreyansh Jain
@ 2018-01-12 14:09     ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2018-01-12 14:09 UTC (permalink / raw)
  To: Shreyansh Jain; +Cc: dev, ferruh.yigit

12/01/2018 13:59, Shreyansh Jain:
> > -----Original Message-----
> > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > Sent: Friday, January 12, 2018 5:15 PM
> > To: Shreyansh Jain <shreyansh.jain@nxp.com>
> > Cc: dev@dpdk.org; ferruh.yigit@intel.com
> > Subject: Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device
> > count
> > 
> > 02/01/2018 14:08, Shreyansh Jain:
> > > FSLMC bus detects a multiple type of logical objects representing
> > > components of the datapath.
> > >
> > > Using the type of device, a newly introduced API
> > > rte_fslmc_get_device_count can return the count of devices
> > > scanned of that device type.
> > >
> > > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > > ---
> > > :: This patch is based on *net-next* tree.
> > 
> > Why is it based on next-net tree?
> 
> Because there is a dependency of a lot of patches for DPAA2 which were only on net-next when I sent this patch out. (dependency for lines, not for logic). Else, those would have conflicted when you would have merged net-next into master.
> 
> If those have been merged into master, this should be applicable on master as well.

I have delegated the patch to Ferruh.
Thanks

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

* Re: [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count
  2018-01-02 13:08 [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count Shreyansh Jain
  2018-01-11  8:56 ` Hemant Agrawal
  2018-01-12 11:45 ` Thomas Monjalon
@ 2018-01-12 18:01 ` Ferruh Yigit
  2 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2018-01-12 18:01 UTC (permalink / raw)
  To: Shreyansh Jain; +Cc: dev

On 1/2/2018 1:08 PM, Shreyansh Jain wrote:
> FSLMC bus detects a multiple type of logical objects representing
> components of the datapath.
> 
> Using the type of device, a newly introduced API
> rte_fslmc_get_device_count can return the count of devices
> scanned of that device type.
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>

Applied to dpdk-next-net/master, thanks.

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

end of thread, other threads:[~2018-01-12 18:01 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-02 13:08 [dpdk-dev] [PATCH] bus/fslmc: add support for scanned device count Shreyansh Jain
2018-01-11  8:56 ` Hemant Agrawal
2018-01-12 11:45 ` Thomas Monjalon
2018-01-12 12:59   ` Shreyansh Jain
2018-01-12 14:09     ` Thomas Monjalon
2018-01-12 18:01 ` Ferruh Yigit

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).