diff options
author | Jesper Dangaard Brouer <brouer@redhat.com> | 2016-03-11 03:44:17 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-03-13 22:35:36 -0400 |
commit | 8ec736e556e3340b4b4295c7567b0766d6629702 (patch) | |
tree | 543137e609a451177fffd4cd2da7aea7d05c356b | |
parent | b4a53379a0c19df784e30b1f35af5841f2b74f30 (diff) |
mlx5: use napi_consume_skb API to get bulk free operations
Bulk free of SKBs happen transparently by the API call napi_consume_skb().
The napi budget parameter is needed by napi_consume_skb() to detect
if called from netpoll.
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en.h | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en_tx.c | 4 | ||||
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en.h b/drivers/net/ethernet/mellanox/mlx5/core/en.h index 0f76d321030f..0cb4a093958b 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en.h +++ b/drivers/net/ethernet/mellanox/mlx5/core/en.h | |||
@@ -629,7 +629,7 @@ netdev_tx_t mlx5e_xmit(struct sk_buff *skb, struct net_device *dev); | |||
629 | void mlx5e_completion_event(struct mlx5_core_cq *mcq); | 629 | void mlx5e_completion_event(struct mlx5_core_cq *mcq); |
630 | void mlx5e_cq_error_event(struct mlx5_core_cq *mcq, enum mlx5_event event); | 630 | void mlx5e_cq_error_event(struct mlx5_core_cq *mcq, enum mlx5_event event); |
631 | int mlx5e_napi_poll(struct napi_struct *napi, int budget); | 631 | int mlx5e_napi_poll(struct napi_struct *napi, int budget); |
632 | bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq); | 632 | bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq, int napi_budget); |
633 | int mlx5e_poll_rx_cq(struct mlx5e_cq *cq, int budget); | 633 | int mlx5e_poll_rx_cq(struct mlx5e_cq *cq, int budget); |
634 | bool mlx5e_post_rx_wqes(struct mlx5e_rq *rq); | 634 | bool mlx5e_post_rx_wqes(struct mlx5e_rq *rq); |
635 | struct mlx5_cqe64 *mlx5e_get_cqe(struct mlx5e_cq *cq); | 635 | struct mlx5_cqe64 *mlx5e_get_cqe(struct mlx5e_cq *cq); |
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c index 94a14f85f70d..1ffc7cb6f78c 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c | |||
@@ -339,7 +339,7 @@ netdev_tx_t mlx5e_xmit(struct sk_buff *skb, struct net_device *dev) | |||
339 | return mlx5e_sq_xmit(sq, skb); | 339 | return mlx5e_sq_xmit(sq, skb); |
340 | } | 340 | } |
341 | 341 | ||
342 | bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq) | 342 | bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq, int napi_budget) |
343 | { | 343 | { |
344 | struct mlx5e_sq *sq; | 344 | struct mlx5e_sq *sq; |
345 | u32 dma_fifo_cc; | 345 | u32 dma_fifo_cc; |
@@ -411,7 +411,7 @@ bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq) | |||
411 | npkts++; | 411 | npkts++; |
412 | nbytes += wi->num_bytes; | 412 | nbytes += wi->num_bytes; |
413 | sqcc += wi->num_wqebbs; | 413 | sqcc += wi->num_wqebbs; |
414 | dev_kfree_skb(skb); | 414 | napi_consume_skb(skb, napi_budget); |
415 | } while (!last_wqe); | 415 | } while (!last_wqe); |
416 | } | 416 | } |
417 | 417 | ||
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c index 66d51a77609e..9bb4395aceeb 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_txrx.c | |||
@@ -60,7 +60,7 @@ int mlx5e_napi_poll(struct napi_struct *napi, int budget) | |||
60 | clear_bit(MLX5E_CHANNEL_NAPI_SCHED, &c->flags); | 60 | clear_bit(MLX5E_CHANNEL_NAPI_SCHED, &c->flags); |
61 | 61 | ||
62 | for (i = 0; i < c->num_tc; i++) | 62 | for (i = 0; i < c->num_tc; i++) |
63 | busy |= mlx5e_poll_tx_cq(&c->sq[i].cq); | 63 | busy |= mlx5e_poll_tx_cq(&c->sq[i].cq, budget); |
64 | 64 | ||
65 | work_done = mlx5e_poll_rx_cq(&c->rq.cq, budget); | 65 | work_done = mlx5e_poll_rx_cq(&c->rq.cq, budget); |
66 | busy |= work_done == budget; | 66 | busy |= work_done == budget; |