Commit 00414126 authored by Alex Vesker's avatar Alex Vesker Committed by Jakub Kicinski

net/mlx5: DR, Allow insertion of duplicate rules

Duplicate rules were not allowed to be configured with SW steering.
This restriction caused failures with the replace rule logic done by
upper layers.

This fix allows for multiple rules with the same match values, in
such case the first inserted rules will match.

Fixes: 41d07074 ("net/mlx5: DR, Expose steering rule functionality")
Signed-off-by: default avatarAlex Vesker <valex@mellanox.com>
Signed-off-by: default avatarTariq Toukan <tariqt@mellanox.com>
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
parent 38dc3b5f
...@@ -788,12 +788,10 @@ dr_rule_handle_ste_branch(struct mlx5dr_rule *rule, ...@@ -788,12 +788,10 @@ dr_rule_handle_ste_branch(struct mlx5dr_rule *rule,
* it means that all the previous stes are the same, * it means that all the previous stes are the same,
* if so, this rule is duplicated. * if so, this rule is duplicated.
*/ */
if (mlx5dr_ste_is_last_in_rule(nic_matcher, if (!mlx5dr_ste_is_last_in_rule(nic_matcher, ste_location))
matched_ste->ste_chain_location)) { return matched_ste;
mlx5dr_info(dmn, "Duplicate rule inserted, aborting!!\n");
return NULL; mlx5dr_dbg(dmn, "Duplicate rule inserted\n");
}
return matched_ste;
} }
if (!skip_rehash && dr_rule_need_enlarge_hash(cur_htbl, dmn, nic_dmn)) { if (!skip_rehash && dr_rule_need_enlarge_hash(cur_htbl, dmn, nic_dmn)) {
......
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