Commit 1207a772 authored by Yevgeny Kliteynik's avatar Yevgeny Kliteynik Committed by Saeed Mahameed
Browse files

net/mlx5: DR, Add function that tells if STE miss addr has been initialized 



Up until now miss address in all the STEs was used to connect miss lists
and to link the last STE in the list to end anchor.
Match range STE will require special handling because its miss address is
part of the 'action'. That is, range action has hit and miss addresses.
Since the range action is always the last action, need to make sure that
its miss address isn't overwritten by the end anchor.

Adding new function mlx5dr_ste_is_miss_addr_set() to answer the question
whether the STE's miss address has already been set as part of STE
initialization. Use a callback that always returns false right now. Once
match range is added, a different callback will be used for that STE type.

Signed-off-by: default avatarYevgeny Kliteynik <kliteyn@nvidia.com>
Reviewed-by: default avatarErez Shitrit <erezsh@nvidia.com>
Reviewed-by: default avatarMark Bloch <mbloch@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent f31bda78

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_rule.c

+3 −0
Original line number Diff line number Diff line
@@ -42,6 +42,9 @@ static void dr_rule_set_last_ste_miss_addr(struct mlx5dr_matcher *matcher, 
	struct mlx5dr_ste_ctx *ste_ctx = matcher->tbl->dmn->ste_ctx;

	u64 icm_addr;



	if (mlx5dr_ste_is_miss_addr_set(ste_ctx, hw_ste))

		return;



	icm_addr = mlx5dr_icm_pool_get_chunk_icm_addr(nic_matcher->e_anchor->chunk);

	mlx5dr_ste_set_miss_addr(ste_ctx, hw_ste, icm_addr);

}

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_ste.c

+10 −0
Original line number Diff line number Diff line
@@ -90,6 +90,16 @@ static void dr_ste_set_always_miss(struct dr_hw_ste_format *hw_ste) 
	hw_ste->mask[0] = 0;

}



bool mlx5dr_ste_is_miss_addr_set(struct mlx5dr_ste_ctx *ste_ctx,

				 u8 *hw_ste_p)

{

	if (!ste_ctx->is_miss_addr_set)

		return false;



	/* check if miss address is already set for this type of STE */

	return ste_ctx->is_miss_addr_set(hw_ste_p);

}



void mlx5dr_ste_set_miss_addr(struct mlx5dr_ste_ctx *ste_ctx,

			      u8 *hw_ste_p, u64 miss_addr)

{

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_ste.h

+1 −0
Original line number Diff line number Diff line
@@ -151,6 +151,7 @@ struct mlx5dr_ste_ctx { 
			 bool is_rx, u16 gvmi);

	void (*set_next_lu_type)(u8 *hw_ste_p, u16 lu_type);

	u16  (*get_next_lu_type)(u8 *hw_ste_p);

	bool (*is_miss_addr_set)(u8 *hw_ste_p);

	void (*set_miss_addr)(u8 *hw_ste_p, u64 miss_addr);

	u64  (*get_miss_addr)(u8 *hw_ste_p);

	void (*set_hit_addr)(u8 *hw_ste_p, u64 icm_addr, u32 ht_size);

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_ste_v1.c

+6 −0
Original line number Diff line number Diff line
@@ -267,6 +267,11 @@ static void dr_ste_v1_set_entry_type(u8 *hw_ste_p, u8 entry_type) 
	MLX5_SET(ste_match_bwc_v1, hw_ste_p, entry_format, entry_type);

}



bool dr_ste_v1_is_miss_addr_set(u8 *hw_ste_p)

{

	return false;

}



void dr_ste_v1_set_miss_addr(u8 *hw_ste_p, u64 miss_addr)

{

	u64 index = miss_addr >> 6;
@@ -2144,6 +2149,7 @@ static struct mlx5dr_ste_ctx ste_ctx_v1 = { 
	.ste_init			= &dr_ste_v1_init,

	.set_next_lu_type		= &dr_ste_v1_set_next_lu_type,

	.get_next_lu_type		= &dr_ste_v1_get_next_lu_type,

	.is_miss_addr_set		= &dr_ste_v1_is_miss_addr_set,

	.set_miss_addr			= &dr_ste_v1_set_miss_addr,

	.get_miss_addr			= &dr_ste_v1_get_miss_addr,

	.set_hit_addr			= &dr_ste_v1_set_hit_addr,

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_ste_v1.h

+1 −0
Original line number Diff line number Diff line
@@ -7,6 +7,7 @@ 
#include "dr_types.h"

#include "dr_ste.h"



bool dr_ste_v1_is_miss_addr_set(u8 *hw_ste_p);

void dr_ste_v1_set_miss_addr(u8 *hw_ste_p, u64 miss_addr);

u64 dr_ste_v1_get_miss_addr(u8 *hw_ste_p);

void dr_ste_v1_set_byte_mask(u8 *hw_ste_p, u16 byte_mask);