diff options
| author |   Zhiyong Yang <zhiyong.yang@intel.com> | 2018-09-04 22:45:17 -0400 |
|---|---|---|
| committer |   John Lo <loj@cisco.com> | 2018-09-06 22:08:45 +0000 |
| commit | 70fda4c443be1c7893e88c68f36b57f8168f4e54 (patch) | |
| tree | 8e7eb1e8d61d38c8da611c7b1ce135fb182d8948 /src/vnet/vxlan/encap.c | |
| parent | ecdfb2cbee547a113d3700da07c8920924bfb62d (diff) | |
vxlan: improve encap performance
1. For vxlan, prefetching one cacheline is enough.
2. Reduce vlib_increment_combined_counter functtion calling
if possible.
Change-Id: If3a72ac40c8988caaf0d5915b695f86d799f15a9
Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
Diffstat (limited to 'src/vnet/vxlan/encap.c')
| -rw-r--r-- | src/vnet/vxlan/encap.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/vnet/vxlan/encap.c b/src/vnet/vxlan/encap.c index c34b0e67221..fdca425889e 100644 --- a/src/vnet/vxlan/encap.c +++ b/src/vnet/vxlan/encap.c @@ -116,8 +116,8 @@ vxlan_encap_inline (vlib_main_t * vm, vlib_prefetch_buffer_header (p2, LOAD); vlib_prefetch_buffer_header (p3, LOAD); - CLIB_PREFETCH (p2->data, 2*CLIB_CACHE_LINE_BYTES, LOAD); - CLIB_PREFETCH (p3->data, 2*CLIB_CACHE_LINE_BYTES, LOAD); + CLIB_PREFETCH (p2->data, CLIB_CACHE_LINE_BYTES, LOAD); + CLIB_PREFETCH (p3->data, CLIB_CACHE_LINE_BYTES, LOAD); } u32 bi0 = to_next[0] = from[0]; @@ -292,10 +292,18 @@ vxlan_encap_inline (vlib_main_t * vm, udp1->checksum = 0xffff; } + if (sw_if_index0 == sw_if_index1) + { + vlib_increment_combined_counter (tx_counter, thread_index, + sw_if_index0, 2, len0 + len1); + } + else + { vlib_increment_combined_counter (tx_counter, thread_index, sw_if_index0, 1, len0); vlib_increment_combined_counter (tx_counter, thread_index, sw_if_index1, 1, len1); + } pkts_encapsulated += 2; if (PREDICT_FALSE(b0->flags & VLIB_BUFFER_IS_TRACED)) |