vlib: fix trace number accounting
[vpp.git] / src / plugins / vmxnet3 / input.c
index 4ff459a..bed4c1e 100644 (file)
@@ -20,6 +20,9 @@
 #include <vlib/pci/pci.h>
 #include <vnet/ethernet/ethernet.h>
 #include <vnet/devices/devices.h>
+#include <vnet/ip/ip6_packet.h>
+#include <vnet/ip/ip4_packet.h>
+#include <vnet/udp/udp_packet.h>
 
 #include <vmxnet3/vmxnet3.h>
 
@@ -27,6 +30,7 @@
   _(BUFFER_ALLOC, "buffer alloc error") \
   _(RX_PACKET_NO_SOP, "Rx packet error - no SOP") \
   _(RX_PACKET, "Rx packet error") \
+  _(RX_PACKET_EOP, "Rx packet error found on EOP") \
   _(NO_BUFFER, "Rx no buffer error")
 
 typedef enum
@@ -70,6 +74,128 @@ vmxnet3_rx_comp_ring_advance_next (vmxnet3_rxq_t * rxq)
     }
 }
 
+static_always_inline void
+vmxnet3_handle_offload (vmxnet3_rx_comp * rx_comp, vlib_buffer_t * hb,
+                       u16 gso_size)
+{
+  u8 l4_hdr_sz = 0;
+
+  if (rx_comp->flags & VMXNET3_RXCF_IP4)
+    {
+      ip4_header_t *ip4 = (ip4_header_t *) (hb->data +
+                                           sizeof (ethernet_header_t));
+
+      vnet_buffer (hb)->l2_hdr_offset = 0;
+      vnet_buffer (hb)->l3_hdr_offset = sizeof (ethernet_header_t);
+      vnet_buffer (hb)->l4_hdr_offset = sizeof (ethernet_header_t) +
+       ip4_header_bytes (ip4);
+      hb->flags |= VNET_BUFFER_F_L2_HDR_OFFSET_VALID |
+       VNET_BUFFER_F_L3_HDR_OFFSET_VALID |
+       VNET_BUFFER_F_L4_HDR_OFFSET_VALID | VNET_BUFFER_F_IS_IP4;
+
+      /* checksum offload */
+      if (!(rx_comp->index & VMXNET3_RXCI_CNC))
+       {
+         if (!(rx_comp->flags & VMXNET3_RXCF_IPC))
+           {
+             hb->flags |= VNET_BUFFER_F_OFFLOAD_IP_CKSUM;
+             ip4->checksum = 0;
+           }
+         if (!(rx_comp->flags & VMXNET3_RXCF_TUC))
+           {
+             if (rx_comp->flags & VMXNET3_RXCF_TCP)
+               {
+                 tcp_header_t *tcp =
+                   (tcp_header_t *) (hb->data +
+                                     vnet_buffer (hb)->l4_hdr_offset);
+                 hb->flags |= VNET_BUFFER_F_OFFLOAD_TCP_CKSUM;
+                 tcp->checksum = 0;
+               }
+             else if (rx_comp->flags & VMXNET3_RXCF_UDP)
+               {
+                 udp_header_t *udp =
+                   (udp_header_t *) (hb->data +
+                                     vnet_buffer (hb)->l4_hdr_offset);
+                 hb->flags |= VNET_BUFFER_F_OFFLOAD_UDP_CKSUM;
+                 udp->checksum = 0;
+               }
+           }
+       }
+
+      if (gso_size)
+       {
+         if (rx_comp->flags & VMXNET3_RXCF_TCP)
+           {
+             tcp_header_t *tcp =
+               (tcp_header_t *) (hb->data + vnet_buffer (hb)->l4_hdr_offset);
+             l4_hdr_sz = tcp_header_bytes (tcp);
+           }
+         else if (rx_comp->flags & VMXNET3_RXCF_UDP)
+           {
+             udp_header_t *udp =
+               (udp_header_t *) (hb->data + vnet_buffer (hb)->l4_hdr_offset);
+             l4_hdr_sz = sizeof (*udp);
+           }
+         vnet_buffer2 (hb)->gso_size = gso_size;
+         vnet_buffer2 (hb)->gso_l4_hdr_sz = l4_hdr_sz;
+         hb->flags |= VNET_BUFFER_F_GSO;
+       }
+    }
+  else if (rx_comp->flags & VMXNET3_RXCF_IP6)
+    {
+      vnet_buffer (hb)->l2_hdr_offset = 0;
+      vnet_buffer (hb)->l3_hdr_offset = sizeof (ethernet_header_t);
+      vnet_buffer (hb)->l4_hdr_offset = sizeof (ethernet_header_t) +
+       sizeof (ip6_header_t);
+      hb->flags |= VNET_BUFFER_F_L2_HDR_OFFSET_VALID |
+       VNET_BUFFER_F_L3_HDR_OFFSET_VALID |
+       VNET_BUFFER_F_L4_HDR_OFFSET_VALID | VNET_BUFFER_F_IS_IP6;
+
+      /* checksum offload */
+      if (!(rx_comp->index & VMXNET3_RXCI_CNC))
+       {
+         if (!(rx_comp->flags & VMXNET3_RXCF_TUC))
+           {
+             if (rx_comp->flags & VMXNET3_RXCF_TCP)
+               {
+                 tcp_header_t *tcp =
+                   (tcp_header_t *) (hb->data +
+                                     vnet_buffer (hb)->l4_hdr_offset);
+                 hb->flags |= VNET_BUFFER_F_OFFLOAD_TCP_CKSUM;
+                 tcp->checksum = 0;
+               }
+             else if (rx_comp->flags & VMXNET3_RXCF_UDP)
+               {
+                 udp_header_t *udp =
+                   (udp_header_t *) (hb->data +
+                                     vnet_buffer (hb)->l4_hdr_offset);
+                 hb->flags |= VNET_BUFFER_F_OFFLOAD_UDP_CKSUM;
+                 udp->checksum = 0;
+               }
+           }
+       }
+
+      if (gso_size)
+       {
+         if (rx_comp->flags & VMXNET3_RXCF_TCP)
+           {
+             tcp_header_t *tcp =
+               (tcp_header_t *) (hb->data + vnet_buffer (hb)->l4_hdr_offset);
+             l4_hdr_sz = tcp_header_bytes (tcp);
+           }
+         else if (rx_comp->flags & VMXNET3_RXCF_UDP)
+           {
+             udp_header_t *udp =
+               (udp_header_t *) (hb->data + vnet_buffer (hb)->l4_hdr_offset);
+             l4_hdr_sz = sizeof (*udp);
+           }
+         vnet_buffer2 (hb)->gso_size = gso_size;
+         vnet_buffer2 (hb)->gso_l4_hdr_sz = l4_hdr_sz;
+         hb->flags |= VNET_BUFFER_F_GSO;
+       }
+    }
+}
+
 static_always_inline uword
 vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                             vlib_frame_t * frame, vmxnet3_device_t * vd,
@@ -79,7 +205,6 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
   uword n_trace = vlib_get_trace_count (vm, node);
   u32 n_rx_packets = 0, n_rx_bytes = 0;
   vmxnet3_rx_comp *rx_comp;
-  u32 comp_idx;
   u32 desc_idx;
   vmxnet3_rxq_t *rxq;
   u32 thread_index = vm->thread_index;
@@ -93,21 +218,21 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
   u8 known_next = 0, got_packet = 0;
   vmxnet3_rx_desc *rxd;
   clib_error_t *error;
+  u16 gso_size = 0;
 
   rxq = vec_elt_at_index (vd->rxqs, qid);
   comp_ring = &rxq->rx_comp_ring;
   bi = buffer_indices;
   next = nexts;
-  while (PREDICT_TRUE (n_rx_packets < VLIB_FRAME_SIZE) &&
-        (comp_ring->gen ==
-         (rxq->rx_comp[comp_ring->next].flags & VMXNET3_RXCF_GEN)))
+  rx_comp = &rxq->rx_comp[comp_ring->next];
+
+  while (PREDICT_TRUE ((n_rx_packets < VLIB_FRAME_SIZE) &&
+                      (comp_ring->gen ==
+                       (rx_comp->flags & VMXNET3_RXCF_GEN))))
     {
       vlib_buffer_t *b0;
       u32 bi0;
 
-      comp_idx = comp_ring->next;
-      rx_comp = &rxq->rx_comp[comp_idx];
-
       rid = vmxnet3_find_rid (vd, rx_comp);
       ring = &rxq->rx_ring[rid];
 
@@ -117,10 +242,15 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
        {
          vlib_error_count (vm, node->node_index,
                            VMXNET3_INPUT_ERROR_NO_BUFFER, 1);
+         if (hb)
+           {
+             vlib_buffer_free_one (vm, vlib_get_buffer_index (vm, hb));
+             hb = 0;
+           }
+         prev_b0 = 0;
          break;
        }
 
-      vmxnet3_rx_comp_ring_advance_next (rxq);
       desc_idx = rx_comp->index & VMXNET3_RXC_INDEX;
       ring->consume = desc_idx;
       rxd = &rxq->rx_desc[rid][desc_idx];
@@ -139,27 +269,34 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
       b0->flags = 0;
       b0->error = 0;
       b0->current_config_index = 0;
-      ASSERT (b0->current_length != 0);
 
       if (PREDICT_FALSE ((rx_comp->index & VMXNET3_RXCI_EOP) &&
                         (rx_comp->len & VMXNET3_RXCL_ERROR)))
        {
          vlib_buffer_free_one (vm, bi0);
          vlib_error_count (vm, node->node_index,
-                           VMXNET3_INPUT_ERROR_RX_PACKET, 1);
+                           VMXNET3_INPUT_ERROR_RX_PACKET_EOP, 1);
          if (hb && vlib_get_buffer_index (vm, hb) != bi0)
            {
              vlib_buffer_free_one (vm, vlib_get_buffer_index (vm, hb));
              hb = 0;
            }
          prev_b0 = 0;
-         continue;
+         goto next;
        }
 
       if (rx_comp->index & VMXNET3_RXCI_SOP)
        {
          ASSERT (!(rxd->flags & VMXNET3_RXF_BTYPE));
          /* start segment */
+         if (vd->gso_enable &&
+             (rx_comp->flags & VMXNET3_RXCF_CT) == VMXNET3_RXCOMP_TYPE_LRO)
+           {
+             vmxnet3_rx_comp_ext *lro = (vmxnet3_rx_comp_ext *) rx_comp;
+
+             gso_size = lro->flags & VMXNET3_RXECF_MSS_MASK;
+           }
+
          hb = b0;
          bi[0] = bi0;
          if (!(rx_comp->index & VMXNET3_RXCI_EOP))
@@ -179,12 +316,19 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
       else if (rx_comp->index & VMXNET3_RXCI_EOP)
        {
          /* end of segment */
-         if (prev_b0)
+         if (PREDICT_TRUE (prev_b0 != 0))
            {
-             prev_b0->flags |= VLIB_BUFFER_NEXT_PRESENT;
-             prev_b0->next_buffer = bi0;
-             hb->total_length_not_including_first_buffer +=
-               b0->current_length;
+             if (PREDICT_TRUE (b0->current_length != 0))
+               {
+                 prev_b0->flags |= VLIB_BUFFER_NEXT_PRESENT;
+                 prev_b0->next_buffer = bi0;
+                 hb->total_length_not_including_first_buffer +=
+                   b0->current_length;
+               }
+             else
+               {
+                 vlib_buffer_free_one (vm, bi0);
+               }
              prev_b0 = 0;
              got_packet = 1;
            }
@@ -199,7 +343,7 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                  vlib_buffer_free_one (vm, vlib_get_buffer_index (vm, hb));
                  hb = 0;
                }
-             continue;
+             goto next;
            }
        }
       else if (prev_b0)                // !sop && !eop
@@ -213,15 +357,21 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
        }
       else
        {
-         ASSERT (0);
+         vlib_error_count (vm, node->node_index,
+                           VMXNET3_INPUT_ERROR_RX_PACKET, 1);
+         vlib_buffer_free_one (vm, bi0);
+         if (hb && vlib_get_buffer_index (vm, hb) != bi0)
+           {
+             vlib_buffer_free_one (vm, vlib_get_buffer_index (vm, hb));
+             hb = 0;
+           }
+         goto next;
        }
 
       n_rx_bytes += b0->current_length;
 
       if (got_packet)
        {
-         ethernet_header_t *e = (ethernet_header_t *) hb->data;
-
          if (PREDICT_FALSE (vd->per_interface_next_index != ~0))
            {
              next_index = vd->per_interface_next_index;
@@ -237,36 +387,14 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
            }
 
          if (PREDICT_FALSE (known_next))
-           {
-             next[0] = next_index;
-           }
+           next[0] = next_index;
          else
            {
-             if (ethernet_frame_is_tagged (e->type))
-               next[0] = VNET_DEVICE_INPUT_NEXT_ETHERNET_INPUT;
-             else
-               {
-                 if (rx_comp->flags & VMXNET3_RXCF_IP4)
-                   {
-                     next[0] = VNET_DEVICE_INPUT_NEXT_IP4_NCS_INPUT;
-                     hb->flags |= VNET_BUFFER_F_IS_IP4;
-                     vlib_buffer_advance (hb,
-                                          device_input_next_node_advance
-                                          [next[0]]);
-                   }
-                 else if (rx_comp->flags & VMXNET3_RXCF_IP6)
-                   {
-                     next[0] = VNET_DEVICE_INPUT_NEXT_IP6_INPUT;
-                     hb->flags |= VNET_BUFFER_F_IS_IP6;
-                     vlib_buffer_advance (hb,
-                                          device_input_next_node_advance
-                                          [next[0]]);
-                   }
-                 else
-                   {
-                     next[0] = VNET_DEVICE_INPUT_NEXT_ETHERNET_INPUT;
-                   }
-               }
+             ethernet_header_t *e = (ethernet_header_t *) hb->data;
+
+             next[0] = VNET_DEVICE_INPUT_NEXT_ETHERNET_INPUT;
+             if (!ethernet_frame_is_tagged (ntohs (e->type)))
+               vmxnet3_handle_offload (rx_comp, hb, gso_size);
            }
 
          n_rx_packets++;
@@ -274,7 +402,12 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
          bi++;
          hb = 0;
          got_packet = 0;
+         gso_size = 0;
        }
+
+    next:
+      vmxnet3_rx_comp_ring_advance_next (rxq);
+      rx_comp = &rxq->rx_comp[comp_ring->next];
     }
 
   if (PREDICT_FALSE ((n_trace = vlib_get_trace_count (vm, node))))
@@ -285,17 +418,18 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
       next = nexts;
       while (n_trace && n_left)
        {
-         vlib_buffer_t *b;
-         vmxnet3_input_trace_t *tr;
-
-         b = vlib_get_buffer (vm, bi[0]);
-         vlib_trace_buffer (vm, node, next[0], b, /* follow_chain */ 0);
-         tr = vlib_add_trace (vm, node, b, sizeof (*tr));
-         tr->next_index = next[0];
-         tr->hw_if_index = vd->hw_if_index;
-         tr->buffer = *b;
-
-         n_trace--;
+         vlib_buffer_t *b = vlib_get_buffer (vm, bi[0]);
+         if (PREDICT_TRUE
+             (vlib_trace_buffer
+              (vm, node, next[0], b, /* follow_chain */ 0)))
+           {
+             vmxnet3_input_trace_t *tr =
+               vlib_add_trace (vm, node, b, sizeof (*tr));
+             tr->next_index = next[0];
+             tr->hw_if_index = vd->hw_if_index;
+             tr->buffer = *b;
+             n_trace--;
+           }
          n_left--;
          bi++;
          next++;
@@ -310,7 +444,7 @@ vmxnet3_device_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
       vlib_increment_combined_counter
        (vnm->interface_main.combined_sw_if_counters +
         VNET_INTERFACE_COUNTER_RX, thread_index,
-        vd->hw_if_index, n_rx_packets, n_rx_bytes);
+        vd->sw_if_index, n_rx_packets, n_rx_bytes);
     }
 
   error = vmxnet3_rxq_refill_ring0 (vm, vd, rxq);
@@ -354,6 +488,7 @@ VLIB_NODE_FN (vmxnet3_input_node) (vlib_main_t * vm,
 VLIB_REGISTER_NODE (vmxnet3_input_node) = {
   .name = "vmxnet3-input",
   .sibling_of = "device-input",
+  .flags = VLIB_NODE_FLAG_TRACE_SUPPORTED,
   .format_trace = format_vmxnet3_input_trace,
   .type = VLIB_NODE_TYPE_INPUT,
   .state = VLIB_NODE_STATE_DISABLED,