diff options
| author | Russ White <russ@riw.us> | 2024-02-20 10:17:46 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-02-20 10:17:46 -0500 |
| commit | 8dbfc57ec6bbfac92beb16bea54a79719f227d1b (patch) | |
| tree | 52a2d1cf2e23e4068d4f21fc9cdd4c561436c076 /lib/routemap.c | |
| parent | 5362d4345672987bdcb9444cfe150234938a6e6c (diff) | |
| parent | 049a2c0c0a4d05c2ac3c22b519bfffbbaade2f34 (diff) | |
Merge pull request #15377 from opensourcerouting/fix/bgpd_evpn_route_map
lib: Do not convert EVPN prefixes into IPv4/IPv6 if not needed
Diffstat (limited to 'lib/routemap.c')
| -rw-r--r-- | lib/routemap.c | 29 |
1 files changed, 5 insertions, 24 deletions
diff --git a/lib/routemap.c b/lib/routemap.c index 6b3f81b4d4..ea917ebd8c 100644 --- a/lib/routemap.c +++ b/lib/routemap.c @@ -2548,7 +2548,6 @@ route_map_result_t route_map_apply_ext(struct route_map *map, struct route_map_index *index = NULL; struct route_map_rule *set = NULL; bool skip_match_clause = false; - struct prefix conv; if (recursion > RMAP_RECURSION_LIMIT) { if (map) @@ -2571,31 +2570,14 @@ route_map_result_t route_map_apply_ext(struct route_map *map, map->applied++; - /* - * Handling for matching evpn_routes in the prefix table. - * - * We convert type2/5 prefix to ipv4/6 prefix to do longest - * prefix matching on. - */ if (prefix->family == AF_EVPN) { - if (evpn_prefix2prefix(prefix, &conv) != 0) { - if (unlikely(CHECK_FLAG(rmap_debug, - DEBUG_ROUTEMAP_DETAIL))) - zlog_debug( - "Unable to convert EVPN prefix %pFX into IPv4/IPv6 prefix. Falling back to non-optimized route-map lookup", - prefix); - } else { - if (unlikely(CHECK_FLAG(rmap_debug, - DEBUG_ROUTEMAP_DETAIL))) - zlog_debug( - "Converted EVPN prefix %pFX into %pFX for optimized route-map lookup", - prefix, &conv); - - prefix = &conv; - } + index = map->head; + } else { + skip_match_clause = true; + index = route_map_get_index(map, prefix, match_object, + &match_ret); } - index = route_map_get_index(map, prefix, match_object, &match_ret); if (index) { index->applied++; if (unlikely(CHECK_FLAG(rmap_debug, DEBUG_ROUTEMAP))) @@ -2619,7 +2601,6 @@ route_map_result_t route_map_apply_ext(struct route_map *map, ret = RMAP_DENYMATCH; goto route_map_apply_end; } - skip_match_clause = true; for (; index; index = index->next) { if (!skip_match_clause) { |