aboutsummaryrefslogtreecommitdiffstats
path: root/hicn-plugin/src
diff options
context:
space:
mode:
authorMauro Sardara <msardara@cisco.com>2020-11-10 14:50:26 +0000
committerMauro Sardara <msardara@cisco.com>2020-11-10 15:16:04 +0000
commit014ea2ed39986793d6b82e7a43533e91c6ec2e18 (patch)
tree6aec6df61debd6c40049e534b3a4377c851160fc /hicn-plugin/src
parentf87f19fa389a1edea861faaabc972d508a58985c (diff)
[HICN-651] Offload checksum computation also for data packets.
Signed-off-by: Mauro Sardara <msardara@cisco.com> Change-Id: I3663ac73cdbaad3e53d4b1973a7bcbb333b3910c
Diffstat (limited to 'hicn-plugin/src')
-rw-r--r--hicn-plugin/src/CMakeLists.txt1
-rw-r--r--hicn-plugin/src/faces/face_node.c22
-rw-r--r--hicn-plugin/src/faces/iface_node.c22
-rw-r--r--hicn-plugin/src/faces/inlines.h42
4 files changed, 63 insertions, 24 deletions
diff --git a/hicn-plugin/src/CMakeLists.txt b/hicn-plugin/src/CMakeLists.txt
index 97284b218..a3480c21d 100644
--- a/hicn-plugin/src/CMakeLists.txt
+++ b/hicn-plugin/src/CMakeLists.txt
@@ -125,6 +125,7 @@ set(HICN_PLUGIN_HEADER_FILES
${CMAKE_CURRENT_SOURCE_DIR}/faces/face.h
${CMAKE_CURRENT_SOURCE_DIR}/faces/face_node.h
${CMAKE_CURRENT_SOURCE_DIR}/faces/iface_node.h
+ ${CMAKE_CURRENT_SOURCE_DIR}/faces/inlines.h
${CMAKE_CURRENT_SOURCE_DIR}/faces/face_dpo.h
${CMAKE_CURRENT_SOURCE_DIR}/faces/app/address_mgr.h
${CMAKE_CURRENT_SOURCE_DIR}/faces/app/face_cons.h
diff --git a/hicn-plugin/src/faces/face_node.c b/hicn-plugin/src/faces/face_node.c
index fd8649e6b..48d97ad51 100644
--- a/hicn-plugin/src/faces/face_node.c
+++ b/hicn-plugin/src/faces/face_node.c
@@ -16,6 +16,7 @@
#include <vnet/adj/adj.h>
#include "face.h"
+#include "inlines.h"
#include "face_node.h"
#include "../strategy_dpo_manager.h"
#include "face.h"
@@ -558,7 +559,6 @@ hicn_face_rewrite_interest (vlib_main_t * vm, vlib_buffer_t * b0,
/* return; */
hicn_header_t *hicn = vlib_buffer_get_current (b0);
- size_t l3_header_size = sizeof(ip6_header_t);
//hicn_face_ip_t *ip_face = (hicn_face_ip_t *) face->data;
@@ -568,24 +568,10 @@ hicn_face_rewrite_interest (vlib_main_t * vm, vlib_buffer_t * b0,
int ret = hicn_ops_vft[type.l1]->rewrite_interest (type, &hicn->protocol,
&face->nat_addr, &temp_addr);
- if (ip46_address_is_ip4(&face->nat_addr)) {
- b0->flags |= VNET_BUFFER_F_OFFLOAD_IP_CKSUM;
- l3_header_size = sizeof(ip4_header_t);
- }
-
- if (ret == HICN_LIB_ERROR_REWRITE_CKSUM_REQUIRED) {
- b0->flags |= VNET_BUFFER_F_OFFLOAD_TCP_CKSUM;
-
- /* Make sure l3_hdr_offset and l4_hdr_offset are set */
- if (!(b0->flags & VNET_BUFFER_F_L3_HDR_OFFSET_VALID)) {
- b0->flags |= VNET_BUFFER_F_L3_HDR_OFFSET_VALID;
- vnet_buffer (b0)->l3_hdr_offset = b0->current_data;
- }
- if (!(b0->flags & VNET_BUFFER_F_L4_HDR_OFFSET_VALID)) {
- b0->flags |= VNET_BUFFER_F_L4_HDR_OFFSET_VALID;
- vnet_buffer (b0)->l4_hdr_offset = vnet_buffer (b0)->l3_hdr_offset + l3_header_size;
+ if (ret == HICN_LIB_ERROR_REWRITE_CKSUM_REQUIRED)
+ {
+ ensure_offload_flags(b0, ip46_address_is_ip4(&face->nat_addr));
}
- }
ASSERT(face->flags & HICN_FACE_FLAGS_FACE);
diff --git a/hicn-plugin/src/faces/iface_node.c b/hicn-plugin/src/faces/iface_node.c
index 433cf0b02..a4fd1885c 100644
--- a/hicn-plugin/src/faces/iface_node.c
+++ b/hicn-plugin/src/faces/iface_node.c
@@ -14,6 +14,7 @@
*/
#include "face.h"
+#include "inlines.h"
#include "../strategy_dpo_manager.h"
#include "../hicn.h"
#include "../infra.h"
@@ -554,9 +555,13 @@ hicn_rewrite_iface_data4 (vlib_main_t * vm, vlib_buffer_t * b0,
ip46_address_t temp_addr;
ip46_address_reset (&temp_addr);
hicn_type_t type = hicn_get_buffer (b0)->type;
- hicn_ops_vft[type.l1]->rewrite_data (type, &hicn->protocol,
- &(iface->nat_addr), &(temp_addr),
- iface->pl_id);
+ int ret = hicn_ops_vft[type.l1]->rewrite_data (type, &hicn->protocol,
+ &(iface->nat_addr), &(temp_addr),
+ iface->pl_id);
+ if (ret == HICN_LIB_ERROR_REWRITE_CKSUM_REQUIRED)
+ {
+ ensure_offload_flags(b0, 1 /* is_v4 */);
+ }
}
static inline void
@@ -583,9 +588,14 @@ hicn_rewrite_iface_data6 (vlib_main_t * vm, vlib_buffer_t * b0,
ip46_address_t temp_addr;
ip46_address_reset (&temp_addr);
hicn_type_t type = hicn_get_buffer (b0)->type;
- hicn_ops_vft[type.l1]->rewrite_data (type, &hicn->protocol,
- &(iface->nat_addr), &(temp_addr),
- iface->pl_id);
+ int ret = hicn_ops_vft[type.l1]->rewrite_data (type, &hicn->protocol,
+ &(iface->nat_addr), &(temp_addr),
+ iface->pl_id);
+
+ if (ret == HICN_LIB_ERROR_REWRITE_CKSUM_REQUIRED)
+ {
+ ensure_offload_flags(b0, 0 /* is_v4 */);
+ }
}
#define iface_output_x1(ipv) \
diff --git a/hicn-plugin/src/faces/inlines.h b/hicn-plugin/src/faces/inlines.h
new file mode 100644
index 000000000..bfe56c8e6
--- /dev/null
+++ b/hicn-plugin/src/faces/inlines.h
@@ -0,0 +1,42 @@
+/*
+ * Copyright (c) 2020 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.
+ */
+
+#ifndef __HICN_FACE_INLINES_H__
+#define __HICN_FACE_INLINES_H__
+
+#include <vlib/buffer.h>
+
+always_inline void
+ensure_offload_flags (vlib_buffer_t * b, int is_v4)
+{
+ b->flags |= VNET_BUFFER_F_OFFLOAD_TCP_CKSUM;
+ b->flags |= is_v4 * VNET_BUFFER_F_OFFLOAD_IP_CKSUM;
+ size_t l3_header_size = is_v4 * sizeof(ip4_header_t) + (!is_v4) * sizeof(ip6_header_t);
+
+ /* Make sure l3_hdr_offset and l4_hdr_offset are set */
+ if (!(b->flags & VNET_BUFFER_F_L3_HDR_OFFSET_VALID))
+ {
+ b->flags |= VNET_BUFFER_F_L3_HDR_OFFSET_VALID;
+ vnet_buffer(b)->l3_hdr_offset = b->current_data;
+ }
+ if (!(b->flags & VNET_BUFFER_F_L4_HDR_OFFSET_VALID))
+ {
+ b->flags |= VNET_BUFFER_F_L4_HDR_OFFSET_VALID;
+ vnet_buffer(b)->l4_hdr_offset =
+ vnet_buffer(b)->l3_hdr_offset + l3_header_size;
+ }
+}
+
+#endif /* __HICN_FACE_INLINES_H__ */ \ No newline at end of file