summaryrefslogtreecommitdiffstats
path: root/bgpd
diff options
context:
space:
mode:
authorSoman K S <somanks@vmware.com>2019-06-12 12:24:37 +0200
committerSoman K S <somanks@vmware.com>2019-06-13 19:04:50 +0200
commit7bfdba54126b08ceb6056e9f93e756c0ec824535 (patch)
tree5ce7178b10932304099b0a32bb4ee85453de43b4 /bgpd
parentMerge pull request #4494 from donaldsharp/no_src_for_you (diff)
downloadfrr-7bfdba54126b08ceb6056e9f93e756c0ec824535.tar.xz
frr-7bfdba54126b08ceb6056e9f93e756c0ec824535.zip
bgpd: Process core when prefix list is applied
* Added debug logs and assert to get more information when bgp process core is observed Signed-off-by: Soman K S <somanks@vmware.com>
Diffstat (limited to 'bgpd')
-rw-r--r--bgpd/bgp_updgrp.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/bgpd/bgp_updgrp.c b/bgpd/bgp_updgrp.c
index d0be2471a..82df1905b 100644
--- a/bgpd/bgp_updgrp.c
+++ b/bgpd/bgp_updgrp.c
@@ -885,6 +885,9 @@ static void update_subgroup_add_peer(struct update_subgroup *subgrp,
bpacket_add_peer(pkt, paf);
bpacket_queue_sanity_check(SUBGRP_PKTQ(subgrp));
+ if (BGP_DEBUG(update_groups, UPDATE_GROUPS))
+ zlog_debug("peer %s added to subgroup s%" PRIu64,
+ paf->peer->host, subgrp->id);
}
/*
@@ -910,6 +913,10 @@ static void update_subgroup_remove_peer_internal(struct update_subgroup *subgrp,
paf->subgroup = NULL;
subgrp->peer_count--;
+ if (BGP_DEBUG(update_groups, UPDATE_GROUPS))
+ zlog_debug("peer %s deleted from subgroup s%"
+ PRIu64 "peer cnt %d",
+ paf->peer->host, subgrp->id, subgrp->peer_count);
SUBGRP_INCR_STAT(subgrp, prune_events);
}
@@ -1826,9 +1833,9 @@ void peer_af_announce_route(struct peer_af *paf, int combine)
*/
if (!combine || !all_pending) {
update_subgroup_split_peer(paf, NULL);
- if (!paf->subgroup)
- return;
+ subgrp = paf->subgroup;
+ assert(subgrp && subgrp->update_group);
if (bgp_debug_update(paf->peer, NULL, subgrp->update_group, 0))
zlog_debug("u%" PRIu64 ":s%" PRIu64
" %s announcing routes",