Code Review / vpp.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
tls session: fix unlisten
[vpp.git] / src / plugins / gtpu / gtpu_encap.c
diff --git a/src/plugins/gtpu/gtpu_encap.c b/src/plugins/gtpu/gtpu_encap.c
index 4442c42..e6b2ce5 100644 (file)
--- a/src/plugins/gtpu/gtpu_encap.c
+++ b/src/plugins/gtpu/gtpu_encap.c
@@ -48,23 +48,6 @@ typedef enum {
     GTPU_ENCAP_N_NEXT,
 } gtpu_encap_next_t;
 
-typedef struct {
-  u32 tunnel_index;
-  u32 teid;
-} gtpu_encap_trace_t;
-
-u8 * format_gtpu_encap_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 *);
-  gtpu_encap_trace_t * t
-      = va_arg (*args, gtpu_encap_trace_t *);
-
-  s = format (s, "GTPU encap to gtpu_tunnel%d teid %d",
-             t->tunnel_index, t->teid);
-  return s;
-}
-
 
 #define foreach_fixed_header4_offset            \
     _(0) _(1) _(2) _(3)
@@ -380,24 +363,6 @@ gtpu_encap_inline (vlib_main_t * vm,
              udp3->length = new_l3;
              udp3->src_port = flow_hash3;
 
-             /* IPv6 UDP checksum is mandatory */
-             udp0->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b0,
-                                                                ip6_0, &bogus);
-             if (udp0->checksum == 0)
-               udp0->checksum = 0xffff;
-             udp1->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b1,
-                                                        ip6_1, &bogus);
-             if (udp1->checksum == 0)
-               udp1->checksum = 0xffff;
-             udp2->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b2,
-                                                                ip6_2, &bogus);
-             if (udp2->checksum == 0)
-               udp2->checksum = 0xffff;
-             udp3->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b3,
-                                                        ip6_3, &bogus);
-             if (udp3->checksum == 0)
-               udp3->checksum = 0xffff;
-
              /* Fix GTPU length */
              gtpu0 = (gtpu_header_t *)(udp0+1);
              new_l0 = clib_host_to_net_u16 (vlib_buffer_length_in_chain(vm, b0)
@@ -419,6 +384,25 @@ gtpu_encap_inline (vlib_main_t * vm,
                                             - sizeof (*ip6_3) - sizeof(*udp3)
                                             - GTPU_V1_HDR_LEN);
              gtpu3->length = new_l3;
+
+             /* IPv6 UDP checksum is mandatory */
+             udp0->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b0,
+                                                                ip6_0, &bogus);
+             if (udp0->checksum == 0)
+               udp0->checksum = 0xffff;
+             udp1->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b1,
+                                                                ip6_1, &bogus);
+             if (udp1->checksum == 0)
+               udp1->checksum = 0xffff;
+             udp2->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b2,
+                                                                ip6_2, &bogus);
+             if (udp2->checksum == 0)
+               udp2->checksum = 0xffff;
+             udp3->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b3,
+                                                                ip6_3, &bogus);
+             if (udp3->checksum == 0)
+               udp3->checksum = 0xffff;
+
            }
 
           pkts_encapsulated += 4;
@@ -429,6 +413,12 @@ gtpu_encap_inline (vlib_main_t * vm,
          stats_n_packets += 4;
          stats_n_bytes += len0 + len1 + len2 + len3;
 
+          /* save inner packet flow_hash for load-balance node */
+          vnet_buffer (b0)->ip.flow_hash = flow_hash0;
+          vnet_buffer (b1)->ip.flow_hash = flow_hash1;
+          vnet_buffer (b2)->ip.flow_hash = flow_hash2;
+          vnet_buffer (b3)->ip.flow_hash = flow_hash3;
+
          /* Batch stats increment on the same gtpu tunnel so counter is not
             incremented per packet. Note stats are still incremented for deleted
             and admin-down tunnel where packets are dropped. It is not worthwhile
@@ -486,6 +476,22 @@ gtpu_encap_inline (vlib_main_t * vm,
               tr->teid = t1->teid;
             }
 
+         if (PREDICT_FALSE(b2->flags & VLIB_BUFFER_IS_TRACED))
+            {
+              gtpu_encap_trace_t *tr =
+                vlib_add_trace (vm, node, b2, sizeof (*tr));
+              tr->tunnel_index = t2 - gtm->tunnels;
+              tr->teid = t2->teid;
+           }
+
+          if (PREDICT_FALSE(b3->flags & VLIB_BUFFER_IS_TRACED))
+            {
+              gtpu_encap_trace_t *tr =
+                vlib_add_trace (vm, node, b3, sizeof (*tr));
+              tr->tunnel_index = t3 - gtm->tunnels;
+              tr->teid = t3->teid;
+            }
+
          vlib_validate_buffer_enqueue_x4 (vm, node, next_index,
                                           to_next, n_left_to_next,
                                           bi0, bi1, bi2, bi3,
@@ -592,18 +598,18 @@ gtpu_encap_inline (vlib_main_t * vm,
              udp0->length = new_l0;
              udp0->src_port = flow_hash0;
 
-             /* IPv6 UDP checksum is mandatory */
-             udp0->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b0,
-                                                                ip6_0, &bogus);
-             if (udp0->checksum == 0)
-               udp0->checksum = 0xffff;
-
              /* Fix GTPU length */
              gtpu0 = (gtpu_header_t *)(udp0+1);
              new_l0 = clib_host_to_net_u16 (vlib_buffer_length_in_chain(vm, b0)
                                             - sizeof (*ip4_0) - sizeof(*udp0)
                                             - GTPU_V1_HDR_LEN);
              gtpu0->length = new_l0;
+
+             /* IPv6 UDP checksum is mandatory */
+             udp0->checksum = ip6_tcp_udp_icmp_compute_checksum(vm, b0,
+                                                                ip6_0, &bogus);
+             if (udp0->checksum == 0)
+               udp0->checksum = 0xffff;
            }
 
           pkts_encapsulated ++;
@@ -611,6 +617,9 @@ gtpu_encap_inline (vlib_main_t * vm,
          stats_n_packets += 1;
          stats_n_bytes += len0;
 
+          /* save inner packet flow_hash for load-balance node */
+          vnet_buffer (b0)->ip.flow_hash = flow_hash0;
+
          /* Batch stats increment on the same gtpu tunnel so counter is not
             incremented per packet. Note stats are still incremented for deleted
             and admin-down tunnel where packets are dropped. It is not worthwhile
@@ -661,16 +670,14 @@ gtpu_encap_inline (vlib_main_t * vm,
   return from_frame->n_vectors;
 }
 
-static uword
-gtpu4_encap (vlib_main_t * vm,
+VLIB_NODE_FN (gtpu4_encap_node) (vlib_main_t * vm,
              vlib_node_runtime_t * node,
              vlib_frame_t * from_frame)
 {
   return gtpu_encap_inline (vm, node, from_frame, /* is_ip4 */ 1);
 }
 
-static uword
-gtpu6_encap (vlib_main_t * vm,
+VLIB_NODE_FN (gtpu6_encap_node) (vlib_main_t * vm,
              vlib_node_runtime_t * node,
              vlib_frame_t * from_frame)
 {
@@ -678,7 +685,6 @@ gtpu6_encap (vlib_main_t * vm,
 }
 
 VLIB_REGISTER_NODE (gtpu4_encap_node) = {
-  .function = gtpu4_encap,
   .name = "gtpu4-encap",
   .vector_size = sizeof (u32),
   .format_trace = format_gtpu_encap_trace,
@@ -693,10 +699,7 @@ VLIB_REGISTER_NODE (gtpu4_encap_node) = {
   },
 };
 
-VLIB_NODE_FUNCTION_MULTIARCH (gtpu4_encap_node, gtpu4_encap)
-
 VLIB_REGISTER_NODE (gtpu6_encap_node) = {
-  .function = gtpu6_encap,
   .name = "gtpu6-encap",
   .vector_size = sizeof (u32),
   .format_trace = format_gtpu_encap_trace,
@@ -711,5 +714,3 @@ VLIB_REGISTER_NODE (gtpu6_encap_node) = {
   },
 };
 
-VLIB_NODE_FUNCTION_MULTIARCH (gtpu6_encap_node, gtpu6_encap)
-
Vector Packet Processing