commit b54db102acdae5694f21d6617eef52f8f7f77c29
author Damjan Marion <damarion@cisco.com> Thu Feb 18 10:11:34 2016 +0100
committer Dave Barach <openvpp@barachs.net> Tue Jun 07 19:29:28 2016 +0000
tree b0e9da4c9a599257855032dce40765f8e9806cf2
parent 19414deeaa15ea9434f86d40a5bc7ca43916eab5

Adjust buffer data offset based on rte_mbuff data_off

This commit removes assumption that all packets have
data offset equal to RTE_PKTMBUF_HEADROOM. Some drivers
like fm10K receive packets with different data offset.

Change-Id: I0aba6296458dab9df6fff639b4b827b2084ddc3e
Signed-off-by: Damjan Marion <damarion@cisco.com>

vnet/vnet/devices/dpdk/node.c

diff --git a/vnet/vnet/devices/dpdk/node.c b/vnet/vnet/devices/dpdk/node.c
index 0dd3cfc..2c90d80 100644
--- a/vnet/vnet/devices/dpdk/node.c
+++ b/vnet/vnet/devices/dpdk/node.c
@@ -502,7 +502,6 @@
   u32 n_trace, trace_cnt __attribute__((unused));
   vlib_buffer_free_list_t * fl;
   u8 efd_discard_burst = 0;
-  u16 ip_align_offset = 0;
   u32 buffer_flags_template;
   
   if (xd->admin_up == 0)
@@ -522,9 +521,6 @@
       return 0;
     }
 
-  if (xd->pmd == VNET_DPDK_PMD_THUNDERX)
-      ip_align_offset = 6;
-
   buffer_flags_template = dm->buffer_flags_template;
 
   vec_reset_length (xd->d_trace_buffers);
@@ -673,15 +669,11 @@
 			sizeof (ethernet_header_t) : 0);
 
           b0->current_data = l3_offset0;
+          /* Some drivers like fm10k receive frames with
+             mb->data_off > RTE_PKTMBUF_HEADROOM */
+          b0->current_data += mb->data_off - RTE_PKTMBUF_HEADROOM;
           b0->current_length = mb->data_len - l3_offset0;
 
-          if (PREDICT_FALSE (ip_align_offset != 0))
-            {
-              if (next0 == DPDK_RX_NEXT_IP4_INPUT ||
-                  next0 == DPDK_RX_NEXT_IP6_INPUT)
-                b0->current_data += ip_align_offset;
-            }
-             
           b0->flags = buffer_flags_template;
 
           if (VMWARE_LENGTH_BUG_WORKAROUND)
@@ -1390,6 +1382,9 @@
                             sizeof (ethernet_header_t) : 0);
 
               b0->current_data = l3_offset0;
+              /* Some drivers like fm10k receive frames with
+                 mb->data_off > RTE_PKTMBUF_HEADROOM */
+              b0->current_data += mb->data_off - RTE_PKTMBUF_HEADROOM;
               b0->current_length = mb->data_len - l3_offset0;
 
               b0->flags = buffer_flags_template;