udp: align udp_encap_t_ to 2 cachelines 25/27525/3
authorVadym Martsynovskyy <vmartsyn@fb.com>
Fri, 12 Jun 2020 16:50:50 +0000 (09:50 -0700)
committerDamjan Marion <dmarion@me.com>
Sat, 27 Jun 2020 10:27:55 +0000 (10:27 +0000)
commit42386fc974148f812ef3eb73ff09a603caa23565
treeed58f0e607b72aa86a0d8cc2a092fa54ef058027
parentedc2ea435b5c407a78f4fcb42d750338d6cdd6d5
udp: align udp_encap_t_ to 2 cachelines

Based on the comments in the struct, udp_encap_t_ is meant to span 2
cachelines. Due to the 64 bit alignment of dpo_id_t, the struct spanned
3 cachelines. This caused fetching ue_ip_proto to trigger an additional
cache miss. This patch rearranges the ordering of the struct fields
so that udp_encap_t_ only spans 2 cachelines as intended.

before:
(gdb) print (int)&((struct udp_encap_t_*)0)->cacheline1
$8 = 128

after:
(gdb) print (int)&((struct udp_encap_t_*)0)->cacheline1
$1 = 64

Type: fix
Signed-off-by: Vadym Martsynovskyy <vmartsyn@fb.com>
Change-Id: I066c08654d4a8ef3e2d3954e957d4c5d382b209f
src/vnet/udp/udp_encap.h