Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7300375f authored by Saeed Mahameed's avatar Saeed Mahameed
Browse files

net/mlx5: Move flow counters data structures from flow steering header 



After the following flow counters API refactoring:
("net/mlx5: Use flow counter IDs and not the wrapping cache object")
flow counters private data structures mlx5_fc_cache and mlx5_fc are
redundantly exposed in fs_core.h, they have nothing to do with flow
steering core and they are private to fs_counter.c, this patch moves them
to where they belong and reduces their exposure in the driver.

Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
parent bdefffd1

drivers/net/ethernet/mellanox/mlx5/core/fs_core.h

+0 −23
Original line number Diff line number Diff line
@@ -145,29 +145,6 @@ struct mlx5_flow_table { 
	struct rhltable			fgs_hash;

};



struct mlx5_fc_cache {

	u64 packets;

	u64 bytes;

	u64 lastuse;

};



struct mlx5_fc {

	struct list_head list;

	struct llist_node addlist;

	struct llist_node dellist;



	/* last{packets,bytes} members are used when calculating the delta since

	 * last reading

	 */

	u64 lastpackets;

	u64 lastbytes;



	u32 id;

	bool aging;



	struct mlx5_fc_cache cache ____cacheline_aligned_in_smp;

};



struct mlx5_ft_underlay_qp {

	struct list_head list;

	u32 qpn;

drivers/net/ethernet/mellanox/mlx5/core/fs_counters.c

+23 −0
Original line number Diff line number Diff line
@@ -41,6 +41,29 @@ 
/* Max number of counters to query in bulk read is 32K */

#define MLX5_SW_MAX_COUNTERS_BULK BIT(15)



struct mlx5_fc_cache {

	u64 packets;

	u64 bytes;

	u64 lastuse;

};



struct mlx5_fc {

	struct list_head list;

	struct llist_node addlist;

	struct llist_node dellist;



	/* last{packets,bytes} members are used when calculating the delta since

	 * last reading

	 */

	u64 lastpackets;

	u64 lastbytes;



	u32 id;

	bool aging;



	struct mlx5_fc_cache cache ____cacheline_aligned_in_smp;

};



/* locking scheme:

 *

 * It is the responsibility of the user to prevent concurrent calls or bad