summaryrefslogtreecommitdiffstats
path: root/pimd/pim_rpf.c
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@cumulusnetworks.com>2019-05-03 00:25:57 +0200
committerDonald Sharp <sharpd@cumulusnetworks.com>2019-06-07 19:27:29 +0200
commitae14da4860df6bcffd848ee765d3db4750a4c23f (patch)
treedfef2923862eb6e9519468dc634d7be0ee22a809 /pimd/pim_rpf.c
parentMerge pull request #4449 from opensourcerouting/py-detect (diff)
downloadfrr-ae14da4860df6bcffd848ee765d3db4750a4c23f.tar.xz
frr-ae14da4860df6bcffd848ee765d3db4750a4c23f.zip
pimd: Remove unnecessary boolean to pim_rpf_update
We already log whether or not we add nht tracking, having an additional boolean to say to log another line is a bit over the top. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_rpf.c')
-rw-r--r--pimd/pim_rpf.c11
1 files changed, 1 insertions, 10 deletions
diff --git a/pimd/pim_rpf.c b/pimd/pim_rpf.c
index d38880245..b1a2e717d 100644
--- a/pimd/pim_rpf.c
+++ b/pimd/pim_rpf.c
@@ -195,8 +195,7 @@ static int nexthop_mismatch(const struct pim_nexthop *nh1,
}
enum pim_rpf_result pim_rpf_update(struct pim_instance *pim,
- struct pim_upstream *up, struct pim_rpf *old,
- uint8_t is_new)
+ struct pim_upstream *up, struct pim_rpf *old)
{
struct pim_rpf *rpf = &up->rpf;
struct pim_rpf saved;
@@ -216,14 +215,6 @@ enum pim_rpf_result pim_rpf_update(struct pim_instance *pim,
saved.source_nexthop = rpf->source_nexthop;
saved.rpf_addr = rpf->rpf_addr;
- if (is_new && PIM_DEBUG_ZEBRA) {
- char source_str[INET_ADDRSTRLEN];
- pim_inet4_dump("<source?>", up->upstream_addr, source_str,
- sizeof(source_str));
- zlog_debug("%s: NHT Register upstream %s addr %s with Zebra.",
- __PRETTY_FUNCTION__, up->sg_str, source_str);
- }
- /* Register addr with Zebra NHT */
nht_p.family = AF_INET;
nht_p.prefixlen = IPV4_MAX_BITLEN;
nht_p.u.prefix4.s_addr = up->upstream_addr.s_addr;