diff options
| author |   Dave Barach <dbarach@cisco.com> | 2019-03-10 09:44:51 -0400 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2019-03-10 23:16:43 +0000 |
| commit | 593eedf2561f639703d366cd21ab0c3ae03bdb0c (patch) | |
| tree | 75c7ed53b941d46127d1c6b88a16f5936570d2ce /src/vlib/threads.c | |
| parent | c7886c5d50782b9653ab41772fb93cff0c0ad9da (diff) | |
Perf tune get_frame_size_info
It turns out that for scalar sizes 0..24, frames are always the same
size. That range includes all current use-cases - and then some - so
get rid of the hash table. Old code preserved under #ifdef
VLIB_SUPPORTS_ARBITRARY_SCALAR_SIZES.
Change-Id: Ic005c7143c9639f77d1a0fadd2fc0e90dccb68c1
Signed-off-by: Dave Barach <dbarach@cisco.com>
Diffstat (limited to 'src/vlib/threads.c')
| -rw-r--r-- | src/vlib/threads.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/vlib/threads.c b/src/vlib/threads.c index e2f7d2634b9..7d17c7b37cc 100644 --- a/src/vlib/threads.c +++ b/src/vlib/threads.c @@ -866,9 +866,11 @@ start_workers (vlib_main_t * vm) nm_clone->processes = vec_dup_aligned (nm->processes, CLIB_CACHE_LINE_BYTES); - /* zap the (per worker) frame freelists, etc */ - nm_clone->frame_sizes = 0; + /* Create per-thread frame freelist */ + nm_clone->frame_sizes = vec_new (vlib_frame_size_t, 1); +#ifdef VLIB_SUPPORTS_ARBITRARY_SCALAR_SIZES nm_clone->frame_size_hash = hash_create (0, sizeof (uword)); +#endif /* Packet trace buffers are guaranteed to be empty, nothing to do here */ |