fib: Move the adjacency midchain nodes into a separate file 37/31837/3
authorNeale Ranns <neale@graphiant.com>
Wed, 31 Mar 2021 07:36:33 +0000 (07:36 +0000)
committerNeale Ranns <neale@graphiant.com>
Thu, 1 Apr 2021 11:37:08 +0000 (11:37 +0000)
Type: refactor

It's only the nodes that require compiling for each arch,

Signed-off-by: Neale Ranns <neale@graphiant.com>
Change-Id: Ibb2bfc756001735cc10ef0fd8db7e8b8358cf8e2

src/vnet/CMakeLists.txt
src/vnet/adj/adj_internal.h
src/vnet/adj/adj_midchain.c
src/vnet/adj/adj_midchain_node.c [new file with mode: 0644]
src/vnet/adj/adj_nsh.h

index 1ff2a8c..0a3b710 100644 (file)
@@ -1227,6 +1227,7 @@ list(APPEND VNET_SOURCES
   adj/adj_nbr.c
   adj/adj_glean.c
   adj/adj_midchain.c
+  adj/adj_midchain_node.c
   adj/adj_midchain_delegate.c
   adj/adj_mcast.c
   adj/adj_l2.c
@@ -1240,7 +1241,7 @@ list(APPEND VNET_SOURCES
 list(APPEND VNET_MULTIARCH_SOURCES
   adj/adj_nsh.c
   adj/adj_l2.c
-  adj/adj_midchain.c
+  adj/adj_midchain_node.c
 )
 
 list(APPEND VNET_HEADERS
index 253c1e9..3dbf7e2 100644 (file)
 #define ADJ_DBG(_e, _fmt, _args...)
 #endif
 
+/*
+ * Vlib nodes
+ */
+extern vlib_node_registration_t adj_nsh_midchain_node;
+extern vlib_node_registration_t adj_nsh_rewrite_node;
+extern vlib_node_registration_t adj_midchain_tx_no_count_node;
+extern vlib_node_registration_t adj_midchain_tx_node;
+
 static inline u32
 adj_get_rewrite_node (vnet_link_t linkt)
 {
index 3d879e9..9f709ad 100644 (file)
@@ -18,7 +18,6 @@
 #include <vnet/adj/adj_l2.h>
 #include <vnet/adj/adj_nsh.h>
 #include <vnet/adj/adj_midchain.h>
-#include <vnet/ethernet/arp_packet.h>
 #include <vnet/dpo/drop_dpo.h>
 #include <vnet/dpo/load_balance.h>
 #include <vnet/fib/fib_walk.h>
 #include <vnet/ip/ip4_inlines.h>
 #include <vnet/ip/ip6_inlines.h>
 
-/**
- * @brief Trace data for packets traversing the midchain tx node
- */
-typedef struct adj_midchain_tx_trace_t_
-{
-    /**
-     * @brief the midchain adj we are traversing
-     */
-    adj_index_t ai;
-} adj_midchain_tx_trace_t;
-
-always_inline uword
-adj_midchain_tx_inline (vlib_main_t * vm,
-                       vlib_node_runtime_t * node,
-                       vlib_frame_t * frame,
-                       int interface_count)
-{
-    vlib_buffer_t *bufs[VLIB_FRAME_SIZE], **b;
-    u16 nexts[VLIB_FRAME_SIZE], *next;
-    u32 * from, n_left, thread_index;
-    vnet_main_t *vnm = vnet_get_main ();
-    vnet_interface_main_t *im = &vnm->interface_main;
-
-    thread_index = vm->thread_index;
-    n_left = frame->n_vectors;
-    from = vlib_frame_vector_args (frame);
-
-    vlib_get_buffers (vm, from, bufs, n_left);
-
-    next = nexts;
-    b = bufs;
-
-    while (n_left > 8)
-    {
-        u32 adj_index0, adj_index1, adj_index2, adj_index3;
-        const ip_adjacency_t *adj0, *adj1, *adj2, *adj3;
-        const dpo_id_t *dpo0, *dpo1, *dpo2, *dpo3;
-
-        /* Prefetch next iteration. */
-        {
-            vlib_prefetch_buffer_header (b[4], LOAD);
-            vlib_prefetch_buffer_header (b[5], LOAD);
-            vlib_prefetch_buffer_header (b[6], LOAD);
-            vlib_prefetch_buffer_header (b[7], LOAD);
-        }
-
-        /* Follow the DPO on which the midchain is stacked */
-        adj_index0 = vnet_buffer(b[0])->ip.adj_index[VLIB_TX];
-        adj_index1 = vnet_buffer(b[1])->ip.adj_index[VLIB_TX];
-        adj_index2 = vnet_buffer(b[2])->ip.adj_index[VLIB_TX];
-        adj_index3 = vnet_buffer(b[3])->ip.adj_index[VLIB_TX];
-
-        adj0 = adj_get(adj_index0);
-        adj1 = adj_get(adj_index1);
-        adj2 = adj_get(adj_index2);
-        adj3 = adj_get(adj_index3);
-
-        dpo0 = &adj0->sub_type.midchain.next_dpo;
-        dpo1 = &adj1->sub_type.midchain.next_dpo;
-        dpo2 = &adj2->sub_type.midchain.next_dpo;
-        dpo3 = &adj3->sub_type.midchain.next_dpo;
-
-        next[0] = dpo0->dpoi_next_node;
-        next[1] = dpo1->dpoi_next_node;
-        next[2] = dpo2->dpoi_next_node;
-        next[3] = dpo3->dpoi_next_node;
-
-        vnet_buffer(b[0])->ip.adj_index[VLIB_TX] = dpo0->dpoi_index;
-        vnet_buffer(b[1])->ip.adj_index[VLIB_TX] = dpo1->dpoi_index;
-        vnet_buffer(b[2])->ip.adj_index[VLIB_TX] = dpo2->dpoi_index;
-        vnet_buffer(b[3])->ip.adj_index[VLIB_TX] = dpo3->dpoi_index;
-
-        if (interface_count)
-        {
-            vlib_increment_combined_counter (im->combined_sw_if_counters
-                                             + VNET_INTERFACE_COUNTER_TX,
-                                             thread_index,
-                                             adj0->rewrite_header.sw_if_index,
-                                             1,
-                                             vlib_buffer_length_in_chain (vm, b[0]));
-            vlib_increment_combined_counter (im->combined_sw_if_counters
-                                             + VNET_INTERFACE_COUNTER_TX,
-                                             thread_index,
-                                             adj1->rewrite_header.sw_if_index,
-                                             1,
-                                             vlib_buffer_length_in_chain (vm, b[1]));
-            vlib_increment_combined_counter (im->combined_sw_if_counters
-                                             + VNET_INTERFACE_COUNTER_TX,
-                                             thread_index,
-                                             adj2->rewrite_header.sw_if_index,
-                                             1,
-                                             vlib_buffer_length_in_chain (vm, b[2]));
-            vlib_increment_combined_counter (im->combined_sw_if_counters
-                                             + VNET_INTERFACE_COUNTER_TX,
-                                             thread_index,
-                                             adj3->rewrite_header.sw_if_index,
-                                             1,
-                                             vlib_buffer_length_in_chain (vm, b[3]));
-        }
-
-        if (PREDICT_FALSE(node->flags & VLIB_NODE_FLAG_TRACE))
-        {
-            if (PREDICT_FALSE(b[0]->flags & VLIB_BUFFER_IS_TRACED))
-            {
-                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
-                                                              b[0], sizeof (*tr));
-                tr->ai = adj_index0;
-            }
-            if (PREDICT_FALSE(b[1]->flags & VLIB_BUFFER_IS_TRACED))
-            {
-                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
-                                                              b[1], sizeof (*tr));
-                tr->ai = adj_index1;
-            }
-            if (PREDICT_FALSE(b[2]->flags & VLIB_BUFFER_IS_TRACED))
-            {
-                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
-                                                              b[2], sizeof (*tr));
-                tr->ai = adj_index2;
-            }
-            if (PREDICT_FALSE(b[3]->flags & VLIB_BUFFER_IS_TRACED))
-            {
-                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
-                                                              b[3], sizeof (*tr));
-                tr->ai = adj_index3;
-            }
-        }
-        n_left -= 4;
-        b += 4;
-        next += 4;
-    }
-
-    while (n_left)
-    {
-        const ip_adjacency_t * adj0;
-        const dpo_id_t *dpo0;
-        u32 adj_index0;
-
-        /* Follow the DPO on which the midchain is stacked */
-        adj_index0 = vnet_buffer(b[0])->ip.adj_index[VLIB_TX];
-        adj0 = adj_get(adj_index0);
-        dpo0 = &adj0->sub_type.midchain.next_dpo;
-        next[0] = dpo0->dpoi_next_node;
-        vnet_buffer(b[0])->ip.adj_index[VLIB_TX] = dpo0->dpoi_index;
-
-        if (interface_count)
-        {
-            vlib_increment_combined_counter (im->combined_sw_if_counters
-                                             + VNET_INTERFACE_COUNTER_TX,
-                                             thread_index,
-                                             adj0->rewrite_header.sw_if_index,
-                                             1,
-                                             vlib_buffer_length_in_chain (vm, b[0]));
-        }
-
-        if (PREDICT_FALSE(b[0]->flags & VLIB_BUFFER_IS_TRACED))
-        {
-            adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
-                                                          b[0], sizeof (*tr));
-            tr->ai = adj_index0;
-        }
-
-        n_left -= 1;
-        b += 1;
-        next += 1;
-    }
-
-    vlib_buffer_enqueue_to_next (vm, node, from, nexts, frame->n_vectors);
-
-    return frame->n_vectors;
-}
-
-static u8 *
-format_adj_midchain_tx_trace (u8 * s, va_list * args)
-{
-    CLIB_UNUSED (vlib_main_t * vm) = va_arg (*args, vlib_main_t *);
-    CLIB_UNUSED (vlib_node_t * node) = va_arg (*args, vlib_node_t *);
-    adj_midchain_tx_trace_t *tr = va_arg (*args, adj_midchain_tx_trace_t*);
-
-    s = format(s, "adj-midchain:[%d]:%U", tr->ai,
-              format_ip_adjacency, tr->ai,
-              FORMAT_IP_ADJACENCY_NONE);
-
-    return (s);
-}
-
-static uword
-adj_midchain_tx (vlib_main_t * vm,
-                vlib_node_runtime_t * node,
-                vlib_frame_t * frame)
-{
-    return (adj_midchain_tx_inline(vm, node, frame, 1));
-}
-
-VLIB_REGISTER_NODE (adj_midchain_tx_node) = {
-    .function = adj_midchain_tx,
-    .name = "adj-midchain-tx",
-    .vector_size = sizeof (u32),
-
-    .format_trace = format_adj_midchain_tx_trace,
-
-    .n_next_nodes = 1,
-    .next_nodes = {
-       [0] = "error-drop",
-    },
-};
-
-static uword
-adj_midchain_tx_no_count (vlib_main_t * vm,
-                         vlib_node_runtime_t * node,
-                         vlib_frame_t * frame)
-{
-    return (adj_midchain_tx_inline(vm, node, frame, 0));
-}
-
-VLIB_REGISTER_NODE (adj_midchain_tx_no_count_node) = {
-    .function = adj_midchain_tx_no_count,
-    .name = "adj-midchain-tx-no-count",
-    .vector_size = sizeof (u32),
-
-    .format_trace = format_adj_midchain_tx_trace,
-    .sibling_of = "adj-midchain-tx",
-};
-
-#ifndef CLIB_MARCH_VARIANT
-
 u8
 adj_is_midchain (adj_index_t ai)
 {
@@ -794,5 +567,3 @@ adj_midchain_module_init (void)
 {
     dpo_register(DPO_ADJACENCY_MIDCHAIN, &adj_midchain_dpo_vft, midchain_nodes);
 }
-
-#endif
diff --git a/src/vnet/adj/adj_midchain_node.c b/src/vnet/adj/adj_midchain_node.c
new file mode 100644 (file)
index 0000000..170ed19
--- /dev/null
@@ -0,0 +1,241 @@
+/*
+ * Copyright (c) 2016 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <vnet/adj/adj_internal.h>
+#include <vnet/adj/adj_midchain.h>
+
+/**
+ * @brief Trace data for packets traversing the midchain tx node
+ */
+typedef struct adj_midchain_tx_trace_t_
+{
+    /**
+     * @brief the midchain adj we are traversing
+     */
+    adj_index_t ai;
+} adj_midchain_tx_trace_t;
+
+always_inline uword
+adj_midchain_tx_inline (vlib_main_t * vm,
+                       vlib_node_runtime_t * node,
+                       vlib_frame_t * frame,
+                       int interface_count)
+{
+    vlib_buffer_t *bufs[VLIB_FRAME_SIZE], **b;
+    u16 nexts[VLIB_FRAME_SIZE], *next;
+    u32 * from, n_left, thread_index;
+    vnet_main_t *vnm = vnet_get_main ();
+    vnet_interface_main_t *im = &vnm->interface_main;
+
+    thread_index = vm->thread_index;
+    n_left = frame->n_vectors;
+    from = vlib_frame_vector_args (frame);
+
+    vlib_get_buffers (vm, from, bufs, n_left);
+
+    next = nexts;
+    b = bufs;
+
+    while (n_left > 8)
+    {
+        u32 adj_index0, adj_index1, adj_index2, adj_index3;
+        const ip_adjacency_t *adj0, *adj1, *adj2, *adj3;
+        const dpo_id_t *dpo0, *dpo1, *dpo2, *dpo3;
+
+        /* Prefetch next iteration. */
+        {
+            vlib_prefetch_buffer_header (b[4], LOAD);
+            vlib_prefetch_buffer_header (b[5], LOAD);
+            vlib_prefetch_buffer_header (b[6], LOAD);
+            vlib_prefetch_buffer_header (b[7], LOAD);
+        }
+
+        /* Follow the DPO on which the midchain is stacked */
+        adj_index0 = vnet_buffer(b[0])->ip.adj_index[VLIB_TX];
+        adj_index1 = vnet_buffer(b[1])->ip.adj_index[VLIB_TX];
+        adj_index2 = vnet_buffer(b[2])->ip.adj_index[VLIB_TX];
+        adj_index3 = vnet_buffer(b[3])->ip.adj_index[VLIB_TX];
+
+        adj0 = adj_get(adj_index0);
+        adj1 = adj_get(adj_index1);
+        adj2 = adj_get(adj_index2);
+        adj3 = adj_get(adj_index3);
+
+        dpo0 = &adj0->sub_type.midchain.next_dpo;
+        dpo1 = &adj1->sub_type.midchain.next_dpo;
+        dpo2 = &adj2->sub_type.midchain.next_dpo;
+        dpo3 = &adj3->sub_type.midchain.next_dpo;
+
+        next[0] = dpo0->dpoi_next_node;
+        next[1] = dpo1->dpoi_next_node;
+        next[2] = dpo2->dpoi_next_node;
+        next[3] = dpo3->dpoi_next_node;
+
+        vnet_buffer(b[0])->ip.adj_index[VLIB_TX] = dpo0->dpoi_index;
+        vnet_buffer(b[1])->ip.adj_index[VLIB_TX] = dpo1->dpoi_index;
+        vnet_buffer(b[2])->ip.adj_index[VLIB_TX] = dpo2->dpoi_index;
+        vnet_buffer(b[3])->ip.adj_index[VLIB_TX] = dpo3->dpoi_index;
+
+        if (interface_count)
+        {
+            vlib_increment_combined_counter (im->combined_sw_if_counters
+                                             + VNET_INTERFACE_COUNTER_TX,
+                                             thread_index,
+                                             adj0->rewrite_header.sw_if_index,
+                                             1,
+                                             vlib_buffer_length_in_chain (vm, b[0]));
+            vlib_increment_combined_counter (im->combined_sw_if_counters
+                                             + VNET_INTERFACE_COUNTER_TX,
+                                             thread_index,
+                                             adj1->rewrite_header.sw_if_index,
+                                             1,
+                                             vlib_buffer_length_in_chain (vm, b[1]));
+            vlib_increment_combined_counter (im->combined_sw_if_counters
+                                             + VNET_INTERFACE_COUNTER_TX,
+                                             thread_index,
+                                             adj2->rewrite_header.sw_if_index,
+                                             1,
+                                             vlib_buffer_length_in_chain (vm, b[2]));
+            vlib_increment_combined_counter (im->combined_sw_if_counters
+                                             + VNET_INTERFACE_COUNTER_TX,
+                                             thread_index,
+                                             adj3->rewrite_header.sw_if_index,
+                                             1,
+                                             vlib_buffer_length_in_chain (vm, b[3]));
+        }
+
+        if (PREDICT_FALSE(node->flags & VLIB_NODE_FLAG_TRACE))
+        {
+            if (PREDICT_FALSE(b[0]->flags & VLIB_BUFFER_IS_TRACED))
+            {
+                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
+                                                              b[0], sizeof (*tr));
+                tr->ai = adj_index0;
+            }
+            if (PREDICT_FALSE(b[1]->flags & VLIB_BUFFER_IS_TRACED))
+            {
+                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
+                                                              b[1], sizeof (*tr));
+                tr->ai = adj_index1;
+            }
+            if (PREDICT_FALSE(b[2]->flags & VLIB_BUFFER_IS_TRACED))
+            {
+                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
+                                                              b[2], sizeof (*tr));
+                tr->ai = adj_index2;
+            }
+            if (PREDICT_FALSE(b[3]->flags & VLIB_BUFFER_IS_TRACED))
+            {
+                adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
+                                                              b[3], sizeof (*tr));
+                tr->ai = adj_index3;
+            }
+        }
+        n_left -= 4;
+        b += 4;
+        next += 4;
+    }
+
+    while (n_left)
+    {
+        const ip_adjacency_t * adj0;
+        const dpo_id_t *dpo0;
+        u32 adj_index0;
+
+        /* Follow the DPO on which the midchain is stacked */
+        adj_index0 = vnet_buffer(b[0])->ip.adj_index[VLIB_TX];
+        adj0 = adj_get(adj_index0);
+        dpo0 = &adj0->sub_type.midchain.next_dpo;
+        next[0] = dpo0->dpoi_next_node;
+        vnet_buffer(b[0])->ip.adj_index[VLIB_TX] = dpo0->dpoi_index;
+
+        if (interface_count)
+        {
+            vlib_increment_combined_counter (im->combined_sw_if_counters
+                                             + VNET_INTERFACE_COUNTER_TX,
+                                             thread_index,
+                                             adj0->rewrite_header.sw_if_index,
+                                             1,
+                                             vlib_buffer_length_in_chain (vm, b[0]));
+        }
+
+        if (PREDICT_FALSE(b[0]->flags & VLIB_BUFFER_IS_TRACED))
+        {
+            adj_midchain_tx_trace_t *tr = vlib_add_trace (vm, node,
+                                                          b[0], sizeof (*tr));
+            tr->ai = adj_index0;
+        }
+
+        n_left -= 1;
+        b += 1;
+        next += 1;
+    }
+
+    vlib_buffer_enqueue_to_next (vm, node, from, nexts, frame->n_vectors);
+
+    return frame->n_vectors;
+}
+
+static u8 *
+format_adj_midchain_tx_trace (u8 * s, va_list * args)
+{
+    CLIB_UNUSED (vlib_main_t * vm) = va_arg (*args, vlib_main_t *);
+    CLIB_UNUSED (vlib_node_t * node) = va_arg (*args, vlib_node_t *);
+    adj_midchain_tx_trace_t *tr = va_arg (*args, adj_midchain_tx_trace_t*);
+
+    s = format(s, "adj-midchain:[%d]:%U", tr->ai,
+              format_ip_adjacency, tr->ai,
+              FORMAT_IP_ADJACENCY_NONE);
+
+    return (s);
+}
+
+static uword
+adj_midchain_tx (vlib_main_t * vm,
+                vlib_node_runtime_t * node,
+                vlib_frame_t * frame)
+{
+    return (adj_midchain_tx_inline(vm, node, frame, 1));
+}
+
+VLIB_REGISTER_NODE (adj_midchain_tx_node) = {
+    .function = adj_midchain_tx,
+    .name = "adj-midchain-tx",
+    .vector_size = sizeof (u32),
+
+    .format_trace = format_adj_midchain_tx_trace,
+
+    .n_next_nodes = 1,
+    .next_nodes = {
+       [0] = "error-drop",
+    },
+};
+
+static uword
+adj_midchain_tx_no_count (vlib_main_t * vm,
+                         vlib_node_runtime_t * node,
+                         vlib_frame_t * frame)
+{
+    return (adj_midchain_tx_inline(vm, node, frame, 0));
+}
+
+VLIB_REGISTER_NODE (adj_midchain_tx_no_count_node) = {
+    .function = adj_midchain_tx_no_count,
+    .name = "adj-midchain-tx-no-count",
+    .vector_size = sizeof (u32),
+
+    .format_trace = format_adj_midchain_tx_trace,
+    .sibling_of = "adj-midchain-tx",
+};
index eff2069..6ec31ee 100644 (file)
@@ -18,9 +18,6 @@
 
 #include <vnet/adj/adj.h>
 
-extern vlib_node_registration_t adj_nsh_midchain_node;
-extern vlib_node_registration_t adj_nsh_rewrite_node;
-
 typedef struct _nsh_main_placeholder
 {
   u8 output_feature_arc_index;