commit f622bf4279466eae6874dead7a56b59669783490
author Damjan Marion <damarion@cisco.com> Tue Dec 14 10:40:48 2021 +0100
committer Ole Tr�an <otroan@employees.org> Tue Dec 14 11:04:42 2021 +0000
tree eb33391f31c934b109b68b18ebaea840fac199c9
parent 162ff5e4122572045ea7976c77ba3286fb05eeb5

vppinfra: display cpu frequency in test_vector_funcs perf test

Type: improvement
Change-Id: I350a30eca0d3d7ec8816e4e7c30889621029c694
Signed-off-by: Damjan Marion <damarion@cisco.com>

src/vppinfra/vector/test/test.c

diff --git a/src/vppinfra/vector/test/test.c b/src/vppinfra/vector/test/test.c
index 7cbd33f..51b6bbf 100644
--- a/src/vppinfra/vector/test/test.c
+++ b/src/vppinfra/vector/test/test.c
@@ -67,13 +67,22 @@
 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 @@
     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 @@
     .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 @@
   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 @@
 	}
       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++)
     {