diff options
| author |   Mohsin Kazmi <sykazmi@cisco.com> | 2022-10-25 19:23:51 +0000 |
|---|---|---|
| committer | Mohsin Kazmi <sykazmi@cisco.com> | 2022-10-25 19:29:30 +0000 |
| commit | 4561d6d251844b504a024d569de8d876203047ac (patch) | |
| tree | 357dde8ee68ff6a8e6584f5f54415cd0ddb2ae60 | |
| parent | 4c89b189b2c712894764837a78e0a532bfe7e12f (diff) | |
devices: fix the l2 hdr offset in af_packet transmit side
Type: fix
af_packet transmit node uses header offsets to set the
appropriate metadata for packet with offload. But
l2_hdr_offset is not correctly set by previous node.
This patch makes use of curren_data field as l2_hdr_offset.
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
Change-Id: I751a725af5c231a443eed22231a867eb7f3894e0
| -rw-r--r-- | src/vnet/devices/af_packet/device.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/vnet/devices/af_packet/device.c b/src/vnet/devices/af_packet/device.c index 2e9b7a4ed83..17291f76b71 100644 --- a/src/vnet/devices/af_packet/device.c +++ b/src/vnet/devices/af_packet/device.c @@ -291,8 +291,7 @@ static_always_inline void fill_gso_offload (vlib_buffer_t *b0, vnet_virtio_net_hdr_t *vnet_hdr) { vnet_buffer_oflags_t oflags = vnet_buffer (b0)->oflags; - i16 l4_hdr_offset = - vnet_buffer (b0)->l4_hdr_offset - vnet_buffer (b0)->l2_hdr_offset; + i16 l4_hdr_offset = vnet_buffer (b0)->l4_hdr_offset - b0->current_data; if (b0->flags & VNET_BUFFER_F_IS_IP4) { ip4_header_t *ip4; @@ -329,8 +328,7 @@ static_always_inline void fill_cksum_offload (vlib_buffer_t *b0, vnet_virtio_net_hdr_t *vnet_hdr) { vnet_buffer_oflags_t oflags = vnet_buffer (b0)->oflags; - i16 l4_hdr_offset = - vnet_buffer (b0)->l4_hdr_offset - vnet_buffer (b0)->l2_hdr_offset; + i16 l4_hdr_offset = vnet_buffer (b0)->l4_hdr_offset - b0->current_data; if (b0->flags & VNET_BUFFER_F_IS_IP4) { ip4_header_t *ip4; |