[PATCH v3 5/8] net/nfb: init only MACs associated with device

[spinler@cesnet.cz]

From: Martin Spinler <spinler@cesnet.cz>

Use information obtained from the new manner of creating NFB ethdevs
and initialize only RxMACs / TxMACs associated with actual ethdev.

This enables separated management and configuration of interface ports.

Signed-off-by: Martin Spinler <spinler@cesnet.cz>
---
 drivers/net/nfb/nfb.h        |   8 +-
 drivers/net/nfb/nfb_ethdev.c | 141 +++++++++++++++++++----------------
 2 files changed, 80 insertions(+), 69 deletions(-)

diff --git a/drivers/net/nfb/nfb.h b/drivers/net/nfb/nfb.h
index 605ea8e875..cfbc50251f 100644
--- a/drivers/net/nfb/nfb.h
+++ b/drivers/net/nfb/nfb.h
@@ -36,10 +36,6 @@ extern int nfb_logtype;
 #define RTE_ETH_NDP_MAX_RX_QUEUES 32
 #define RTE_ETH_NDP_MAX_TX_QUEUES 32
 
-/* Max index of rx/tx dmas */
-#define RTE_MAX_NC_RXMAC 256
-#define RTE_MAX_NC_TXMAC 256
-
 #define RTE_NFB_DRIVER_NAME net_nfb
 
 /* Device arguments */
@@ -56,8 +52,8 @@ extern int nfb_logtype;
 struct pmd_internals {
 	uint16_t         max_rxmac;
 	uint16_t         max_txmac;
-	struct nc_rxmac *rxmac[RTE_MAX_NC_RXMAC];
-	struct nc_txmac *txmac[RTE_MAX_NC_TXMAC];
+	struct nc_rxmac **rxmac;
+	struct nc_txmac **txmac;
 	struct nfb_device *nfb;
 
 	TAILQ_ENTRY(pmd_internals) eth_dev_list;
diff --git a/drivers/net/nfb/nfb_ethdev.c b/drivers/net/nfb/nfb_ethdev.c
index f7c50ea3c4..59e5a51b58 100644
--- a/drivers/net/nfb/nfb_ethdev.c
+++ b/drivers/net/nfb/nfb_ethdev.c
@@ -44,81 +44,94 @@ static const struct rte_ether_addr eth_addr = {
 };
 
 /**
- * Open all RX DMA queues
+ * Open RX MAC components associated with current ifc
  *
- * @param dev
- *   Pointer to nfb device.
- * @param[out] rxmac
- *   Pointer to output array of nc_rxmac
- * @param[out] max_rxmac
- *   Pointer to output max index of rxmac
+ * @param priv
+ *   Pointer to driver private structure
+ * @param params
+ *   Pointer to init parameters structure
  */
-static void
-nfb_nc_rxmac_init(struct nfb_device *nfb,
-	struct nc_rxmac *rxmac[RTE_MAX_NC_RXMAC],
-	uint16_t *max_rxmac)
+static int
+nfb_nc_rxmac_init(struct pmd_internals *priv, struct nfb_init_params *params)
 {
-	*max_rxmac = 0;
-	while ((rxmac[*max_rxmac] = nc_rxmac_open_index(nfb, *max_rxmac)))
-		++(*max_rxmac);
+	int i, j;
+	struct nc_ifc_info *ifc = params->ifc_info;
+	struct nc_ifc_map_info *mi = &params->map_info;
+
+	priv->rxmac = rte_calloc("NFB RxMAC", ifc->eth_cnt, sizeof(*priv->rxmac), 0);
+	if (ifc->eth_cnt && priv->rxmac == NULL)
+		return -ENOMEM;
+
+	for (i = 0, j = 0; i < mi->eth_cnt && j < ifc->eth_cnt; i++) {
+		if (mi->eth[i].ifc != ifc->id)
+			continue;
+		priv->rxmac[j] = nc_rxmac_open(priv->nfb, mi->eth[i].node_rxmac);
+		if (priv->rxmac[j])
+			j++;
+	}
+
+	priv->max_rxmac = j;
+	return 0;
 }
 
 /**
- * Open all TX DMA queues
+ * Open TX MAC components associated with current ifc
  *
- * @param dev
- *   Pointer to nfb device.
- * @param[out] txmac
- *   Pointer to output array of nc_txmac
- * @param[out] max_rxmac
- *   Pointer to output max index of txmac
+ * @param priv
+ *   Pointer to driver private structure
+ * @param params
+ *   Pointer to init parameters structure
  */
-static void
-nfb_nc_txmac_init(struct nfb_device *nfb,
-	struct nc_txmac *txmac[RTE_MAX_NC_TXMAC],
-	uint16_t *max_txmac)
+static int
+nfb_nc_txmac_init(struct pmd_internals *priv, struct nfb_init_params *params)
 {
-	*max_txmac = 0;
-	while ((txmac[*max_txmac] = nc_txmac_open_index(nfb, *max_txmac)))
-		++(*max_txmac);
+	int i, j;
+	struct nc_ifc_info *ifc = params->ifc_info;
+	struct nc_ifc_map_info *mi = &params->map_info;
+
+	priv->txmac = rte_calloc("NFB TxMAC", ifc->eth_cnt, sizeof(*priv->txmac), 0);
+	if (ifc->eth_cnt && priv->txmac == NULL)
+		return -ENOMEM;
+
+	for (i = 0, j = 0; i < mi->eth_cnt && j < ifc->eth_cnt; i++) {
+		if (mi->eth[i].ifc != ifc->id)
+			continue;
+		priv->txmac[j] = nc_txmac_open(priv->nfb, mi->eth[i].node_txmac);
+		if (priv->txmac[j])
+			j++;
+	}
+
+	priv->max_txmac = j;
+	return 0;
 }
 
 /**
- * Close all RX DMA queues
- *
- * @param rxmac
- *   Pointer to array of nc_rxmac
- * @param max_rxmac
- *   Maximum index of rxmac
+ * Close all RX MAC components
+ * @param priv
+ *   Pointer to driver private structure
  */
 static void
-nfb_nc_rxmac_deinit(struct nc_rxmac *rxmac[RTE_MAX_NC_RXMAC],
-	uint16_t max_rxmac)
+nfb_nc_rxmac_deinit(struct pmd_internals *priv)
 {
 	uint16_t i;
-	for (i = 0; i < max_rxmac; i++) {
-		nc_rxmac_close(rxmac[i]);
-		rxmac[i] = NULL;
-	}
+	for (i = 0; i < priv->max_rxmac; i++)
+		nc_rxmac_close(priv->rxmac[i]);
+
+	rte_free(priv->rxmac);
 }
 
 /**
- * Close all TX DMA queues
- *
- * @param txmac
- *   Pointer to array of nc_txmac
- * @param max_txmac
- *   Maximum index of txmac
+ * Close all TX MAC components
+ * @param priv
+ *   Pointer to driver private structure
  */
 static void
-nfb_nc_txmac_deinit(struct nc_txmac *txmac[RTE_MAX_NC_TXMAC],
-	uint16_t max_txmac)
+nfb_nc_txmac_deinit(struct pmd_internals *priv)
 {
 	uint16_t i;
-	for (i = 0; i < max_txmac; i++) {
-		nc_txmac_close(txmac[i]);
-		txmac[i] = NULL;
-	}
+	for (i = 0; i < priv->max_txmac; i++)
+		nc_txmac_close(priv->txmac[i]);
+	rte_free(priv->txmac);
 }
 
 /**
@@ -335,7 +348,7 @@ nfb_eth_link_update(struct rte_eth_dev *dev,
 	link.link_duplex = RTE_ETH_LINK_FULL_DUPLEX;
 	link.link_autoneg = RTE_ETH_LINK_SPEED_FIXED;
 
-	if (internals->rxmac[0] != NULL) {
+	if (internals->max_rxmac) {
 		nc_rxmac_read_status(internals->rxmac[0], &status);
 
 		switch (status.speed) {
@@ -559,12 +572,12 @@ nfb_eth_dev_init(struct rte_eth_dev *dev, void *init_data)
 	max_rx_queues = ifc->rxq_cnt;
 	max_tx_queues = ifc->txq_cnt;
 
-	nfb_nc_rxmac_init(internals->nfb,
-		internals->rxmac,
-		&internals->max_rxmac);
-	nfb_nc_txmac_init(internals->nfb,
-		internals->txmac,
-		&internals->max_txmac);
+	ret = nfb_nc_rxmac_init(internals, params);
+	if (ret)
+		goto err_rxmac_init;
+	ret = nfb_nc_txmac_init(internals, params);
+	if (ret)
+		goto err_txmac_init;
 
 	/* Set rx, tx burst functions */
 	dev->rx_pkt_burst = nfb_eth_ndp_rx;
@@ -630,8 +643,10 @@ nfb_eth_dev_init(struct rte_eth_dev *dev, void *init_data)
 err_malloc_mac_addrs:
 	rte_free(priv->queue_map_rx);
 err_alloc_queue_map:
-	nfb_nc_rxmac_deinit(internals->rxmac, internals->max_rxmac);
-	nfb_nc_txmac_deinit(internals->txmac, internals->max_txmac);
+	nfb_nc_txmac_deinit(internals);
+err_txmac_init:
+	nfb_nc_rxmac_deinit(internals);
+err_rxmac_init:
 	nfb_close(internals->nfb);
 
 err_nfb_open:
@@ -660,8 +675,8 @@ nfb_eth_dev_uninit(struct rte_eth_dev *dev)
 
 	TAILQ_REMOVE(&nfb_eth_dev_list, internals, eth_dev_list);
 
-	nfb_nc_rxmac_deinit(internals->rxmac, internals->max_rxmac);
-	nfb_nc_txmac_deinit(internals->txmac, internals->max_txmac);
+	nfb_nc_rxmac_deinit(internals);
+	nfb_nc_txmac_deinit(internals);
 	nfb_close(internals->nfb);
 
 	rte_free(internals);
-- 
2.52.0