summaryrefslogtreecommitdiffstats
path: root/src/vnet
diff options
context:
space:
mode:
authorNeale Ranns <nranns@cisco.com>2017-10-13 02:43:33 -0700
committerFlorin Coras <florin.coras@gmail.com>2017-10-13 23:43:35 +0000
commitf9342023c19887da656133e2688a90d70383b0c5 (patch)
treece87af209b57e2210276f6b5cebedc1ec69fdf8e /src/vnet
parent69cce166535e138e66d0a95f1cfe6a3591489e10 (diff)
Enforce FIB table creation before use
last i the serise of the use of the FIB table create/delete API. VPP now forces the tables to have been explicitly creted before they are used. Change-Id: Ifde3b1bbb76697a01ab71bce4f5264e6d1725467 Signed-off-by: Neale Ranns <nranns@cisco.com>
Diffstat (limited to 'src/vnet')
-rw-r--r--src/vnet/interface_api.c43
-rw-r--r--src/vnet/ip/ip.api3
-rw-r--r--src/vnet/ip/ip_api.c37
-rw-r--r--src/vnet/mfib/mfib_table.h1
-rw-r--r--src/vnet/mpls/mpls.api5
5 files changed, 11 insertions, 78 deletions
diff --git a/src/vnet/interface_api.c b/src/vnet/interface_api.c
index c19f0a84297..ee13135f086 100644
--- a/src/vnet/interface_api.c
+++ b/src/vnet/interface_api.c
@@ -365,6 +365,14 @@ ip_table_bind (fib_protocol_t fproto,
fib_source_t src;
mfib_source_t msrc;
+ fib_index = fib_table_find (fproto, table_id);
+ mfib_index = mfib_table_find (fproto, table_id);
+
+ if (~0 == fib_index || ~0 == mfib_index)
+ {
+ return (VNET_API_ERROR_NO_SUCH_FIB);
+ }
+
if (is_api)
{
src = FIB_SOURCE_API;
@@ -376,32 +384,6 @@ ip_table_bind (fib_protocol_t fproto,
msrc = MFIB_SOURCE_CLI;
}
- /*
- * This is temporary whilst I do the song and dance with the CSIT version
- */
- if (0 != table_id)
- {
- fib_index = fib_table_find_or_create_and_lock (fproto, table_id, src);
- mfib_index =
- mfib_table_find_or_create_and_lock (fproto, table_id, msrc);
- }
- else
- {
- fib_index = 0;
- mfib_index = 0;
- }
-
- /*
- * This if table does not exist = error is what we want in the end.
- */
- /* fib_index = fib_table_find (fproto, table_id); */
- /* mfib_index = mfib_table_find (fproto, table_id); */
-
- /* if (~0 == fib_index || ~0 == mfib_index) */
- /* { */
- /* return (VNET_API_ERROR_NO_SUCH_FIB); */
- /* } */
-
if (FIB_PROTOCOL_IP6 == fproto)
{
/*
@@ -508,15 +490,6 @@ ip_table_bind (fib_protocol_t fproto,
ip4_main.mfib_index_by_sw_if_index[sw_if_index] = mfib_index;
}
- /*
- * Temporary. undo the locks from the find and create at the staart
- */
- if (0 != table_id)
- {
- fib_table_unlock (fib_index, fproto, src);
- mfib_table_unlock (mfib_index, fproto, msrc);
- }
-
return (0);
}
diff --git a/src/vnet/ip/ip.api b/src/vnet/ip/ip.api
index e302b1e7a44..df9ce6e615a 100644
--- a/src/vnet/ip/ip.api
+++ b/src/vnet/ip/ip.api
@@ -362,7 +362,6 @@ autoreply define sw_interface_ip6_set_link_local_address
@param vrf_id - fib table /vrf associated with the route
@param lookup_in_vrf -
@param classify_table_index -
- @param create_vrf_if_needed -
@param is_add - 1 if adding the route, 0 if deleting
@param is_drop - Drop the packet
@param is_unreach - Drop the packet and rate limit send ICMP unreachable
@@ -388,7 +387,6 @@ autoreply define ip_add_del_route
u32 table_id;
u32 classify_table_index;
u32 next_hop_table_id;
- u8 create_vrf_if_needed;
u8 is_add;
u8 is_drop;
u8 is_unreach;
@@ -430,7 +428,6 @@ autoreply define ip_mroute_add_del
u32 itf_flags;
u32 rpf_id;
u16 grp_address_length;
- u8 create_vrf_if_needed;
u8 is_add;
u8 is_ipv6;
u8 is_local;
diff --git a/src/vnet/ip/ip_api.c b/src/vnet/ip/ip_api.c
index d9ec4b47998..f3712d3fd29 100644
--- a/src/vnet/ip/ip_api.c
+++ b/src/vnet/ip/ip_api.c
@@ -986,23 +986,11 @@ add_del_route_check (fib_protocol_t table_proto,
{
vnet_main_t *vnm = vnet_get_main ();
- /* Temporaray whilst I do the CSIT dance */
- u8 create_missing_tables = 1;
-
*fib_index = fib_table_find (table_proto, ntohl (table_id));
if (~0 == *fib_index)
{
- if (create_missing_tables)
- {
- *fib_index = fib_table_find_or_create_and_lock (table_proto,
- ntohl (table_id),
- FIB_SOURCE_API);
- }
- else
- {
- /* No such VRF, and we weren't asked to create one */
- return VNET_API_ERROR_NO_SUCH_FIB;
- }
+ /* No such VRF */
+ return VNET_API_ERROR_NO_SUCH_FIB;
}
if (!is_rpf_id && ~0 != ntohl (next_hop_sw_if_index))
@@ -1031,26 +1019,7 @@ add_del_route_check (fib_protocol_t table_proto,
if (~0 == *next_hop_fib_index)
{
- if (create_missing_tables)
- {
- if (is_rpf_id)
- *next_hop_fib_index =
- mfib_table_find_or_create_and_lock (fib_nh_proto,
- ntohl
- (next_hop_table_id),
- MFIB_SOURCE_API);
- else
- *next_hop_fib_index =
- fib_table_find_or_create_and_lock (fib_nh_proto,
- ntohl
- (next_hop_table_id),
- FIB_SOURCE_API);
- }
- else
- {
- /* No such VRF, and we weren't asked to create one */
- return VNET_API_ERROR_NO_SUCH_FIB;
- }
+ return VNET_API_ERROR_NO_SUCH_FIB;
}
}
diff --git a/src/vnet/mfib/mfib_table.h b/src/vnet/mfib/mfib_table.h
index 6a5810291fc..4c1077f900d 100644
--- a/src/vnet/mfib/mfib_table.h
+++ b/src/vnet/mfib/mfib_table.h
@@ -347,7 +347,6 @@ extern u32 mfib_table_find_or_create_and_lock_w_name(fib_protocol_t proto,
mfib_source_t source,
const u8 *name);
-
/**
* @brief
* Take a reference counting lock on the table
diff --git a/src/vnet/mpls/mpls.api b/src/vnet/mpls/mpls.api
index 3c817db1306..8cc1ea88a9a 100644
--- a/src/vnet/mpls/mpls.api
+++ b/src/vnet/mpls/mpls.api
@@ -22,7 +22,6 @@ vl_api_version 1.0.0
@param mb_mpls_table_id - The MPLS table-id the MPLS entry will be added in
@param mb_label - The MPLS label value to bind
@param mb_ip_table_id - The IP table-id of the IP prefix to bind to.
- @param mb_create_table_if_needed - Create either/both tables if required.
@param mb_is_bind - Bind or unbind
@param mb_is_ip4 - The prefix to bind to is IPv4
@param mb_address_length - Length of IP prefix
@@ -35,7 +34,6 @@ autoreply define mpls_ip_bind_unbind
u32 mb_mpls_table_id;
u32 mb_label;
u32 mb_ip_table_id;
- u8 mb_create_table_if_needed;
u8 mb_is_bind;
u8 mb_is_ip4;
u8 mb_address_length;
@@ -164,8 +162,6 @@ autoreply define mpls_table_add_del
@param mr_table_id - The MPLS table-id the route is added in
@param mr_classify_table_index - If this is a classify route,
this is the classify table index
- @param mr_create_table_if_needed - If the MPLS or IP tables do not exist,
- create them
@param mr_is_add - Is this a route add or delete
@param mr_is_classify - Is this route result a classify
@param mr_is_multicast - Is this a multicast route
@@ -193,7 +189,6 @@ autoreply define mpls_route_add_del
u8 mr_eos;
u32 mr_table_id;
u32 mr_classify_table_index;
- u8 mr_create_table_if_needed;
u8 mr_is_add;
u8 mr_is_classify;
u8 mr_is_multicast;