diff options
| author |   Andreas Schultz <andreas.schultz@travelping.com> | 2019-07-15 15:40:56 +0200 |
|---|---|---|
| committer |   Dave Barach <openvpp@barachs.net> | 2019-07-18 15:07:26 +0000 |
| commit | 58b2eb1af562c292feb6d3cdce4656746e61da75 (patch) | |
| tree | ba2d239dd4a2d4673ef1352143d1133a9f88ccd4 /src/vnet/unix/gdb_funcs.c | |
| parent | a0cb32cb9fba93ec4b91e7d227c07d1b2099091b (diff) | |
vlib: convert frame_index into real pointers
The fast path almost always has to deal with the real
pointers. Deriving the frame pointer from a frame_index requires a
load of the 32bit frame_index from memory, another 64bit load of the
heap base pointer and some calculations.
Lets store the full pointer instead and do a single 64bit load only.
This helps avoiding problems when the heap is grown and frames are
allocated below vm->heap_aligned_base.
Type: refactor
Change-Id: Ifa6e6e984aafe1e2755bff80f0a4dfcddee3623c
Signed-off-by: Andreas Schultz <andreas.schultz@travelping.com>
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/vnet/unix/gdb_funcs.c')
| -rw-r--r-- | src/vnet/unix/gdb_funcs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/vnet/unix/gdb_funcs.c b/src/vnet/unix/gdb_funcs.c index 7349897d35d..2ebdd5960ca 100644 --- a/src/vnet/unix/gdb_funcs.c +++ b/src/vnet/unix/gdb_funcs.c @@ -111,8 +111,8 @@ vlib_dump_frame_ownership (void) nm->nodes[this_node_runtime->node_index]->name, index - first_nf_index, nm->nodes[owned_runtime->node_index]->name); - fformat (stderr, " nf index %d nf->frame_index %d\n", - nf - vm->node_main.next_frames, nf->frame_index); + fformat (stderr, " nf index %d nf->frame %p\n", + nf - vm->node_main.next_frames, nf->frame); } } } |