buffers: improve cache occupancy 79/28679/2
authorDamjan Marion <damarion@cisco.com>
Fri, 4 Sep 2020 10:34:58 +0000 (12:34 +0200)
committerDamjan Marion <dmarion@me.com>
Fri, 4 Sep 2020 12:45:46 +0000 (12:45 +0000)
Adjust buffer allocation so it always have odd number of cache lines.
That should result in better distribution of cachelines among cache sets.

Type: improvement
Change-Id: I0d39d4cf01cff36ad6f70a700730823a96448c22
Signed-off-by: Damjan Marion <damarion@cisco.com>
src/vlib/buffer.c

index a645438..17e7d24 100644 (file)
@@ -507,8 +507,14 @@ vlib_buffer_chain_append_data_with_alloc (vlib_main_t * vm,
 static uword
 vlib_buffer_alloc_size (uword ext_hdr_size, uword data_size)
 {
-  return CLIB_CACHE_LINE_ROUND (ext_hdr_size + sizeof (vlib_buffer_t) +
-                               data_size);
+  uword alloc_size = ext_hdr_size + sizeof (vlib_buffer_t) + data_size;
+  alloc_size = CLIB_CACHE_LINE_ROUND (alloc_size);
+
+  /* in case when we have even number of cachelines, we add one more for
+   * better cache occupancy */
+  alloc_size |= CLIB_CACHE_LINE_BYTES;
+
+  return alloc_size;
 }
 
 u8