]> git.proxmox.com Git - mirror_frr.git/blobdiff - zebra/rt_socket.c
Merge pull request #3384 from donaldsharp/topotests
[mirror_frr.git] / zebra / rt_socket.c
index 18328d380900d9b030d84cc999661d181fc69ee5..19d41a402e7fb013c9edc92fb605cdfc834e4594 100644 (file)
@@ -91,7 +91,7 @@ static int kernel_rtm_add_labels(struct mpls_label_stack *nh_label,
 
 /* Interface between zebra message and rtm message. */
 static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
-                          struct route_entry *re)
+                          const struct nexthop_group *ng, uint32_t metric)
 
 {
        struct sockaddr_in *mask = NULL;
@@ -126,7 +126,7 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
 #endif /* HAVE_STRUCT_SOCKADDR_IN_SIN_LEN */
 
        /* Make gateway. */
-       for (ALL_NEXTHOPS(re->ng, nexthop)) {
+       for (ALL_NEXTHOPS_PTR(ng, nexthop)) {
                if (CHECK_FLAG(nexthop->flags, NEXTHOP_FLAG_RECURSIVE))
                        continue;
 
@@ -139,8 +139,7 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
                 * other than ADD and DELETE?
                 */
                if ((cmd == RTM_ADD && NEXTHOP_IS_ACTIVE(nexthop->flags))
-                   || (cmd == RTM_DELETE
-                       && CHECK_FLAG(nexthop->flags, NEXTHOP_FLAG_FIB))) {
+                   || (cmd == RTM_DELETE)) {
                        if (nexthop->type == NEXTHOP_TYPE_IPV4
                            || nexthop->type == NEXTHOP_TYPE_IPV4_IFINDEX) {
                                sin_gate.sin_addr = nexthop->gate.ipv4;
@@ -181,14 +180,13 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
                                          (union sockunion *)mask,
                                          gate ? (union sockunion *)&sin_gate
                                               : NULL,
-                                         smplsp, ifindex, bh_type, re->metric);
+                                         smplsp, ifindex, bh_type, metric);
 
-                       if (IS_ZEBRA_DEBUG_RIB) {
+                       if (IS_ZEBRA_DEBUG_KERNEL) {
                                if (!gate) {
                                        zlog_debug(
-                                               "%s: %s: attention! gate not found for re %p",
-                                               __func__, prefix_buf, re);
-                                       route_entry_dump(p, NULL, re);
+                                               "%s: %s: attention! gate not found for re",
+                                               __func__, prefix_buf);
                                } else
                                        inet_ntop(AF_INET, &sin_gate.sin_addr,
                                                  gate_buf, INET_ADDRSTRLEN);
@@ -199,10 +197,15 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
                         * did its work. */
                        case ZEBRA_ERR_NOERROR:
                                nexthop_num++;
-                               if (IS_ZEBRA_DEBUG_RIB)
+                               if (IS_ZEBRA_DEBUG_KERNEL)
                                        zlog_debug(
                                                "%s: %s: successfully did NH %s",
                                                __func__, prefix_buf, gate_buf);
+
+                               if (cmd == RTM_ADD)
+                                       SET_FLAG(nexthop->flags,
+                                                NEXTHOP_FLAG_FIB);
+
                                break;
 
                        /* The only valid case for this error is kernel's
@@ -214,21 +217,15 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
                        case ZEBRA_ERR_RTEXIST:
                                if (cmd != RTM_ADD)
                                        flog_err(
-                                               LIB_ERR_SYSTEM_CALL,
+                                               EC_LIB_SYSTEM_CALL,
                                                "%s: rtm_write() returned %d for command %d",
                                                __func__, error, cmd);
                                continue;
-                               break;
 
-                       /* Given that our NEXTHOP_FLAG_FIB matches real kernel
-                        * FIB, it isn't
-                        * normal to get any other messages in ANY case.
-                        */
-                       case ZEBRA_ERR_RTNOEXIST:
-                       case ZEBRA_ERR_RTUNREACH:
+                       /* Note any unexpected status returns */
                        default:
                                flog_err(
-                                       LIB_ERR_SYSTEM_CALL,
+                                       EC_LIB_SYSTEM_CALL,
                                        "%s: %s: rtm_write() unexpectedly returned %d for command %s",
                                        __func__,
                                        prefix2str(p, prefix_buf,
@@ -238,16 +235,20 @@ static int kernel_rtm_ipv4(int cmd, const struct prefix *p,
                                break;
                        }
                } /* if (cmd and flags make sense) */
-               else if (IS_ZEBRA_DEBUG_RIB)
+               else if (IS_ZEBRA_DEBUG_KERNEL)
                        zlog_debug("%s: odd command %s for flags %d", __func__,
                                   lookup_msg(rtm_type_str, cmd, NULL),
                                   nexthop->flags);
        } /* for (ALL_NEXTHOPS(...))*/
 
        /* If there was no useful nexthop, then complain. */
-       if (nexthop_num == 0 && IS_ZEBRA_DEBUG_KERNEL)
-               zlog_debug("%s: No useful nexthops were found in RIB entry %p",
-                          __func__, re);
+       if (nexthop_num == 0) {
+               if (IS_ZEBRA_DEBUG_KERNEL)
+                       zlog_debug("%s: No useful nexthops were found in RIB prefix %s",
+                                  __func__, prefix2str(p, prefix_buf,
+                                                       sizeof(prefix_buf)));
+               return 1;
+       }
 
        return 0; /*XXX*/
 }
@@ -278,7 +279,7 @@ static int sin6_masklen(struct in6_addr mask)
 
 /* Interface between zebra message and rtm message. */
 static int kernel_rtm_ipv6(int cmd, const struct prefix *p,
-                          struct route_entry *re)
+                          const struct nexthop_group *ng, uint32_t metric)
 {
        struct sockaddr_in6 *mask;
        struct sockaddr_in6 sin_dest, sin_mask, sin_gate;
@@ -309,7 +310,7 @@ static int kernel_rtm_ipv6(int cmd, const struct prefix *p,
 #endif /* HAVE_STRUCT_SOCKADDR_IN_SIN_LEN */
 
        /* Make gateway. */
-       for (ALL_NEXTHOPS(re->ng, nexthop)) {
+       for (ALL_NEXTHOPS_PTR(ng, nexthop)) {
                if (CHECK_FLAG(nexthop->flags, NEXTHOP_FLAG_RECURSIVE))
                        continue;
 
@@ -364,8 +365,11 @@ static int kernel_rtm_ipv6(int cmd, const struct prefix *p,
                error = rtm_write(cmd, (union sockunion *)&sin_dest,
                                  (union sockunion *)mask,
                                  gate ? (union sockunion *)&sin_gate : NULL,
-                                 smplsp, ifindex, bh_type, re->metric);
-               (void)error;
+                                 smplsp, ifindex, bh_type, metric);
+
+               /* Update installed nexthop info on success */
+               if ((cmd == RTM_ADD) && (error == ZEBRA_ERR_NOERROR))
+                       SET_FLAG(nexthop->flags, NEXTHOP_FLAG_FIB);
 
                nexthop_num++;
        }
@@ -374,60 +378,70 @@ static int kernel_rtm_ipv6(int cmd, const struct prefix *p,
        if (nexthop_num == 0) {
                if (IS_ZEBRA_DEBUG_KERNEL)
                        zlog_debug("kernel_rtm_ipv6(): No useful nexthop.");
-               return 0;
+               return 1;
        }
 
        return 0; /*XXX*/
 }
 
-static int kernel_rtm(int cmd, const struct prefix *p, struct route_entry *re)
+static int kernel_rtm(int cmd, const struct prefix *p,
+                     const struct nexthop_group *ng, uint32_t metric)
 {
        switch (PREFIX_FAMILY(p)) {
        case AF_INET:
-               return kernel_rtm_ipv4(cmd, p, re);
+               return kernel_rtm_ipv4(cmd, p, ng, metric);
        case AF_INET6:
-               return kernel_rtm_ipv6(cmd, p, re);
+               return kernel_rtm_ipv6(cmd, p, ng, metric);
        }
        return 0;
 }
 
-enum dp_req_result kernel_route_rib(struct route_node *rn,
                                  const struct prefix *p,
-                                   const struct prefix *src_p,
-                                   struct route_entry *old,
-                                   struct route_entry *new)
+/*
* Update or delete a prefix from the kernel,
+ * using info from a dataplane context struct.
+ */
+enum zebra_dplane_result kernel_route_update(struct zebra_dplane_ctx *ctx)
 {
-       int route = 0;
+       enum zebra_dplane_result res = ZEBRA_DPLANE_REQUEST_SUCCESS;
 
-       if (src_p && src_p->prefixlen) {
-               flog_warn(EC_ZEBRA_UNSUPPORTED_V6_SRCDEST,
-                         "%s: IPv6 sourcedest routes unsupported!", __func__);
-               return DP_REQUEST_FAILURE;
+       if (dplane_ctx_get_src(ctx) != NULL) {
+               zlog_err("route add: IPv6 sourcedest routes unsupported!");
+               res = ZEBRA_DPLANE_REQUEST_FAILURE;
+               goto done;
        }
 
        frr_elevate_privs(&zserv_privs) {
 
-               if (old)
-                       route |= kernel_rtm(RTM_DELETE, p, old);
-
-               if (new)
-                       route |= kernel_rtm(RTM_ADD, p, new);
-
-       }
+               if (dplane_ctx_get_op(ctx) == DPLANE_OP_ROUTE_DELETE)
+                       kernel_rtm(RTM_DELETE, dplane_ctx_get_dest(ctx),
+                                  dplane_ctx_get_ng(ctx),
+                                  dplane_ctx_get_metric(ctx));
+               else if (dplane_ctx_get_op(ctx) == DPLANE_OP_ROUTE_INSTALL)
+                       kernel_rtm(RTM_ADD, dplane_ctx_get_dest(ctx),
+                                  dplane_ctx_get_ng(ctx),
+                                  dplane_ctx_get_metric(ctx));
+               else if (dplane_ctx_get_op(ctx) == DPLANE_OP_ROUTE_UPDATE) {
+                       /* Must do delete and add separately -
+                        * no update available
+                        */
+                       kernel_rtm(RTM_DELETE, dplane_ctx_get_dest(ctx),
+                                  dplane_ctx_get_old_ng(ctx),
+                                  dplane_ctx_get_old_metric(ctx));
+
+                       kernel_rtm(RTM_ADD, dplane_ctx_get_dest(ctx),
+                                  dplane_ctx_get_ng(ctx),
+                                  dplane_ctx_get_metric(ctx));
+               } else {
+                       zlog_err("Invalid routing socket update op %s (%u)",
+                                dplane_op2str(dplane_ctx_get_op(ctx)),
+                                dplane_ctx_get_op(ctx));
+                       res = ZEBRA_DPLANE_REQUEST_FAILURE;
+               }
+       } /* Elevated privs */
 
-       if (new) {
-               kernel_route_rib_pass_fail(
-                       rn, p, new,
-                       (!route) ? DP_INSTALL_SUCCESS
-                                : DP_INSTALL_FAILURE);
-       } else {
-               kernel_route_rib_pass_fail(rn, p, old,
-                                          (!route)
-                                                  ? DP_DELETE_SUCCESS
-                                                  : DP_DELETE_FAILURE);
-       }
+done:
 
-       return DP_REQUEST_SUCCESS;
+       return res;
 }
 
 int kernel_neigh_update(int add, int ifindex, uint32_t addr, char *lla,
@@ -453,13 +467,13 @@ int kernel_del_vtep(vni_t vni, struct interface *ifp, struct in_addr *vtep_ip)
 }
 
 int kernel_add_mac(struct interface *ifp, vlanid_t vid, struct ethaddr *mac,
-                  struct in_addr vtep_ip, uint8_t sticky)
+                  struct in_addr vtep_ip, bool sticky)
 {
        return 0;
 }
 
 int kernel_del_mac(struct interface *ifp, vlanid_t vid, struct ethaddr *mac,
-                  struct in_addr vtep_ip, int local)
+                  struct in_addr vtep_ip)
 {
        return 0;
 }