diff options
author | Ray Kinsella <mdr@ashroe.eu> | 2021-05-21 16:17:39 +0100 |
---|---|---|
committer | Damjan Marion <dmarion@me.com> | 2021-05-26 06:45:40 +0000 |
commit | dbf90d499bb9e36e3bbb57a0b964003e3eb183c5 (patch) | |
tree | 0d80d4cf62a09a41c9adfd97ae77f13b6994d502 | |
parent | a2d6d352c6926d2f8d4e50aeb1ec59802f32b37b (diff) |
perfmon: revert raw column support
Revert raw column from the perfmon plugin.
Type: refactor
Signed-off-by: Ray Kinsella <mdr@ashroe.eu>
Change-Id: If127f57ee2022cc1c0ea5177f1655a792f195f1d
-rw-r--r-- | src/plugins/perfmon/cli.c | 19 | ||||
-rw-r--r-- | src/plugins/perfmon/intel/bundle/branch_mispred.c | 10 | ||||
-rw-r--r-- | src/plugins/perfmon/perfmon.h | 1 |
3 files changed, 0 insertions, 30 deletions
diff --git a/src/plugins/perfmon/cli.c b/src/plugins/perfmon/cli.c index f6678772d84..3305480031d 100644 --- a/src/plugins/perfmon/cli.c +++ b/src/plugins/perfmon/cli.c @@ -275,19 +275,10 @@ show_perfmon_stats_command_fn (vlib_main_t *vm, unformat_input_t *input, perfmon_instance_t *in; u8 *s = 0; int n_row = 0; - u8 raw = 0; if (b == 0) return clib_error_return (0, "no bundle selected"); - while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT) - { - if (unformat (input, "raw")) - raw = 1; - else - break; - } - n_instances = vec_len (it->instances); vec_validate (readings, n_instances - 1); @@ -314,13 +305,6 @@ show_perfmon_stats_command_fn (vlib_main_t *vm, unformat_input_t *input, char **hdr = b->column_headers; while (hdr[0]) table_format_cell (t, -1, n_row++, "%s", hdr++[0]); - - if (b->raw_column_headers && raw) - { - hdr = b->raw_column_headers; - while (hdr[0]) - table_format_cell (t, -1, n_row++, "%s", hdr++[0]); - } } int col = 0; @@ -358,9 +342,6 @@ show_perfmon_stats_command_fn (vlib_main_t *vm, unformat_input_t *input, vlib_cli_output (vm, "%U\n", format_table, t); table_free (t); - if (raw) - vlib_cli_output (vm, "Sample time is %.4f seconds \n", pm->sample_time); - if (b->footer) vlib_cli_output (vm, "\n%s\n", b->footer); diff --git a/src/plugins/perfmon/intel/bundle/branch_mispred.c b/src/plugins/perfmon/intel/bundle/branch_mispred.c index 343adc7bc44..bb5bf48b2fa 100644 --- a/src/plugins/perfmon/intel/bundle/branch_mispred.c +++ b/src/plugins/perfmon/intel/bundle/branch_mispred.c @@ -40,15 +40,6 @@ format_branch_mispredictions (u8 *s, va_list *args) case 4: s = format (s, "%05.2f", (ns->value[2] / (f64) ns->value[0]) * 100); break; - case 5: - s = format (s, "%9f", (f64) ns->value[0]); - break; - case 6: - s = format (s, "%9f", (f64) ns->value[1]); - break; - case 7: - s = format (s, "%9f", (f64) ns->value[2]); - break; } return s; } @@ -65,5 +56,4 @@ PERFMON_REGISTER_BUNDLE (branch_mispredictions) = { .format_fn = format_branch_mispredictions, .column_headers = PERFMON_STRINGS ("Branches/call", "Branches/pkt", "Taken/call", "Taken/pkt", "% MisPred"), - .raw_column_headers = PERFMON_STRINGS ("branches", "taken", "misses"), }; diff --git a/src/plugins/perfmon/perfmon.h b/src/plugins/perfmon/perfmon.h index 3d0bc0608f9..bba22cf6b1d 100644 --- a/src/plugins/perfmon/perfmon.h +++ b/src/plugins/perfmon/perfmon.h @@ -110,7 +110,6 @@ typedef struct perfmon_bundle perfmon_bundle_init_fn_t *init_fn; char **column_headers; - char **raw_column_headers; format_function_t *format_fn; clib_cpu_supports_func_t cpu_supports; |