]> git.proxmox.com Git - mirror_frr.git/blobdiff - zebra/zebra_rnh.c
zebra, lib: fix the ZEBRA_INTERFACE_VRF_UPDATE zapi message
[mirror_frr.git] / zebra / zebra_rnh.c
index 30062a0728676c2ab832153aebf5a30ca4cb5018..b1fbe8a6536de6dbdf2bf91b6f520ec8664a7266 100644 (file)
@@ -36,6 +36,7 @@
 #include "nexthop.h"
 #include "vrf.h"
 
+#include "zebra/zebra_router.h"
 #include "zebra/rib.h"
 #include "zebra/rt.h"
 #include "zebra/zserv.h"
@@ -171,8 +172,8 @@ struct rnh *zebra_lookup_rnh(struct prefix *p, vrf_id_t vrfid, rnh_type_t type)
 void zebra_free_rnh(struct rnh *rnh)
 {
        rnh->flags |= ZEBRA_NHT_DELETED;
-       list_delete_and_null(&rnh->client_list);
-       list_delete_and_null(&rnh->zebra_pseudowire_list);
+       list_delete(&rnh->client_list);
+       list_delete(&rnh->zebra_pseudowire_list);
        free_state(rnh->vrf_id, rnh->state, rnh->node);
        XFREE(MTYPE_RNH, rnh);
 }
@@ -212,10 +213,14 @@ void zebra_add_rnh_client(struct rnh *rnh, struct zserv *client,
                           zebra_route_string(client->proto),
                           rnh_str(rnh, buf, sizeof(buf)), type);
        }
-       if (!listnode_lookup(rnh->client_list, client)) {
+       if (!listnode_lookup(rnh->client_list, client))
                listnode_add(rnh->client_list, client);
-               send_client(rnh, client, type, vrf_id);
-       }
+
+       /*
+        * We always need to respond with known information,
+        * currently multiple daemons expect this behavior
+        */
+       send_client(rnh, client, type, vrf_id);
 }
 
 void zebra_remove_rnh_client(struct rnh *rnh, struct zserv *client,
@@ -260,13 +265,18 @@ void zebra_register_rnh_pseudowire(vrf_id_t vrf_id, struct zebra_pw *pw)
        struct prefix nh;
        struct rnh *rnh;
        bool exists;
+       struct zebra_vrf *zvrf;
+
+       zvrf = vrf_info_lookup(vrf_id);
+       if (!zvrf)
+               return;
 
        addr2hostprefix(pw->af, &pw->nexthop, &nh);
        rnh = zebra_add_rnh(&nh, vrf_id, RNH_NEXTHOP_TYPE, &exists);
        if (rnh && !listnode_lookup(rnh->zebra_pseudowire_list, pw)) {
                listnode_add(rnh->zebra_pseudowire_list, pw);
                pw->rnh = rnh;
-               zebra_evaluate_rnh(vrf_id, pw->af, 1, RNH_NEXTHOP_TYPE, &nh);
+               zebra_evaluate_rnh(zvrf, pw->af, 1, RNH_NEXTHOP_TYPE, &nh);
        }
 }
 
@@ -289,7 +299,8 @@ void zebra_deregister_rnh_pseudowire(vrf_id_t vrf_id, struct zebra_pw *pw)
 /* Apply the NHT route-map for a client to the route (and nexthops)
  * resolving a NH.
  */
-static int zebra_rnh_apply_nht_rmap(int family, struct route_node *prn,
+static int zebra_rnh_apply_nht_rmap(int family, struct zebra_vrf *zvrf,
+                                   struct route_node *prn,
                                    struct route_entry *re, int proto)
 {
        int at_least_one = 0;
@@ -302,8 +313,8 @@ static int zebra_rnh_apply_nht_rmap(int family, struct route_node *prn,
        if (prn && re) {
                for (nexthop = re->ng.nexthop; nexthop;
                     nexthop = nexthop->next) {
-                       ret = zebra_nht_route_map_check(rmap_family, proto,
-                                                       &prn->p, re, nexthop);
+                       ret = zebra_nht_route_map_check(
+                               rmap_family, proto, &prn->p, zvrf, re, nexthop);
                        if (ret != RMAP_DENYMATCH) {
                                SET_FLAG(nexthop->flags, NEXTHOP_FLAG_ACTIVE);
                                at_least_one++; /* at least one valid NH */
@@ -320,7 +331,7 @@ static int zebra_rnh_apply_nht_rmap(int family, struct route_node *prn,
  * for BGP route for import.
  */
 static struct route_entry *
-zebra_rnh_resolve_import_entry(vrf_id_t vrfid, int family,
+zebra_rnh_resolve_import_entry(struct zebra_vrf *zvrf, int family,
                               struct route_node *nrn, struct rnh *rnh,
                               struct route_node **prn)
 {
@@ -330,7 +341,7 @@ zebra_rnh_resolve_import_entry(vrf_id_t vrfid, int family,
 
        *prn = NULL;
 
-       route_table = zebra_vrf_table(family2afi(family), SAFI_UNICAST, vrfid);
+       route_table = zvrf->table[family2afi(family)][SAFI_UNICAST];
        if (!route_table) // unexpected
                return NULL;
 
@@ -402,11 +413,9 @@ static void zebra_rnh_eval_import_check_entry(vrf_id_t vrfid, int family,
 /*
  * Notify clients registered for this nexthop about a change.
  */
-static void zebra_rnh_notify_protocol_clients(vrf_id_t vrfid, int family,
-                                             struct route_node *nrn,
-                                             struct rnh *rnh,
-                                             struct route_node *prn,
-                                             struct route_entry *re)
+static void zebra_rnh_notify_protocol_clients(
+       struct zebra_vrf *zvrf, int family, struct route_node *nrn,
+       struct rnh *rnh, struct route_node *prn, struct route_entry *re)
 {
        struct listnode *node;
        struct zserv *client;
@@ -418,11 +427,11 @@ static void zebra_rnh_notify_protocol_clients(vrf_id_t vrfid, int family,
                prefix2str(&nrn->p, bufn, INET6_ADDRSTRLEN);
                if (prn && re) {
                        prefix2str(&prn->p, bufp, INET6_ADDRSTRLEN);
-                       zlog_debug("%u:%s: NH resolved over route %s", vrfid,
-                                  bufn, bufp);
+                       zlog_debug("%u:%s: NH resolved over route %s",
+                                  zvrf->vrf->vrf_id, bufn, bufp);
                } else
-                       zlog_debug("%u:%s: NH has become unresolved", vrfid,
-                                  bufn);
+                       zlog_debug("%u:%s: NH has become unresolved",
+                                  zvrf->vrf->vrf_id, bufn);
        }
 
        for (ALL_LIST_ELEMENTS_RO(rnh->client_list, node, client)) {
@@ -432,7 +441,7 @@ static void zebra_rnh_notify_protocol_clients(vrf_id_t vrfid, int family,
                         * nexthop to see if it is filtered or not.
                         */
                        num_resolving_nh = zebra_rnh_apply_nht_rmap(
-                               family, prn, re, client->proto);
+                               family, zvrf, prn, re, client->proto);
                        if (num_resolving_nh)
                                rnh->filtered[client->proto] = 0;
                        else
@@ -441,7 +450,7 @@ static void zebra_rnh_notify_protocol_clients(vrf_id_t vrfid, int family,
                        if (IS_ZEBRA_DEBUG_NHT)
                                zlog_debug(
                                        "%u:%s: Notifying client %s about NH %s",
-                                       vrfid, bufn,
+                                       zvrf->vrf->vrf_id, bufn,
                                        zebra_route_string(client->proto),
                                        num_resolving_nh
                                                ? ""
@@ -451,11 +460,11 @@ static void zebra_rnh_notify_protocol_clients(vrf_id_t vrfid, int family,
                        if (IS_ZEBRA_DEBUG_NHT)
                                zlog_debug(
                                        "%u:%s: Notifying client %s about NH (unreachable)",
-                                       vrfid, bufn,
+                                       zvrf->vrf->vrf_id, bufn,
                                        zebra_route_string(client->proto));
                }
 
-               send_client(rnh, client, RNH_NEXTHOP_TYPE, vrfid);
+               send_client(rnh, client, RNH_NEXTHOP_TYPE, zvrf->vrf->vrf_id);
        }
 }
 
@@ -465,12 +474,11 @@ static void zebra_rnh_process_pbr_tables(int family,
                                         struct route_node *prn,
                                         struct route_entry *re)
 {
-       struct zebra_ns_table *znst;
+       struct zebra_router_table *zrt;
        struct route_entry *o_re;
        struct route_node *o_rn;
        struct listnode *node;
        struct zserv *client;
-       struct zebra_ns *zns;
        afi_t afi = AFI_IP;
 
        if (family == AF_INET6)
@@ -488,13 +496,12 @@ static void zebra_rnh_process_pbr_tables(int family,
        if (!client)
                return;
 
-       zns = zebra_ns_lookup(NS_DEFAULT);
-       RB_FOREACH (znst, zebra_ns_table_head, &zns->ns_tables) {
-               if (afi != znst->afi)
+       RB_FOREACH (zrt, zebra_router_table_head, &zrouter.tables) {
+               if (afi != zrt->afi)
                        continue;
 
-               for (o_rn = route_top(znst->table);
-                    o_rn; o_rn = srcdest_route_next(o_rn)) {
+               for (o_rn = route_top(zrt->table); o_rn;
+                    o_rn = srcdest_route_next(o_rn)) {
                        RNODE_FOREACH_RE (o_rn, o_re) {
                                if (o_re->type == ZEBRA_ROUTE_PBR)
                                        break;
@@ -514,22 +521,34 @@ static void zebra_rnh_process_pbr_tables(int family,
        }
 }
 
+/*
+ * Utility to determine whether a candidate nexthop is useable. We make this
+ * check in a couple of places, so this is a single home for the logic we
+ * use.
+ */
+static bool rnh_nexthop_valid(const struct nexthop *nh)
+{
+       return (CHECK_FLAG(nh->flags, NEXTHOP_FLAG_FIB)
+               && CHECK_FLAG(nh->flags, NEXTHOP_FLAG_ACTIVE));
+}
+
 /*
  * Determine appropriate route (route entry) resolving a tracked
  * nexthop.
  */
 static struct route_entry *
-zebra_rnh_resolve_nexthop_entry(vrf_id_t vrfid, int family,
+zebra_rnh_resolve_nexthop_entry(struct zebra_vrf *zvrf, int family,
                                struct route_node *nrn, struct rnh *rnh,
                                struct route_node **prn)
 {
        struct route_table *route_table;
        struct route_node *rn;
        struct route_entry *re;
+       struct nexthop *nexthop;
 
        *prn = NULL;
 
-       route_table = zebra_vrf_table(family2afi(family), SAFI_UNICAST, vrfid);
+       route_table = zvrf->table[family2afi(family)][SAFI_UNICAST];
        if (!route_table)
                return NULL;
 
@@ -558,12 +577,22 @@ zebra_rnh_resolve_nexthop_entry(vrf_id_t vrfid, int family,
                        if (!CHECK_FLAG(re->flags, ZEBRA_FLAG_SELECTED))
                                continue;
 
+                       /* Just being SELECTED isn't quite enough - must
+                        * have an installed nexthop to be useful.
+                        */
+                       for (ALL_NEXTHOPS(re->ng, nexthop)) {
+                               if (rnh_nexthop_valid(nexthop))
+                                       break;
+                       }
+
+                       if (nexthop == NULL)
+                               continue;
+
                        if (CHECK_FLAG(rnh->flags, ZEBRA_NHT_CONNECTED)) {
                                if ((re->type == ZEBRA_ROUTE_CONNECT)
                                    || (re->type == ZEBRA_ROUTE_STATIC))
                                        break;
                                if (re->type == ZEBRA_ROUTE_NHRP) {
-                                       struct nexthop *nexthop;
 
                                        for (nexthop = re->ng.nexthop; nexthop;
                                             nexthop = nexthop->next)
@@ -606,8 +635,8 @@ static void zebra_rnh_process_pseudowires(vrf_id_t vrfid, struct rnh *rnh)
  * take appropriate action; this involves notifying any clients and/or
  * scheduling dependent static routes for processing.
  */
-static void zebra_rnh_eval_nexthop_entry(vrf_id_t vrfid, int family, int force,
-                                        struct route_node *nrn,
+static void zebra_rnh_eval_nexthop_entry(struct zebra_vrf *zvrf, int family,
+                                        int force, struct route_node *nrn,
                                         struct rnh *rnh,
                                         struct route_node *prn,
                                         struct route_entry *re)
@@ -636,20 +665,20 @@ static void zebra_rnh_eval_nexthop_entry(vrf_id_t vrfid, int family, int force,
                 * rnh->state.
                 */
                /* Notify registered protocol clients. */
-               zebra_rnh_notify_protocol_clients(vrfid, family, nrn, rnh, prn,
+               zebra_rnh_notify_protocol_clients(zvrf, family, nrn, rnh, prn,
                                                  rnh->state);
 
-               zebra_rnh_process_pbr_tables(family, nrn, rnh, prn,
-                                            rnh->state);
+               zebra_rnh_process_pbr_tables(family, nrn, rnh, prn, rnh->state);
 
                /* Process pseudowires attached to this nexthop */
-               zebra_rnh_process_pseudowires(vrfid, rnh);
+               zebra_rnh_process_pseudowires(zvrf->vrf->vrf_id, rnh);
        }
 }
 
 /* Evaluate one tracked entry */
-static void zebra_rnh_evaluate_entry(vrf_id_t vrfid, int family, int force,
-                                    rnh_type_t type, struct route_node *nrn)
+static void zebra_rnh_evaluate_entry(struct zebra_vrf *zvrf, int family,
+                                    int force, rnh_type_t type,
+                                    struct route_node *nrn)
 {
        struct rnh *rnh;
        struct route_entry *re;
@@ -658,18 +687,18 @@ static void zebra_rnh_evaluate_entry(vrf_id_t vrfid, int family, int force,
 
        if (IS_ZEBRA_DEBUG_NHT) {
                prefix2str(&nrn->p, bufn, INET6_ADDRSTRLEN);
-               zlog_debug("%u:%s: Evaluate RNH, type %d %s", vrfid, bufn, type,
-                          force ? "(force)" : "");
+               zlog_debug("%u:%s: Evaluate RNH, type %d %s", zvrf->vrf->vrf_id,
+                          bufn, type, force ? "(force)" : "");
        }
 
        rnh = nrn->info;
 
        /* Identify route entry (RE) resolving this tracked entry. */
        if (type == RNH_IMPORT_CHECK_TYPE)
-               re = zebra_rnh_resolve_import_entry(vrfid, family, nrn, rnh,
+               re = zebra_rnh_resolve_import_entry(zvrf, family, nrn, rnh,
                                                    &prn);
        else
-               re = zebra_rnh_resolve_nexthop_entry(vrfid, family, nrn, rnh,
+               re = zebra_rnh_resolve_nexthop_entry(zvrf, family, nrn, rnh,
                                                     &prn);
 
        /* If the entry cannot be resolved and that is also the existing state,
@@ -680,11 +709,11 @@ static void zebra_rnh_evaluate_entry(vrf_id_t vrfid, int family, int force,
 
        /* Process based on type of entry. */
        if (type == RNH_IMPORT_CHECK_TYPE)
-               zebra_rnh_eval_import_check_entry(vrfid, family, force, nrn,
-                                                 rnh, re);
+               zebra_rnh_eval_import_check_entry(zvrf->vrf->vrf_id, family,
+                                                 force, nrn, rnh, re);
        else
-               zebra_rnh_eval_nexthop_entry(vrfid, family, force, nrn, rnh,
-                                            prn, re);
+               zebra_rnh_eval_nexthop_entry(zvrf, family, force, nrn, rnh, prn,
+                                            re);
 }
 
 /*
@@ -696,7 +725,7 @@ static void zebra_rnh_evaluate_entry(vrf_id_t vrfid, int family, int force,
  * we can have a situation where one re entry
  * covers multiple nexthops we are interested in.
  */
-static void zebra_rnh_clear_nhc_flag(vrf_id_t vrfid, int family,
+static void zebra_rnh_clear_nhc_flag(struct zebra_vrf *zvrf, int family,
                                     rnh_type_t type, struct route_node *nrn)
 {
        struct rnh *rnh;
@@ -707,10 +736,10 @@ static void zebra_rnh_clear_nhc_flag(vrf_id_t vrfid, int family,
 
        /* Identify route entry (RIB) resolving this tracked entry. */
        if (type == RNH_IMPORT_CHECK_TYPE)
-               re = zebra_rnh_resolve_import_entry(vrfid, family, nrn, rnh,
+               re = zebra_rnh_resolve_import_entry(zvrf, family, nrn, rnh,
                                                    &prn);
        else
-               re = zebra_rnh_resolve_nexthop_entry(vrfid, family, nrn, rnh,
+               re = zebra_rnh_resolve_nexthop_entry(zvrf, family, nrn, rnh,
                                                     &prn);
 
        if (re) {
@@ -722,13 +751,13 @@ static void zebra_rnh_clear_nhc_flag(vrf_id_t vrfid, int family,
 /* Evaluate all tracked entries (nexthops or routes for import into BGP)
  * of a particular VRF and address-family or a specific prefix.
  */
-void zebra_evaluate_rnh(vrf_id_t vrfid, int family, int force, rnh_type_t type,
-                       struct prefix *p)
+void zebra_evaluate_rnh(struct zebra_vrf *zvrf, int family, int force,
+                       rnh_type_t type, struct prefix *p)
 {
        struct route_table *rnh_table;
        struct route_node *nrn;
 
-       rnh_table = get_rnh_table(vrfid, family, type);
+       rnh_table = get_rnh_table(zvrf->vrf->vrf_id, family, type);
        if (!rnh_table) // unexpected
                return;
 
@@ -736,7 +765,7 @@ void zebra_evaluate_rnh(vrf_id_t vrfid, int family, int force, rnh_type_t type,
                /* Evaluating a specific entry, make sure it exists. */
                nrn = route_node_lookup(rnh_table, p);
                if (nrn && nrn->info)
-                       zebra_rnh_evaluate_entry(vrfid, family, force, type,
+                       zebra_rnh_evaluate_entry(zvrf, family, force, type,
                                                 nrn);
 
                if (nrn)
@@ -746,14 +775,14 @@ void zebra_evaluate_rnh(vrf_id_t vrfid, int family, int force, rnh_type_t type,
                nrn = route_top(rnh_table);
                while (nrn) {
                        if (nrn->info)
-                               zebra_rnh_evaluate_entry(vrfid, family, force,
+                               zebra_rnh_evaluate_entry(zvrf, family, force,
                                                         type, nrn);
                        nrn = route_next(nrn); /* this will also unlock nrn */
                }
                nrn = route_top(rnh_table);
                while (nrn) {
                        if (nrn->info)
-                               zebra_rnh_clear_nhc_flag(vrfid, family, type,
+                               zebra_rnh_clear_nhc_flag(zvrf, family, type,
                                                         nrn);
                        nrn = route_next(nrn); /* this will also unlock nrn */
                }
@@ -884,10 +913,9 @@ static int send_client(struct rnh *rnh, struct zserv *client, rnh_type_t type,
                num = 0;
                nump = stream_get_endp(s);
                stream_putc(s, 0);
-               for (nh = re->ng.nexthop; nh; nh = nh->next)
-                       if ((CHECK_FLAG(nh->flags, NEXTHOP_FLAG_FIB)
-                            || CHECK_FLAG(nh->flags, NEXTHOP_FLAG_RECURSIVE))
-                           && CHECK_FLAG(nh->flags, NEXTHOP_FLAG_ACTIVE)) {
+               for (ALL_NEXTHOPS(re->ng, nh))
+                       if (rnh_nexthop_valid(nh)) {
+                               stream_putl(s, nh->vrf_id);
                                stream_putc(s, nh->type);
                                switch (nh->type) {
                                case NEXTHOP_TYPE_IPV4: