aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDamjan Marion <damarion@cisco.com>2021-12-14 10:40:48 +0100
committerOle Tr�an <otroan@employees.org>2021-12-14 11:04:42 +0000
commitf622bf4279466eae6874dead7a56b59669783490 (patch)
treeeb33391f31c934b109b68b18ebaea840fac199c9
parent162ff5e4122572045ea7976c77ba3286fb05eeb5 (diff)
vppinfra: display cpu frequency in test_vector_funcs perf test
Type: improvement Change-Id: I350a30eca0d3d7ec8816e4e7c30889621029c694 Signed-off-by: Damjan Marion <damarion@cisco.com>
-rw-r--r--src/vppinfra/vector/test/test.c31
-rw-r--r--src/vppinfra/vector/test/test.h1
2 files changed, 24 insertions, 8 deletions
diff --git a/src/vppinfra/vector/test/test.c b/src/vppinfra/vector/test/test.c
index 7cbd33f5a73..51b6bbf4bb2 100644
--- a/src/vppinfra/vector/test/test.c
+++ b/src/vppinfra/vector/test/test.c
@@ -67,13 +67,22 @@ typedef struct
static u8 *
format_test_perf_bundle_default (u8 *s, va_list *args)
{
+ test_main_t *tm = &test_main;
test_perf_event_bundle_t __clib_unused *b =
va_arg (*args, test_perf_event_bundle_t *);
test_perf_t *tp = va_arg (*args, test_perf_t *);
u64 *data = va_arg (*args, u64 *);
+ if (tm->ref_clock > 0)
+ {
+ if (data)
+ s = format (s, "%8.1f", tm->ref_clock * data[0] / data[1] / 1e9);
+ else
+ s = format (s, "%8s", "Freq");
+ }
+
if (data)
- s = format (s, "%5.2f", (f64) data[1] / data[0]);
+ s = format (s, "%5.2f", (f64) data[2] / data[0]);
else
s = format (s, "%5s", "IPC");
@@ -83,17 +92,17 @@ format_test_perf_bundle_default (u8 *s, va_list *args)
s = format (s, "%8s", "Clks/Op");
if (data)
- s = format (s, "%8.2f", (f64) data[1] / tp->n_ops);
+ s = format (s, "%8.2f", (f64) data[2] / tp->n_ops);
else
s = format (s, "%8s", "Inst/Op");
if (data)
- s = format (s, "%9.2f", (f64) data[2] / tp->n_ops);
+ s = format (s, "%9.2f", (f64) data[3] / tp->n_ops);
else
s = format (s, "%9s", "Brnch/Op");
if (data)
- s = format (s, "%10.2f", (f64) data[3] / tp->n_ops);
+ s = format (s, "%10.2f", (f64) data[4] / tp->n_ops);
else
s = format (s, "%10s", "BrMiss/Op");
return s;
@@ -131,10 +140,11 @@ test_perf_event_bundle_t perf_bundles[] = {
.desc = "IPC, Clocks/Operatiom, Instr/Operation, Branch Total & Miss",
.type = PERF_TYPE_HARDWARE,
.config[0] = PERF_COUNT_HW_CPU_CYCLES,
- .config[1] = PERF_COUNT_HW_INSTRUCTIONS,
- .config[2] = PERF_COUNT_HW_BRANCH_INSTRUCTIONS,
- .config[3] = PERF_COUNT_HW_BRANCH_MISSES,
- .n_events = 4,
+ .config[1] = PERF_COUNT_HW_REF_CPU_CYCLES,
+ .config[2] = PERF_COUNT_HW_INSTRUCTIONS,
+ .config[3] = PERF_COUNT_HW_BRANCH_INSTRUCTIONS,
+ .config[4] = PERF_COUNT_HW_BRANCH_MISSES,
+ .n_events = 5,
.format_fn = format_test_perf_bundle_default,
}
#ifdef __x86_64__
@@ -178,6 +188,8 @@ test_perf (test_main_t *tm)
for (int i = 0; i < TEST_PERF_MAX_EVENTS; i++)
fds[i] = -1;
+ tm->ref_clock = os_cpu_clock_frequency ();
+
if (tm->bundle)
{
for (int i = 0; i < ARRAY_LEN (perf_bundles); i++)
@@ -213,6 +225,9 @@ test_perf (test_main_t *tm)
}
fds[i] = fd;
}
+ fformat (stdout, "Warming up...\n");
+ for (u64 i = 0; i < (u64) tm->ref_clock; i++)
+ asm inline("" : : "r"(i * i) : "memory");
for (int i = 0; i < CLIB_MARCH_TYPE_N_VARIANTS; i++)
{
diff --git a/src/vppinfra/vector/test/test.h b/src/vppinfra/vector/test/test.h
index b1478fc6fe6..4334dfa53a1 100644
--- a/src/vppinfra/vector/test/test.h
+++ b/src/vppinfra/vector/test/test.h
@@ -55,6 +55,7 @@ typedef struct
u32 repeat;
u8 *filter;
u8 *bundle;
+ f64 ref_clock;
} test_main_t;
extern test_main_t test_main;