Commit c89fb18b authored by Tariq Toukan's avatar Tariq Toukan Committed by David S. Miller

net/mlx5e: Move common case counters within sq_stats struct

For data cache locality considerations, we moved the nop and
csum_offload_inner within sq_stats struct as they are more
commonly accessed in xmit path.
Signed-off-by: default avatarTariq Toukan <tariqt@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3b619524
...@@ -260,26 +260,28 @@ static const char sq_stats_strings[][ETH_GSTRING_LEN] = { ...@@ -260,26 +260,28 @@ static const char sq_stats_strings[][ETH_GSTRING_LEN] = {
"tso_bytes", "tso_bytes",
"tso_inner_packets", "tso_inner_packets",
"tso_inner_bytes", "tso_inner_bytes",
"csum_offload_none",
"csum_offload_inner", "csum_offload_inner",
"nop",
"csum_offload_none",
"stopped", "stopped",
"wake", "wake",
"dropped", "dropped",
"nop"
}; };
struct mlx5e_sq_stats { struct mlx5e_sq_stats {
/* commonly accessed in data path */
u64 packets; u64 packets;
u64 tso_packets; u64 tso_packets;
u64 tso_bytes; u64 tso_bytes;
u64 tso_inner_packets; u64 tso_inner_packets;
u64 tso_inner_bytes; u64 tso_inner_bytes;
u64 csum_offload_none;
u64 csum_offload_inner; u64 csum_offload_inner;
u64 nop;
/* less likely accessed in data path */
u64 csum_offload_none;
u64 stopped; u64 stopped;
u64 wake; u64 wake;
u64 dropped; u64 dropped;
u64 nop;
#define NUM_SQ_STATS 11 #define NUM_SQ_STATS 11
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment