aboutsummaryrefslogtreecommitdiffstats
path: root/router
diff options
context:
space:
mode:
authorHongjun Ni <hongjun.ni@intel.com>2018-08-07 23:40:48 +0800
committerHongjun Ni <hongjun.ni@intel.com>2018-08-21 19:04:54 +0800
commit31a24b656050878af31122b085a345cfe35e742b (patch)
treea62a43d41827d8316f22674c83a6fe0f45700371 /router
parent38f03b2d6ca0a83846171a0f57b0ae3834f6c2a5 (diff)
Remove FIB_1.0 support and fix FIB_2.0 issue
Change-Id: I90e593993851a364b728034c13e8e69852b558c4 Signed-off-by: Hongjun Ni <hongjun.ni@intel.com>
Diffstat (limited to 'router')
-rw-r--r--router/Makefile.am13
-rw-r--r--router/router/tap_inject.c36
-rw-r--r--router/router/tap_inject.h1
-rw-r--r--router/router/tap_inject_netlink.c40
4 files changed, 12 insertions, 78 deletions
diff --git a/router/Makefile.am b/router/Makefile.am
index b9de6a1..a4387dd 100644
--- a/router/Makefile.am
+++ b/router/Makefile.am
@@ -3,10 +3,15 @@ AUTOMAKE_OPTIONS = foreign subdir-objects
AM_CFLAGS = -Wall -I@TOOLKIT_INCLUDE@
lib_LTLIBRARIES = router.la
-router_la_SOURCES = router/tap_inject.c \
- router/tap_inject_netlink.c \
- router/tap_inject_node.c \
- router/tap_inject_tap.c
+router_la_SOURCES = \
+ router/tap_inject.c \
+ router/tap_inject_netlink.c \
+ router/tap_inject_node.c \
+ router/tap_inject_tap.c
+
+nobase_include_HEADERS = \
+ router/tap_inject.h
+
router_la_LDFLAGS = -module
router_la_LIBADD = -lrtnl
diff --git a/router/router/tap_inject.c b/router/router/tap_inject.c
index e8d3f98..f41ae86 100644
--- a/router/router/tap_inject.c
+++ b/router/router/tap_inject.c
@@ -19,12 +19,7 @@
#include <vnet/mfib/mfib_table.h>
#include <vnet/ip/ip.h>
#include <vnet/ip/lookup.h>
-#ifdef ip6_add_del_route_next_hop
-#define FIB_VERSION 1
-#else
#include <vnet/fib/fib.h>
-#define FIB_VERSION 2
-#endif
static tap_inject_main_t tap_inject_main;
extern dpo_type_t tap_inject_dpo_type;
@@ -146,36 +141,6 @@ tap_inject_enable (void)
ip6_register_protocol (IP_PROTOCOL_TCP, im->tx_node_index);
ip6_register_protocol (IP_PROTOCOL_UDP, im->tx_node_index);
-#if FIB_VERSION == 1
- /* Add IPv4 multicast route. */
- {
- ip4_add_del_route_args_t a;
- ip_adjacency_t add_adj;
- u32 next_node_index;
-
- memset (&a, 0, sizeof (a));
- memset (&add_adj, 0, sizeof (add_adj));
-
- a.add_adj = &add_adj;
- a.n_add_adj = 1;
-
- a.flags = IP4_ROUTE_FLAG_TABLE_ID | IP4_ROUTE_FLAG_ADD;
- a.table_index_or_table_id = 0;
- a.dst_address.as_u32 = 0x000000E0; /* 224.0.0.0 */
- a.dst_address_length = 24;
- a.adj_index = ~0;
-
- next_node_index = vlib_node_add_next (vm, ip4_lookup_node.index,
- im->tx_node_index);
-
- add_adj.explicit_fib_index = ~0;
- add_adj.rewrite_header.node_index = ip4_rewrite_node.index;
- add_adj.lookup_next_index = next_node_index;
- add_adj.if_address_index = ~0;
-
- ip4_add_del_route (&ip4_main, &a);
- }
-#else
{
dpo_id_t dpo = DPO_INVALID;
@@ -203,7 +168,6 @@ tap_inject_enable (void)
dpo_reset(&dpo);
}
-#endif /* FIB_VERSION == 1 */
im->flags |= TAP_INJECT_F_ENABLED;
diff --git a/router/router/tap_inject.h b/router/router/tap_inject.h
index 001ab52..ec5121a 100644
--- a/router/router/tap_inject.h
+++ b/router/router/tap_inject.h
@@ -20,7 +20,6 @@
#include <vnet/plugin/plugin.h>
#include <vnet/ip/ip.h>
-
#ifndef ETHER_ADDR_LEN
#define ETHER_ADDR_LEN 6
#endif
diff --git a/router/router/tap_inject_netlink.c b/router/router/tap_inject_netlink.c
index 3dca013..bd5da49 100644
--- a/router/router/tap_inject_netlink.c
+++ b/router/router/tap_inject_netlink.c
@@ -14,25 +14,18 @@
* limitations under the License.
*/
-#include "tap_inject.h"
#include <librtnl/netns.h>
#include <vlibmemory/api.h>
-#include <vnet/ethernet/arp_packet.h>
#include <vnet/ip/ip6_neighbor.h>
-
-#include <vnet/ip/ip.h>
#include <vnet/ip/lookup.h>
-#ifdef ip6_add_del_route_next_hop
-#define FIB_VERSION 1
-#else
#include <vnet/fib/fib.h>
-#define FIB_VERSION 2
-#endif
-
+#include <vnet/ethernet/arp.h>
#include <arpa/inet.h>
#include <linux/mpls.h>
#include <vnet/mpls/packet.h>
+#include "tap_inject.h"
+
static void
add_del_addr (ns_addr_t * a, int is_del)
{
@@ -124,37 +117,24 @@ add_del_neigh (ns_neigh_t * n, int is_del)
if (n->nd.ndm_state & NUD_REACHABLE)
{
-#if FIB_VERSION == 1
- vnet_arp_set_ip4_over_ethernet (vnet_main, sw_if_index, ~0, &a, 0);
-#else
vnet_arp_set_ip4_over_ethernet (vnet_main, sw_if_index,
&a, 0 /* static */ ,
0 /* no fib entry */);
-#endif /* FIB_VERSION == 1 */
}
else if (n->nd.ndm_state & NUD_FAILED)
{
-#if FIB_VERSION == 1
- vnet_arp_unset_ip4_over_ethernet (vnet_main, sw_if_index, ~0, &a);
-#else
vnet_arp_unset_ip4_over_ethernet (vnet_main, sw_if_index, &a);
-#endif /* FIB_VERSION == 1 */
}
}
else if (n->nd.ndm_family == AF_INET6)
{
if (n->nd.ndm_state & NUD_REACHABLE)
{
-#if FIB_VERSION == 1
- vnet_set_ip6_ethernet_neighbor (vm, sw_if_index,
- (ip6_address_t *) n->dst, n->lladdr, ETHER_ADDR_LEN, 0);
-#else
vnet_set_ip6_ethernet_neighbor (vm, sw_if_index,
(ip6_address_t *) n->dst, n->lladdr, ETHER_ADDR_LEN,
0 /* static */,
0 /* no fib entry */);
-#endif /* FIB_VERSION == 1 */
}
else
vnet_unset_ip6_ethernet_neighbor (vm, sw_if_index,
@@ -194,12 +174,6 @@ add_del_route (ns_route_t * r, int is_del)
{
u32 stack[MPLS_STACK_DEPTH] = {0};
-#if FIB_VERSION == 1
- ip4_add_del_route_next_hop (&ip4_main,
- is_del ? IP4_ROUTE_FLAG_DEL : IP4_ROUTE_FLAG_ADD,
- (ip4_address_t *) r->dst, r->rtm.rtm_dst_len,
- (ip4_address_t *) r->gateway, sw_if_index, 0, ~0, 0);
-#else
fib_prefix_t prefix;
ip46_address_t nh;
@@ -234,16 +208,9 @@ add_del_route (ns_route_t * r, int is_del)
FIB_ENTRY_FLAG_NONE,
rpaths);
}
-#endif /* FIB_VERSION == 1 */
}
else if (r->rtm.rtm_family == AF_INET6)
{
-#if FIB_VERSION == 1
- ip6_add_del_route_next_hop (&ip6_main,
- is_del ? IP6_ROUTE_FLAG_DEL : IP6_ROUTE_FLAG_ADD,
- (ip6_address_t *) r->dst, r->rtm.rtm_dst_len,
- (ip6_address_t *) r->gateway, sw_if_index, 0, ~0, 0);
-#else
fib_prefix_t prefix;
ip46_address_t nh;
memset (&prefix, 0, sizeof (prefix));
@@ -257,7 +224,6 @@ add_del_route (ns_route_t * r, int is_del)
&nh, sw_if_index, 0,
0 /* weight */, NULL,
FIB_ROUTE_PATH_FLAG_NONE);
-#endif /* FIB_VERSION == 1 */
}
else if (r->rtm.rtm_family == AF_MPLS)
{