acl-plugin: fill in the 5tuple structure all at once, avoid short writes 93/13493/3
authorAndrew Yourtchenko <ayourtch@gmail.com>
Mon, 16 Jul 2018 13:28:47 +0000 (15:28 +0200)
committerFlorin Coras <florin.coras@gmail.com>
Fri, 3 Aug 2018 01:18:13 +0000 (01:18 +0000)
This change avoids the long-read-after-short-write, resulting
in a small performance improvement.

Change-Id: Ic01d1fd19182e96100ccaf441d7ab5e5898b9734
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
src/plugins/acl/dataplane_node.c

index 7db0efa..00cad47 100644 (file)
@@ -144,11 +144,9 @@ acl_fa_node_fn (vlib_main_t * vm,
        * Extract the L3/L4 matching info into a 5-tuple structure.
        */
 
-      acl_plugin_fill_5tuple_inline (&acl_main, lc_index0, b[0], is_ip6,
-                                    is_input, is_l2_path,
-                                    (fa_5tuple_opaque_t *) & fa_5tuple);
-      fa_5tuple.l4.lsb_of_sw_if_index = sw_if_index0 & 0xffff;
-      fa_5tuple.pkt.mask_type_index_lsb = ~0;
+      acl_fill_5tuple (&acl_main, sw_if_index0, b[0], is_ip6,
+                      is_input, is_l2_path, &fa_5tuple);
+
 #ifdef FA_NODE_VERBOSE_DEBUG
       clib_warning
        ("ACL_FA_NODE_DBG: packet 5-tuple %016llx %016llx %016llx %016llx %016llx %016llx",