diff options
author | Dave Barach <dave@barachs.net> | 2019-07-23 10:22:31 -0400 |
---|---|---|
committer | Florin Coras <florin.coras@gmail.com> | 2019-07-23 17:02:04 +0000 |
commit | 687c9021fda009caa2b7eb17bea2eaa51d275bde (patch) | |
tree | f4b1ab097c40477a847d902430abdea5bf31acc3 /src/vlib/node.h | |
parent | 3b7261978ee4ffdc1e92336e708ae05e2be25f71 (diff) |
vlib: address vlib_error_t scaling issue
Encoding the vpp node index into the vlib_error_t as a 10-bit quantity
limits us to 1K graph nodes. Unfortunately, a few nodes need 6 bit
per-node error codes. Only a very few nodes have so many counters.
It turns out that there are about 2K total error counters in the system,
which is (approximately) the maximum error heap index.
The current (index,code) encoding limits the number of interfaces to
around 250, since each interface has two associated graph nodes and we
have about 500 "normal, interior" graph node
This patch adds an error-index to node-index map, so we can store
error heap indices directly in the vlib_buffer_t.
Type: refactor
Change-Id: I28101cad3d8750819e27b8785fc0cf71ff54f79a
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/vlib/node.h')
-rw-r--r-- | src/vlib/node.h | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/src/vlib/node.h b/src/vlib/node.h index bec0ed2d2c0..40d0165f402 100644 --- a/src/vlib/node.h +++ b/src/vlib/node.h @@ -760,8 +760,27 @@ typedef struct /* Node registrations added by constructors */ vlib_node_registration_t *node_registrations; + + /* Node index from error code */ + u32 *node_by_error; } vlib_node_main_t; +typedef u16 vlib_error_t; + +always_inline u32 +vlib_error_get_node (vlib_node_main_t * nm, vlib_error_t e) +{ + return nm->node_by_error[e]; +} + +always_inline u32 +vlib_error_get_code (vlib_node_main_t * nm, vlib_error_t e) +{ + u32 node_index = nm->node_by_error[e]; + vlib_node_t *n = nm->nodes[node_index]; + u32 error_code = e - n->error_heap_index; + return error_code; +} #define FRAME_QUEUE_MAX_NELTS 64 typedef struct |