diff options
| author |   Amir Vadai <amir@vadai.me> | 2016-08-18 21:09:06 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-08-19 16:09:55 -0700 |
| commit | 6c3b4f90861c7ed59d0287b0ff7f2623f9d93d73 (patch) | |
| tree | a1bdb22fd938edbd0f726b2a284d3087138d17cf /drivers/net/ethernet/mellanox | |
| parent | net/mlx5: Added missing check of msg length in verifying its signature (diff) | |
| download | linux-dev-6c3b4f90861c7ed59d0287b0ff7f2623f9d93d73.tar.xz linux-dev-6c3b4f90861c7ed59d0287b0ff7f2623f9d93d73.zip | |
net/mlx5: Update last-use statistics for flow rules
Set lastuse statistic, when number of packets is changed compared to
last query. This was wrongly dropped when bulk counter reading was added.
Fixes: a351a1b03bf1 ('net/mlx5: Introduce bulk reading of flow counters')
Signed-off-by: Amir Vadai <amirva@mellanox.com>
Reported-by: Paul Blakey <paulb@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/mellanox')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c b/drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c index c2877e9de8a1..3a9195b4169d 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c @@ -126,12 +126,21 @@ static struct rb_node *mlx5_fc_stats_query(struct mlx5_core_dev *dev, for (node = &first->node; node; node = rb_next(node)) { struct mlx5_fc *counter = rb_entry(node, struct mlx5_fc, node); struct mlx5_fc_cache *c = &counter->cache; + u64 packets; + u64 bytes; if (counter->id > last_id) break; mlx5_cmd_fc_bulk_get(dev, b, - counter->id, &c->packets, &c->bytes); + counter->id, &packets, &bytes); + + if (c->packets == packets) + continue; + + c->packets = packets; + c->bytes = bytes; + c->lastuse = jiffies; } out: |