From d4a2bc11adc87fb6dc7d460080d65fde017a964f Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Wed, 14 Sep 2016 19:17:54 -0400 Subject: [PATCH] pimd, zebra: Pass the ifindex for NEXTHOP_TYPE_IPV4 NEXTHOP_TYPE_IPV4 has the ifindex of the route. Pass it along so the other side can use it if it is needed. This will make pim much happier in that we will need to do less recursive lookups. Signed-off-by: Donald Sharp --- pimd/pim_zlookup.c | 21 +++------------------ zebra/zserv.c | 2 -- 2 files changed, 3 insertions(+), 20 deletions(-) diff --git a/pimd/pim_zlookup.c b/pimd/pim_zlookup.c index f649098060..f81c6e8781 100644 --- a/pimd/pim_zlookup.c +++ b/pimd/pim_zlookup.c @@ -233,8 +233,10 @@ static int zclient_read_nexthop(struct zclient *zlookup, switch (nexthop_type) { case NEXTHOP_TYPE_IFINDEX: case NEXTHOP_TYPE_IPV4_IFINDEX: + case NEXTHOP_TYPE_IPV4: nexthop_tab[num_ifindex].nexthop_addr.family = AF_INET; - if (nexthop_type == NEXTHOP_TYPE_IPV4_IFINDEX) { + if (nexthop_type == NEXTHOP_TYPE_IPV4_IFINDEX || + nexthop_type == NEXTHOP_TYPE_IPV4) { nexthop_tab[num_ifindex].nexthop_addr.u.prefix4.s_addr = stream_get_ipv4(s); } else { @@ -245,23 +247,6 @@ static int zclient_read_nexthop(struct zclient *zlookup, nexthop_tab[num_ifindex].route_metric = metric; ++num_ifindex; break; - case NEXTHOP_TYPE_IPV4: - nexthop_tab[num_ifindex].nexthop_addr.family = AF_INET; - nexthop_tab[num_ifindex].nexthop_addr.u.prefix4.s_addr = stream_get_ipv4(s); - nexthop_tab[num_ifindex].ifindex = 0; - nexthop_tab[num_ifindex].protocol_distance = distance; - nexthop_tab[num_ifindex].route_metric = metric; - if (PIM_DEBUG_ZEBRA) { - char addr_str[100]; - char nexthop_str[100]; - pim_inet4_dump("", addr, addr_str, sizeof(addr_str)); - pim_addr_dump("", &nexthop_tab[num_ifindex].nexthop_addr, nexthop_str, sizeof(nexthop_str)); - zlog_debug("%s %s: zebra returned recursive nexthop %s for address %s", - __FILE__, __PRETTY_FUNCTION__, - nexthop_str, addr_str); - } - ++num_ifindex; - break; case NEXTHOP_TYPE_IPV6_IFINDEX: nexthop_tab[num_ifindex].nexthop_addr.family = AF_INET6; stream_get (&nexthop_tab[num_ifindex].nexthop_addr.u.prefix6, s, 16); diff --git a/zebra/zserv.c b/zebra/zserv.c index 44e29a8bb0..bdfa03e1ee 100644 --- a/zebra/zserv.c +++ b/zebra/zserv.c @@ -786,8 +786,6 @@ zsend_write_nexthop (struct stream *s, struct nexthop *nexthop) switch (nexthop->type) { case NEXTHOP_TYPE_IPV4: - stream_put_in_addr (s, &nexthop->gate.ipv4); - break; case NEXTHOP_TYPE_IPV4_IFINDEX: stream_put_in_addr (s, &nexthop->gate.ipv4); stream_putl (s, nexthop->ifindex); -- 2.39.5