Skip to content

Commit 5c29f55

Browse files
committed
Revert "bgpd: Force adj-rib-out updates if MRAI is kicked in"
This reverts commit 1eed792. The workaround is no longer needed. Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>
1 parent fb40a24 commit 5c29f55

File tree

1 file changed

+3
-17
lines changed

1 file changed

+3
-17
lines changed

bgpd/bgp_packet.c

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,6 @@ void bgp_generate_updgrp_packets(struct event *event)
428428
uint32_t generated = 0;
429429
afi_t afi;
430430
safi_t safi;
431-
enum bgp_af_index index;
432431

433432
wpq = atomic_load_explicit(&peer->bgp->wpkt_quanta,
434433
memory_order_relaxed);
@@ -445,23 +444,8 @@ void bgp_generate_updgrp_packets(struct event *event)
445444
|| bgp_update_delay_active(peer->bgp))
446445
return;
447446

448-
if (peer->connection->t_routeadv) {
449-
/* If MRAI is running, we have to hint "adj-rib-out" to
450-
* ignore suppression of updates for this peer, because
451-
* if we don't, we will miss some updates that are very
452-
* quick (flapping/= del/add) during the MRAI wait time.
453-
*/
454-
for (index = BGP_AF_START; index < BGP_AF_MAX; index++) {
455-
paf = peer->peer_af_array[index];
456-
if (!paf)
457-
continue;
458-
459-
if (paf && paf->subgroup)
460-
SET_FLAG(paf->subgroup->sflags, SUBGRP_STATUS_FORCE_UPDATES);
461-
}
462-
447+
if (peer->connection->t_routeadv)
463448
return;
464-
}
465449

466450
/*
467451
* Since the following is a do while loop
@@ -480,6 +464,8 @@ void bgp_generate_updgrp_packets(struct event *event)
480464
return;
481465

482466
do {
467+
enum bgp_af_index index;
468+
483469
s = NULL;
484470
for (index = BGP_AF_START; index < BGP_AF_MAX; index++) {
485471
paf = peer->peer_af_array[index];

0 commit comments

Comments
 (0)