From a26aa0d5e21632fa5fd2bf2195b59bac29a7bfbc Mon Sep 17 00:00:00 2001 From: Yangchao Zhou Date: Mon, 20 Apr 2020 10:48:50 +0800 Subject: [PATCH 1/1] lpm: skip table entries update if rules found Table entries do not need to be updated if the same rules can be found. Signed-off-by: Yangchao Zhou Acked-by: Vladimir Medvedkin --- lib/librte_lpm/rte_lpm.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c index 2687564194..38ab512a42 100644 --- a/lib/librte_lpm/rte_lpm.c +++ b/lib/librte_lpm/rte_lpm.c @@ -283,8 +283,12 @@ rule_add(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth, for (; rule_index < last_rule; rule_index++) { - /* If rule already exists update its next_hop and return. */ + /* If rule already exists update next hop and return. */ if (lpm->rules_tbl[rule_index].ip == ip_masked) { + + if (lpm->rules_tbl[rule_index].next_hop + == next_hop) + return -EEXIST; lpm->rules_tbl[rule_index].next_hop = next_hop; return rule_index; @@ -674,6 +678,12 @@ rte_lpm_add(struct rte_lpm *lpm, uint32_t ip, uint8_t depth, /* Add the rule to the rule table. */ rule_index = rule_add(lpm, ip_masked, depth, next_hop); + /* Skip table entries update if The rule is the same as + * the rule in the rules table. + */ + if (rule_index == -EEXIST) + return 0; + /* If the is no space available for new rule return error. */ if (rule_index < 0) { return rule_index; -- 2.20.1