Commit 1a05ca8ead for frr
commit 1a05ca8ead0f0ea564b0dfcfb002895580976cab
Author: Enke Chen <enchen@paloaltonetworks.com>
Date: Thu Nov 5 10:40:50 2026 -0800
bgpd: remove SUBGRP_STATUS_FORCE_UPDATES from update_group_announce_walkcb
The flag was added in 05ab8ceda4 ("bgpd: fix bgp max-med on-startup")
to work around duplicate suppression where the MED value was written
directly from bgp->maxmed_value at packet build time, bypassing
attribute comparison.
This is now fixed by cfa3e00025 ("bgpd: set max-med MED value in
subgroup_announce_check") which sets attr->med so duplicate suppression
works correctly.
Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>
diff --git a/bgpd/bgp_updgrp_adv.c b/bgpd/bgp_updgrp_adv.c
index d28ec0abbc..7c980e5db1 100644
--- a/bgpd/bgp_updgrp_adv.c
+++ b/bgpd/bgp_updgrp_adv.c
@@ -435,14 +435,8 @@ static int update_group_announce_walkcb(struct update_group *updgrp, void *arg)
{
struct update_subgroup *subgrp;
- UPDGRP_FOREACH_SUBGRP (updgrp, subgrp) {
- /* Avoid suppressing duplicate routes later
- * when processing in subgroup_announce_table().
- */
- SET_FLAG(subgrp->sflags, SUBGRP_STATUS_FORCE_UPDATES);
-
+ UPDGRP_FOREACH_SUBGRP (updgrp, subgrp)
subgroup_announce_all(subgrp);
- }
return UPDWALK_CONTINUE;
}