From 703908aaf4f80ef6bc78166d46eba3a3b3aa15f7 Mon Sep 17 00:00:00 2001 From: Ole Troan Date: Tue, 26 Feb 2019 16:37:03 +0100 Subject: stats: Add interface name to sw_if_index as /if/names Expose interface name to sw_if_index table as a directory node in the stats segment. Change-Id: Ie50e4af01997d141880f02e3a8496bfeb91b9944 Signed-off-by: Ole Troan --- src/vpp/app/vpp_get_stats.c | 9 +++++++ src/vpp/stats/stat_segment.c | 60 +++++++++++++++++++++++++++++++++++++++++++- src/vpp/stats/stat_segment.h | 22 +++++++++------- 3 files changed, 81 insertions(+), 10 deletions(-) (limited to 'src/vpp') diff --git a/src/vpp/app/vpp_get_stats.c b/src/vpp/app/vpp_get_stats.c index 1c1298e2afd..1d878029dda 100644 --- a/src/vpp/app/vpp_get_stats.c +++ b/src/vpp/app/vpp_get_stats.c @@ -220,6 +220,15 @@ reconnect: fformat (stdout, "%.2f %s\n", res[i].scalar_value, res[i].name); break; + case STAT_DIR_TYPE_NAME_VECTOR: + if (res[i].name_vector == 0) + continue; + for (k = 0; k < vec_len (res[i].name_vector); k++) + if (res[i].name_vector[k]) + fformat (stdout, "[%d]: %s %s\n", k, res[i].name_vector[k], + res[i].name); + break; + default: ; } diff --git a/src/vpp/stats/stat_segment.c b/src/vpp/stats/stat_segment.c index ca78a103522..aa32aede3a5 100644 --- a/src/vpp/stats/stat_segment.c +++ b/src/vpp/stats/stat_segment.c @@ -268,7 +268,7 @@ vlib_map_stat_segment_init (void) /* Scalar stats and node counters */ vec_validate (sm->directory_vector, STAT_COUNTERS - 1); #define _(E,t,n,p) \ - strcpy(sm->directory_vector[STAT_COUNTER_##E].name, "/sys" #p "/" #n); \ + strcpy(sm->directory_vector[STAT_COUNTER_##E].name, #p "/" #n); \ sm->directory_vector[STAT_COUNTER_##E].type = STAT_DIR_TYPE_##t; foreach_stat_segment_counter_name #undef _ @@ -686,8 +686,66 @@ statseg_config (vlib_main_t * vm, unformat_input_t * input) return 0; } +static clib_error_t * +statseg_sw_interface_add_del (vnet_main_t * vnm, u32 sw_if_index, u32 is_add) +{ + stat_segment_main_t *sm = &stat_segment_main; + stat_segment_shared_header_t *shared_header = sm->shared_header; + + void *oldheap = vlib_stats_push_heap (); + vlib_stat_segment_lock (); + + vec_validate (sm->interfaces, sw_if_index); + if (is_add) + { + vnet_sw_interface_t *si = vnet_get_sw_interface (vnm, sw_if_index); + vnet_sw_interface_t *si_sup = + vnet_get_sup_sw_interface (vnm, si->sw_if_index); + vnet_hw_interface_t *hi_sup; + + ASSERT (si_sup->type == VNET_SW_INTERFACE_TYPE_HARDWARE); + hi_sup = vnet_get_hw_interface (vnm, si_sup->hw_if_index); + + u8 *s = 0; + s = format (s, "%v", hi_sup->name); + if (si->type != VNET_SW_INTERFACE_TYPE_HARDWARE) + s = format (s, ".%d", si->sub.id); + s = format (s, "%c", 0); + sm->interfaces[sw_if_index] = s; + } + else + { + vec_free (sm->interfaces[sw_if_index]); + sm->interfaces[sw_if_index] = 0; + } + + stat_segment_directory_entry_t *ep; + ep = &sm->directory_vector[STAT_COUNTER_INTERFACE_NAMES]; + ep->offset = stat_segment_offset (shared_header, sm->interfaces); + + int i; + u64 *offset_vector = + ep->offset_vector ? stat_segment_pointer (shared_header, + ep->offset_vector) : 0; + + vec_validate (offset_vector, vec_len (sm->interfaces) - 1); + for (i = 0; i < vec_len (sm->interfaces); i++) + { + offset_vector[i] = + sm->interfaces[i] ? stat_segment_offset (shared_header, + sm->interfaces[i]) : 0; + } + ep->offset_vector = stat_segment_offset (shared_header, offset_vector); + + vlib_stat_segment_unlock (); + clib_mem_set_heap (oldheap); + + return 0; +} + VLIB_INIT_FUNCTION (statseg_init); VLIB_EARLY_CONFIG_FUNCTION (statseg_config, "statseg"); +VNET_SW_INTERFACE_ADD_DEL_FUNCTION (statseg_sw_interface_add_del); /* *INDENT-OFF* */ VLIB_REGISTER_NODE (stat_segment_collector, static) = diff --git a/src/vpp/stats/stat_segment.h b/src/vpp/stats/stat_segment.h index fb450f38f53..917b512aecb 100644 --- a/src/vpp/stats/stat_segment.h +++ b/src/vpp/stats/stat_segment.h @@ -32,19 +32,21 @@ typedef enum STAT_COUNTER_NODE_VECTORS, STAT_COUNTER_NODE_CALLS, STAT_COUNTER_NODE_SUSPENDS, + STAT_COUNTER_INTERFACE_NAMES, STAT_COUNTERS } stat_segment_counter_t; #define foreach_stat_segment_counter_name \ - _(VECTOR_RATE, SCALAR_INDEX, vector_rate,) \ - _(INPUT_RATE, SCALAR_INDEX, input_rate,) \ - _(LAST_UPDATE, SCALAR_INDEX, last_update,) \ - _(LAST_STATS_CLEAR, SCALAR_INDEX, last_stats_clear,) \ - _(HEARTBEAT, SCALAR_INDEX, heartbeat,) \ - _(NODE_CLOCKS, COUNTER_VECTOR_SIMPLE, clocks, /node) \ - _(NODE_VECTORS, COUNTER_VECTOR_SIMPLE, vectors, /node) \ - _(NODE_CALLS, COUNTER_VECTOR_SIMPLE, calls, /node) \ - _(NODE_SUSPENDS, COUNTER_VECTOR_SIMPLE, suspends, /node) + _(VECTOR_RATE, SCALAR_INDEX, vector_rate, /sys) \ + _(INPUT_RATE, SCALAR_INDEX, input_rate, /sys) \ + _(LAST_UPDATE, SCALAR_INDEX, last_update, /sys) \ + _(LAST_STATS_CLEAR, SCALAR_INDEX, last_stats_clear, /sys) \ + _(HEARTBEAT, SCALAR_INDEX, heartbeat, /sys) \ + _(NODE_CLOCKS, COUNTER_VECTOR_SIMPLE, clocks, /sys/node) \ + _(NODE_VECTORS, COUNTER_VECTOR_SIMPLE, vectors, /sys/node) \ + _(NODE_CALLS, COUNTER_VECTOR_SIMPLE, calls, /sys/node) \ + _(NODE_SUSPENDS, COUNTER_VECTOR_SIMPLE, suspends, /sys/node) \ + _(INTERFACE_NAMES, NAME_VECTOR, names, /if) typedef struct { @@ -101,6 +103,8 @@ typedef struct /* statistics segment */ uword *directory_vector_by_name; stat_segment_directory_entry_t *directory_vector; + u8 **interfaces; + clib_spinlock_t *stat_segment_lockp; clib_socket_t *socket; u8 *socket_name; -- cgit 1.2.3-korg