Commit a6224985 authored by Mark Bloch's avatar Mark Bloch Committed by Leon Romanovsky

net/mlx5: Group similer rules under the same fte

When adding a new rule, if we can match it with compare_match_value and
flow tag we might be able to insert the rule to the same fte.
In order to do that, there must be an overlap between the actions of the
fte and the new rule.

When updating the action of an existing fte, we must tell the firmware
we are doing so.
Signed-off-by: default avatarMark Bloch <markb@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 814fb875
...@@ -920,7 +920,8 @@ static struct mlx5_flow_rule *alloc_rule(struct mlx5_flow_destination *dest) ...@@ -920,7 +920,8 @@ static struct mlx5_flow_rule *alloc_rule(struct mlx5_flow_destination *dest)
/* fte should not be deleted while calling this function */ /* fte should not be deleted while calling this function */
static struct mlx5_flow_rule *add_rule_fte(struct fs_fte *fte, static struct mlx5_flow_rule *add_rule_fte(struct fs_fte *fte,
struct mlx5_flow_group *fg, struct mlx5_flow_group *fg,
struct mlx5_flow_destination *dest) struct mlx5_flow_destination *dest,
bool update_action)
{ {
struct mlx5_flow_table *ft; struct mlx5_flow_table *ft;
struct mlx5_flow_rule *rule; struct mlx5_flow_rule *rule;
...@@ -931,6 +932,9 @@ static struct mlx5_flow_rule *add_rule_fte(struct fs_fte *fte, ...@@ -931,6 +932,9 @@ static struct mlx5_flow_rule *add_rule_fte(struct fs_fte *fte,
if (!rule) if (!rule)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
if (update_action)
modify_mask |= BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_ACTION);
fs_get_obj(ft, fg->node.parent); fs_get_obj(ft, fg->node.parent);
/* Add dest to dests list- we need flow tables to be in the /* Add dest to dests list- we need flow tables to be in the
* end of the list for forward to next prio rules. * end of the list for forward to next prio rules.
...@@ -1109,7 +1113,9 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg, ...@@ -1109,7 +1113,9 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg,
fs_for_each_fte(fte, fg) { fs_for_each_fte(fte, fg) {
nested_lock_ref_node(&fte->node, FS_MUTEX_CHILD); nested_lock_ref_node(&fte->node, FS_MUTEX_CHILD);
if (compare_match_value(&fg->mask, match_value, &fte->val) && if (compare_match_value(&fg->mask, match_value, &fte->val) &&
action == fte->action && flow_tag == fte->flow_tag) { (action & fte->action) && flow_tag == fte->flow_tag) {
int old_action = fte->action;
rule = find_flow_rule(fte, dest); rule = find_flow_rule(fte, dest);
if (rule) { if (rule) {
atomic_inc(&rule->node.refcount); atomic_inc(&rule->node.refcount);
...@@ -1117,11 +1123,15 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg, ...@@ -1117,11 +1123,15 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg,
unlock_ref_node(&fg->node); unlock_ref_node(&fg->node);
return rule; return rule;
} }
rule = add_rule_fte(fte, fg, dest); fte->action |= action;
if (IS_ERR(rule)) rule = add_rule_fte(fte, fg, dest,
old_action != action);
if (IS_ERR(rule)) {
fte->action = old_action;
goto unlock_fte; goto unlock_fte;
else } else {
goto add_rule; goto add_rule;
}
} }
unlock_ref_node(&fte->node); unlock_ref_node(&fte->node);
} }
...@@ -1138,7 +1148,7 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg, ...@@ -1138,7 +1148,7 @@ static struct mlx5_flow_rule *add_rule_fg(struct mlx5_flow_group *fg,
} }
tree_init_node(&fte->node, 0, del_fte); tree_init_node(&fte->node, 0, del_fte);
nested_lock_ref_node(&fte->node, FS_MUTEX_CHILD); nested_lock_ref_node(&fte->node, FS_MUTEX_CHILD);
rule = add_rule_fte(fte, fg, dest); rule = add_rule_fte(fte, fg, dest, false);
if (IS_ERR(rule)) { if (IS_ERR(rule)) {
kfree(fte); kfree(fte);
goto unlock_fg; goto unlock_fg;
......
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