]> git.puffer.fish Git - matthieu/frr.git/commitdiff
Revert "zebra: Add connected with noprefixroute"
authorDonatas Abraitis <donatas@opensourcerouting.org>
Sun, 21 Jan 2024 21:11:43 +0000 (23:11 +0200)
committerDonatas Abraitis <donatas@opensourcerouting.org>
Sun, 21 Jan 2024 21:11:43 +0000 (23:11 +0200)
This reverts commit 6e207024fed2144e569629598b830d11955774ec.

lib/if.h
zebra/connected.c
zebra/interface.c

index a60486945214fe5ef464aa05d39b0177fe6663b1..c6b4fd216a371252c9391c0bccab48d17c9d7373 100644 (file)
--- a/lib/if.h
+++ b/lib/if.h
@@ -417,8 +417,6 @@ struct connected {
 #define ZEBRA_IFA_SECONDARY    (1 << 0)
 #define ZEBRA_IFA_PEER         (1 << 1)
 #define ZEBRA_IFA_UNNUMBERED   (1 << 2)
-#define ZEBRA_IFA_NOPREFIXROUTE (1 << 3)
-
        /* N.B. the ZEBRA_IFA_PEER flag should be set if and only if
           a peer address has been configured.  If this flag is set,
           the destination field must contain the peer address.
index 2977aebfbcd76e86bcab07c2e53935a9626ef1fc..ee0823f56f458ac8ebbf7427c2ace23878af5baa 100644 (file)
@@ -276,15 +276,13 @@ void connected_up(struct interface *ifp, struct connected *ifc)
                        return;
        }
 
-       if (!CHECK_FLAG(ifc->flags, ZEBRA_IFA_NOPREFIXROUTE)) {
-               rib_add(afi, SAFI_UNICAST, zvrf->vrf->vrf_id,
-                       ZEBRA_ROUTE_CONNECT, 0, flags, &p, NULL, &nh, 0,
-                       zvrf->table_id, metric, 0, 0, 0, false);
-
-               rib_add(afi, SAFI_MULTICAST, zvrf->vrf->vrf_id,
-                       ZEBRA_ROUTE_CONNECT, 0, flags, &p, NULL, &nh, 0,
-                       zvrf->table_id, metric, 0, 0, 0, false);
-       }
+       rib_add(afi, SAFI_UNICAST, zvrf->vrf->vrf_id, ZEBRA_ROUTE_CONNECT, 0,
+               flags, &p, NULL, &nh, 0, zvrf->table_id, metric, 0, 0, 0,
+               false);
+
+       rib_add(afi, SAFI_MULTICAST, zvrf->vrf->vrf_id, ZEBRA_ROUTE_CONNECT, 0,
+               flags, &p, NULL, &nh, 0, zvrf->table_id, metric, 0, 0, 0,
+               false);
 
        /* Schedule LSP forwarding entries for processing, if appropriate. */
        if (zvrf->vrf->vrf_id == VRF_DEFAULT) {
@@ -459,15 +457,11 @@ void connected_down(struct interface *ifp, struct connected *ifc)
         * Same logic as for connected_up(): push the changes into the
         * head.
         */
-       if (!CHECK_FLAG(ifc->flags, ZEBRA_IFA_NOPREFIXROUTE)) {
-               rib_delete(afi, SAFI_UNICAST, zvrf->vrf->vrf_id,
-                          ZEBRA_ROUTE_CONNECT, 0, 0, &p, NULL, &nh, 0,
-                          zvrf->table_id, 0, 0, false);
-
-               rib_delete(afi, SAFI_MULTICAST, zvrf->vrf->vrf_id,
-                          ZEBRA_ROUTE_CONNECT, 0, 0, &p, NULL, &nh, 0,
-                          zvrf->table_id, 0, 0, false);
-       }
+       rib_delete(afi, SAFI_UNICAST, zvrf->vrf->vrf_id, ZEBRA_ROUTE_CONNECT, 0,
+                  0, &p, NULL, &nh, 0, zvrf->table_id, 0, 0, false);
+
+       rib_delete(afi, SAFI_MULTICAST, zvrf->vrf->vrf_id, ZEBRA_ROUTE_CONNECT,
+                  0, 0, &p, NULL, &nh, 0, zvrf->table_id, 0, 0, false);
 
        /* Schedule LSP forwarding entries for processing, if appropriate. */
        if (zvrf->vrf->vrf_id == VRF_DEFAULT) {
index 6b16969bb9db158ee8fa99747031a10d005d9282..952a1aca8092f998a2888a24ec6a74dbcf9ee646 100644 (file)
@@ -1311,9 +1311,6 @@ static void zebra_if_addr_update_ctx(struct zebra_dplane_ctx *ctx,
        if (dplane_ctx_intf_is_secondary(ctx))
                SET_FLAG(flags, ZEBRA_IFA_SECONDARY);
 
-       if (dplane_ctx_intf_is_noprefixroute(ctx))
-               SET_FLAG(flags, ZEBRA_IFA_NOPREFIXROUTE);
-
        /* Label? */
        if (dplane_ctx_intf_has_label(ctx))
                label = dplane_ctx_get_intf_label(ctx);
@@ -1625,12 +1622,6 @@ static void connected_dump_vty(struct vty *vty, json_object *json,
        else if (CHECK_FLAG(connected->flags, ZEBRA_IFA_SECONDARY))
                vty_out(vty, " secondary");
 
-       if (json)
-               json_object_boolean_add(json_addr, "noPrefixRoute",
-                                       CHECK_FLAG(connected->flags, ZEBRA_IFA_NOPREFIXROUTE));
-       else if (CHECK_FLAG(connected->flags, ZEBRA_IFA_NOPREFIXROUTE))
-               vty_out(vty, " noprefixroute");
-
        if (json)
                json_object_boolean_add(
                        json_addr, "unnumbered",