diff options
| author | Maxim Mikityanskiy <maximmi@nvidia.com> | 2022-01-25 12:53:00 +0200 |
|---|---|---|
| committer | Saeed Mahameed <saeedm@nvidia.com> | 2022-02-14 22:30:52 -0800 |
| commit | 71753b8ec103fd71d6ee90e522d797ccf978e4ed (patch) | |
| tree | 30aa7887b97ddaaa6dc511f7f86db9fbeb32b120 | |
| parent | 3a9e5fff2ab0d6f4af701757d35b9453dc563b78 (diff) | |
net/mlx5e: Optimize the common case condition in mlx5e_select_queue
Check all booleans for special queues at once, when deciding whether to
go to the fast path in mlx5e_select_queue. Pack them into bitfields to
have some room for extensibility.
Signed-off-by: Maxim Mikityanskiy <maximmi@nvidia.com>
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en/selq.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/selq.c b/drivers/net/ethernet/mellanox/mlx5/core/en/selq.c index 667bc95a0d44..d98a277eb7f8 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en/selq.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/selq.c @@ -12,8 +12,13 @@ struct mlx5e_selq_params { unsigned int num_regular_queues; unsigned int num_channels; unsigned int num_tcs; - bool is_htb; - bool is_ptp; + union { + u8 is_special_queues; + struct { + bool is_htb : 1; + bool is_ptp : 1; + }; + }; }; int mlx5e_selq_init(struct mlx5e_selq *selq, struct mutex *state_lock) @@ -164,7 +169,7 @@ u16 mlx5e_select_queue(struct net_device *dev, struct sk_buff *skb, if (unlikely(!selq)) return 0; - if (likely(!selq->is_ptp && !selq->is_htb)) { + if (likely(!selq->is_special_queues)) { /* No special queues, netdev_pick_tx returns one of the regular ones. */ txq_ix = netdev_pick_tx(dev, skb, NULL); |