diff options
| author |   Nathan Skrzypczak <nathan.skrzypczak@gmail.com> | 2020-11-02 17:29:52 +0100 |
|---|---|---|
| committer |   Matthew Smith <mgsmith@netgate.com> | 2020-11-05 16:47:38 +0000 |
| commit | 38049e48a7a8da115f555bbb476b0fdbfb9cbb76 (patch) | |
| tree | 1d7db4dde8e977a36eaf28cc5a57fcbf7b92573a /src/plugins/cnat/cnat_node_snat.c | |
| parent | e019cacbe2399fadd41b3f22b6837431dd631e2a (diff) | |
cnat: reduce compile time
Compile time reduced from ~30s to 2s for
cnat_node_vip.c & cnat_node_snat.c This doesn't
impact performance for now as ts update rwlock
is the main bottleneck.
Type: improvement
Change-Id: Ic92df300ae0dfddc5235c350bd021e73e7c850d9
Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
Diffstat (limited to 'src/plugins/cnat/cnat_node_snat.c')
| -rw-r--r-- | src/plugins/cnat/cnat_node_snat.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/src/plugins/cnat/cnat_node_snat.c b/src/plugins/cnat/cnat_node_snat.c index d92200fec05..8166df6a757 100644 --- a/src/plugins/cnat/cnat_node_snat.c +++ b/src/plugins/cnat/cnat_node_snat.c @@ -55,17 +55,17 @@ format_cnat_snat_trace (u8 * s, va_list * args) /* CNat sub for source NAT as a feature arc on ip[46]-unicast This node's sub shouldn't apply to the same flows as cnat_vip_inline */ -always_inline uword -cnat_snat_inline (vlib_main_t * vm, - vlib_node_runtime_t * node, - vlib_buffer_t * b, - cnat_node_ctx_t * ctx, int rv, cnat_session_t * session) +static uword +cnat_snat_node_fn (vlib_main_t * vm, + vlib_node_runtime_t * node, + vlib_buffer_t * b, + cnat_node_ctx_t * ctx, int rv, cnat_session_t * session) { cnat_main_t *cm = &cnat_main; int created_session = 0; - ip4_header_t *ip4; + ip4_header_t *ip4 = NULL; ip_protocol_t iproto; - ip6_header_t *ip6; + ip6_header_t *ip6 = NULL; udp_header_t *udp0; u32 arc_next0; u16 next0; @@ -182,9 +182,9 @@ VLIB_NODE_FN (cnat_snat_ip4_node) (vlib_main_t * vm, vlib_frame_t * frame) { if (PREDICT_FALSE ((node->flags & VLIB_NODE_FLAG_TRACE))) - return cnat_node_inline (vm, node, frame, cnat_snat_inline, AF_IP4, + return cnat_node_inline (vm, node, frame, cnat_snat_node_fn, AF_IP4, 1 /* do_trace */ ); - return cnat_node_inline (vm, node, frame, cnat_snat_inline, AF_IP4, + return cnat_node_inline (vm, node, frame, cnat_snat_node_fn, AF_IP4, 0 /* do_trace */ ); } @@ -193,9 +193,9 @@ VLIB_NODE_FN (cnat_snat_ip6_node) (vlib_main_t * vm, vlib_frame_t * frame) { if (PREDICT_FALSE ((node->flags & VLIB_NODE_FLAG_TRACE))) - return cnat_node_inline (vm, node, frame, cnat_snat_inline, AF_IP6, + return cnat_node_inline (vm, node, frame, cnat_snat_node_fn, AF_IP6, 1 /* do_trace */ ); - return cnat_node_inline (vm, node, frame, cnat_snat_inline, AF_IP6, + return cnat_node_inline (vm, node, frame, cnat_snat_node_fn, AF_IP6, 0 /* do_trace */ ); } |