* [dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
@ 2016-10-11 14:44 Nelio Laranjeiro
2016-10-11 15:01 ` Adrien Mazarguil
0 siblings, 1 reply; 3+ messages in thread
From: Nelio Laranjeiro @ 2016-10-11 14:44 UTC (permalink / raw)
To: dev; +Cc: Adrien Mazarguil
mlx5_rx_queue_setup() was setting the Rx function by itself instead of
using priv_select_rx_function() written for that purpose.
Fixes: cdab90cb5c8d ("net/mlx5: add Tx/Rx burst function selection wrapper")
Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
---
drivers/net/mlx5/mlx5_rxq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c
index b9a5fe6..fe27d22 100644
--- a/drivers/net/mlx5/mlx5_rxq.c
+++ b/drivers/net/mlx5/mlx5_rxq.c
@@ -1264,7 +1264,7 @@ mlx5_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
(void *)dev, (void *)rxq_ctrl);
(*priv->rxqs)[idx] = &rxq_ctrl->rxq;
/* Update receive callback. */
- dev->rx_pkt_burst = mlx5_rx_burst;
+ priv_select_rx_function(priv);
}
priv_unlock(priv);
return -ret;
--
2.1.4
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
2016-10-11 14:44 [dpdk-dev] [PATCH] net/mlx5: fix Rx function selection Nelio Laranjeiro
@ 2016-10-11 15:01 ` Adrien Mazarguil
2016-10-13 9:39 ` Bruce Richardson
0 siblings, 1 reply; 3+ messages in thread
From: Adrien Mazarguil @ 2016-10-11 15:01 UTC (permalink / raw)
To: Nelio Laranjeiro; +Cc: dev
On Tue, Oct 11, 2016 at 04:44:50PM +0200, Nelio Laranjeiro wrote:
> mlx5_rx_queue_setup() was setting the Rx function by itself instead of
> using priv_select_rx_function() written for that purpose.
>
> Fixes: cdab90cb5c8d ("net/mlx5: add Tx/Rx burst function selection wrapper")
>
> Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
> ---
> drivers/net/mlx5/mlx5_rxq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c
> index b9a5fe6..fe27d22 100644
> --- a/drivers/net/mlx5/mlx5_rxq.c
> +++ b/drivers/net/mlx5/mlx5_rxq.c
> @@ -1264,7 +1264,7 @@ mlx5_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
> (void *)dev, (void *)rxq_ctrl);
> (*priv->rxqs)[idx] = &rxq_ctrl->rxq;
> /* Update receive callback. */
> - dev->rx_pkt_burst = mlx5_rx_burst;
> + priv_select_rx_function(priv);
> }
> priv_unlock(priv);
> return -ret;
> --
> 2.1.4
>
--
Adrien Mazarguil
6WIND
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
2016-10-11 15:01 ` Adrien Mazarguil
@ 2016-10-13 9:39 ` Bruce Richardson
0 siblings, 0 replies; 3+ messages in thread
From: Bruce Richardson @ 2016-10-13 9:39 UTC (permalink / raw)
To: Adrien Mazarguil; +Cc: Nelio Laranjeiro, dev
On Tue, Oct 11, 2016 at 05:01:36PM +0200, Adrien Mazarguil wrote:
> On Tue, Oct 11, 2016 at 04:44:50PM +0200, Nelio Laranjeiro wrote:
> > mlx5_rx_queue_setup() was setting the Rx function by itself instead of
> > using priv_select_rx_function() written for that purpose.
> >
> > Fixes: cdab90cb5c8d ("net/mlx5: add Tx/Rx burst function selection wrapper")
> >
> > Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
>
> Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
>
Applied to dpdk-next-net/rel_16_11
/Bruce
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-10-13 9:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-11 14:44 [dpdk-dev] [PATCH] net/mlx5: fix Rx function selection Nelio Laranjeiro
2016-10-11 15:01 ` Adrien Mazarguil
2016-10-13 9:39 ` Bruce Richardson
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).