[dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
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 --- 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
[dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
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 Acked-by: Adrien Mazarguil > --- > 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
[dpdk-dev] [PATCH] net/mlx5: fix Rx function selection
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 > > Acked-by: Adrien Mazarguil > Applied to dpdk-next-net/rel_16_11 /Bruce