diff options
| author |   Dave Barach <dave@barachs.net> | 2017-02-15 09:01:01 -0500 |
|---|---|---|
| committer |   Damjan Marion <damarion@cisco.com> | 2017-02-17 13:32:42 +0100 |
| commit | 681abe46c9ea30e5c386db74c997714433470c4c (patch) | |
| tree | 748946309bca26c11ebaa460689ba91a66d2313a /src/vnet/l2/l2_fwd.c | |
| parent | 6e9cf3b9b53e69408514f9451235119a31045ed6 (diff) | |
l2 input: avoid per-packet trace checks in the fast path
Change-Id: Ib0c8572773499d8dd4d81b3a565c24412ccc3510
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/vnet/l2/l2_fwd.c')
| -rw-r--r-- | src/vnet/l2/l2_fwd.c | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/src/vnet/l2/l2_fwd.c b/src/vnet/l2/l2_fwd.c index 710a9d9e8c3..f7e2ccb65b7 100644 --- a/src/vnet/l2/l2_fwd.c +++ b/src/vnet/l2/l2_fwd.c @@ -195,9 +195,9 @@ l2fwd_process (vlib_main_t * vm, } -static uword -l2fwd_node_fn (vlib_main_t * vm, - vlib_node_runtime_t * node, vlib_frame_t * frame) +static_always_inline uword +l2fwd_node_inline (vlib_main_t * vm, vlib_node_runtime_t * node, + vlib_frame_t * frame, int do_trace) { u32 n_left_from, *from, *to_next; l2fwd_next_t next_index; @@ -281,7 +281,7 @@ l2fwd_node_fn (vlib_main_t * vm, h2 = vlib_buffer_get_current (b2); h3 = vlib_buffer_get_current (b3); - if (PREDICT_FALSE ((node->flags & VLIB_NODE_FLAG_TRACE))) + if (do_trace) { if (b0->flags & VLIB_BUFFER_IS_TRACED) { @@ -388,8 +388,7 @@ l2fwd_node_fn (vlib_main_t * vm, h0 = vlib_buffer_get_current (b0); - if (PREDICT_FALSE ((node->flags & VLIB_NODE_FLAG_TRACE) - && (b0->flags & VLIB_BUFFER_IS_TRACED))) + if (do_trace && PREDICT_FALSE (b0->flags & VLIB_BUFFER_IS_TRACED)) { l2fwd_trace_t *t = vlib_add_trace (vm, node, b0, sizeof (*t)); t->sw_if_index = sw_if_index0; @@ -420,6 +419,15 @@ l2fwd_node_fn (vlib_main_t * vm, return frame->n_vectors; } +static uword +l2fwd_node_fn (vlib_main_t * vm, + vlib_node_runtime_t * node, vlib_frame_t * frame) +{ + if (PREDICT_FALSE ((node->flags & VLIB_NODE_FLAG_TRACE))) + return l2fwd_node_inline (vm, node, frame, 1 /* do_trace */ ); + return l2fwd_node_inline (vm, node, frame, 0 /* do_trace */ ); +} + /* *INDENT-OFF* */ VLIB_REGISTER_NODE (l2fwd_node,static) = { .function = l2fwd_node_fn, |