diff mbox series

[v5,16/18] net/mana: add function to start/stop device

Message ID 1661560509-11009-17-git-send-email-longli@linuxonhyperv.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers
Series Introduce Microsoft Azure Network Adatper (MANA) PMD |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Long Li Aug. 27, 2022, 12:35 a.m. UTC 
  From: Long Li <longli@microsoft.com>

Add support for starting/stopping the device.

Signed-off-by: Long Li <longli@microsoft.com>
---
Change log:
v2:
Use spinlock for memory registration cache.
Add prefix mana_ to all function names.

 drivers/net/mana/mana.c | 70 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)

Comments

Suanming Mou Aug. 29, 2022, 3:38 a.m. UTC | #1 
Hi,

> -----Original Message-----
> From: longli@linuxonhyperv.com <longli@linuxonhyperv.com>
> Sent: Saturday, August 27, 2022 8:35 AM
> To: Ferruh Yigit <ferruh.yigit@xilinx.com>
> Cc: dev@dpdk.org; Ajay Sharma <sharmaajay@microsoft.com>; Stephen
> Hemminger <sthemmin@microsoft.com>; NBU-Contact-longli (EXTERNAL)
> <longli@microsoft.com>
> Subject: [Patch v5 16/18] net/mana: add function to start/stop device
> 
> From: Long Li <longli@microsoft.com>
> 
> Add support for starting/stopping the device.
> 
> Signed-off-by: Long Li <longli@microsoft.com>
> ---
> Change log:
> v2:
> Use spinlock for memory registration cache.
> Add prefix mana_ to all function names.
> 
>  drivers/net/mana/mana.c | 70
> +++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 70 insertions(+)
> 
> diff --git a/drivers/net/mana/mana.c b/drivers/net/mana/mana.c index
> 0dcd3f3124..119fdfd8bc 100644
> --- a/drivers/net/mana/mana.c
> +++ b/drivers/net/mana/mana.c
> @@ -97,6 +97,74 @@ static int mana_dev_configure(struct rte_eth_dev *dev)
> 
>  static int mana_intr_uninstall(struct mana_priv *priv);
> 
> +static int
> +mana_dev_start(struct rte_eth_dev *dev) {
> +	int ret;
> +	struct mana_priv *priv = dev->data->dev_private;
> +
> +	rte_spinlock_init(&priv->mr_btree_lock);
> +	ret = mana_mr_btree_init(&priv->mr_btree,

Refer to the previous patch, looks mana_mr_btree_free() is needed in the later failure routine, right?

> MANA_MR_BTREE_CACHE_N,
> +				 dev->device->numa_node);
> +	if (ret) {
> +		DRV_LOG(ERR, "Failed to init device MR btree %d", ret);
> +		return ret;
> +	}
> +
> +	ret = mana_start_tx_queues(dev);
> +	if (ret) {
> +		DRV_LOG(ERR, "failed to start tx queues %d", ret);
> +		return ret;
> +	}
> +
> +	ret = mana_start_rx_queues(dev);
> +	if (ret) {
> +		DRV_LOG(ERR, "failed to start rx queues %d", ret);
> +		mana_stop_tx_queues(dev);
> +		return ret;
> +	}
> +
> +	rte_wmb();
> +
> +	dev->tx_pkt_burst = mana_tx_burst;
> +	dev->rx_pkt_burst = mana_rx_burst;
> +
> +	DRV_LOG(INFO, "TX/RX queues have started");
> +
> +	/* Enable datapath for secondary processes */
> +	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_START_RXTX);
> +
> +	return 0;
> +}
> +
> +static int
> +mana_dev_stop(struct rte_eth_dev *dev __rte_unused) {
> +	int ret;
> +
> +	dev->tx_pkt_burst = mana_tx_burst_removed;
> +	dev->rx_pkt_burst = mana_rx_burst_removed;
> +
> +	/* Stop datapath on secondary processes */
> +	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_STOP_RXTX);
> +
> +	rte_wmb();
> +
> +	ret = mana_stop_tx_queues(dev);
> +	if (ret) {
> +		DRV_LOG(ERR, "failed to stop tx queues");
> +		return ret;
> +	}
> +
> +	ret = mana_stop_rx_queues(dev);
> +	if (ret) {
> +		DRV_LOG(ERR, "failed to stop tx queues");
> +		return ret;
> +	}
> +
> +	return 0;
> +}
> +
>  static int
>  mana_dev_close(struct rte_eth_dev *dev)  { @@ -435,6 +503,8 @@ static int
> mana_dev_link_update(struct rte_eth_dev *dev,
> 
>  const struct eth_dev_ops mana_dev_ops = {
>  	.dev_configure		= mana_dev_configure,
> +	.dev_start		= mana_dev_start,
> +	.dev_stop		= mana_dev_stop,
>  	.dev_close		= mana_dev_close,
>  	.dev_infos_get		= mana_dev_info_get,
>  	.txq_info_get		= mana_dev_tx_queue_info,
> --
> 2.34.1
Long Li Aug. 30, 2022, 1:09 a.m. UTC | #2 
> > Subject: [Patch v5 16/18] net/mana: add function to start/stop device
> >
> > From: Long Li <longli@microsoft.com>
> >
> > Add support for starting/stopping the device.
> >
> > Signed-off-by: Long Li <longli@microsoft.com>
> > ---
> > Change log:
> > v2:
> > Use spinlock for memory registration cache.
> > Add prefix mana_ to all function names.
> >
> >  drivers/net/mana/mana.c | 70
> > +++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 70 insertions(+)
> >
> > diff --git a/drivers/net/mana/mana.c b/drivers/net/mana/mana.c index
> > 0dcd3f3124..119fdfd8bc 100644
> > --- a/drivers/net/mana/mana.c
> > +++ b/drivers/net/mana/mana.c
> > @@ -97,6 +97,74 @@ static int mana_dev_configure(struct rte_eth_dev
> > *dev)
> >
> >  static int mana_intr_uninstall(struct mana_priv *priv);
> >
> > +static int
> > +mana_dev_start(struct rte_eth_dev *dev) {
> > +	int ret;
> > +	struct mana_priv *priv = dev->data->dev_private;
> > +
> > +	rte_spinlock_init(&priv->mr_btree_lock);
> > +	ret = mana_mr_btree_init(&priv->mr_btree,
> 
> Refer to the previous patch, looks mana_mr_btree_free() is needed in the later
> failure routine, right?

Thank you, will fix this.


> 
> > MANA_MR_BTREE_CACHE_N,
> > +				 dev->device->numa_node);
> > +	if (ret) {
> > +		DRV_LOG(ERR, "Failed to init device MR btree %d", ret);
> > +		return ret;
> > +	}
> > +
> > +	ret = mana_start_tx_queues(dev);
> > +	if (ret) {
> > +		DRV_LOG(ERR, "failed to start tx queues %d", ret);
> > +		return ret;
> > +	}
> > +
> > +	ret = mana_start_rx_queues(dev);
> > +	if (ret) {
> > +		DRV_LOG(ERR, "failed to start rx queues %d", ret);
> > +		mana_stop_tx_queues(dev);
> > +		return ret;
> > +	}
> > +
> > +	rte_wmb();
> > +
> > +	dev->tx_pkt_burst = mana_tx_burst;
> > +	dev->rx_pkt_burst = mana_rx_burst;
> > +
> > +	DRV_LOG(INFO, "TX/RX queues have started");
> > +
> > +	/* Enable datapath for secondary processes */
> > +	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_START_RXTX);
> > +
> > +	return 0;
> > +}
> > +
> > +static int
> > +mana_dev_stop(struct rte_eth_dev *dev __rte_unused) {
> > +	int ret;
> > +
> > +	dev->tx_pkt_burst = mana_tx_burst_removed;
> > +	dev->rx_pkt_burst = mana_rx_burst_removed;
> > +
> > +	/* Stop datapath on secondary processes */
> > +	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_STOP_RXTX);
> > +
> > +	rte_wmb();
> > +
> > +	ret = mana_stop_tx_queues(dev);
> > +	if (ret) {
> > +		DRV_LOG(ERR, "failed to stop tx queues");
> > +		return ret;
> > +	}
> > +
> > +	ret = mana_stop_rx_queues(dev);
> > +	if (ret) {
> > +		DRV_LOG(ERR, "failed to stop tx queues");
> > +		return ret;
> > +	}
> > +
> > +	return 0;
> > +}
> > +
> >  static int
> >  mana_dev_close(struct rte_eth_dev *dev)  { @@ -435,6 +503,8 @@ static
> > int mana_dev_link_update(struct rte_eth_dev *dev,
> >
> >  const struct eth_dev_ops mana_dev_ops = {
> >  	.dev_configure		= mana_dev_configure,
> > +	.dev_start		= mana_dev_start,
> > +	.dev_stop		= mana_dev_stop,
> >  	.dev_close		= mana_dev_close,
> >  	.dev_infos_get		= mana_dev_info_get,
> >  	.txq_info_get		= mana_dev_tx_queue_info,
> > --
> > 2.34.1
diff mbox series

Patch

diff --git a/drivers/net/mana/mana.c b/drivers/net/mana/mana.c
index 0dcd3f3124..119fdfd8bc 100644
--- a/drivers/net/mana/mana.c
+++ b/drivers/net/mana/mana.c
@@ -97,6 +97,74 @@  static int mana_dev_configure(struct rte_eth_dev *dev)
 
 static int mana_intr_uninstall(struct mana_priv *priv);
 
+static int
+mana_dev_start(struct rte_eth_dev *dev)
+{
+	int ret;
+	struct mana_priv *priv = dev->data->dev_private;
+
+	rte_spinlock_init(&priv->mr_btree_lock);
+	ret = mana_mr_btree_init(&priv->mr_btree, MANA_MR_BTREE_CACHE_N,
+				 dev->device->numa_node);
+	if (ret) {
+		DRV_LOG(ERR, "Failed to init device MR btree %d", ret);
+		return ret;
+	}
+
+	ret = mana_start_tx_queues(dev);
+	if (ret) {
+		DRV_LOG(ERR, "failed to start tx queues %d", ret);
+		return ret;
+	}
+
+	ret = mana_start_rx_queues(dev);
+	if (ret) {
+		DRV_LOG(ERR, "failed to start rx queues %d", ret);
+		mana_stop_tx_queues(dev);
+		return ret;
+	}
+
+	rte_wmb();
+
+	dev->tx_pkt_burst = mana_tx_burst;
+	dev->rx_pkt_burst = mana_rx_burst;
+
+	DRV_LOG(INFO, "TX/RX queues have started");
+
+	/* Enable datapath for secondary processes */
+	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_START_RXTX);
+
+	return 0;
+}
+
+static int
+mana_dev_stop(struct rte_eth_dev *dev __rte_unused)
+{
+	int ret;
+
+	dev->tx_pkt_burst = mana_tx_burst_removed;
+	dev->rx_pkt_burst = mana_rx_burst_removed;
+
+	/* Stop datapath on secondary processes */
+	mana_mp_req_on_rxtx(dev, MANA_MP_REQ_STOP_RXTX);
+
+	rte_wmb();
+
+	ret = mana_stop_tx_queues(dev);
+	if (ret) {
+		DRV_LOG(ERR, "failed to stop tx queues");
+		return ret;
+	}
+
+	ret = mana_stop_rx_queues(dev);
+	if (ret) {
+		DRV_LOG(ERR, "failed to stop tx queues");
+		return ret;
+	}
+
+	return 0;
+}
+
 static int
 mana_dev_close(struct rte_eth_dev *dev)
 {
@@ -435,6 +503,8 @@  static int mana_dev_link_update(struct rte_eth_dev *dev,
 
 const struct eth_dev_ops mana_dev_ops = {
 	.dev_configure		= mana_dev_configure,
+	.dev_start		= mana_dev_start,
+	.dev_stop		= mana_dev_stop,
 	.dev_close		= mana_dev_close,
 	.dev_infos_get		= mana_dev_info_get,
 	.txq_info_get		= mana_dev_tx_queue_info,