Changeset ebb0835 in mainline for uspace/app/hbench/hbench.h

Timestamp:

2019-01-07T12:56:22Z (5 years ago)

Author:

Vojtech Horky <vojtech.horky@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c2db02a

Parents:

a787081

Message:

hbench: add tiny wrapper around stopwatch_t

This prepares the harness for future extensions when more than wallclock
time would be prepared. The data would be stored inside the new
structure and the actual benchmarks would not need to be modified at all
(they really should not care which metrics are collected).

File:

• uspace/app/hbench/hbench.h (modified) (2 diffs)

uspace/app/hbench/hbench.h

@@ -41 +41 @@
 #include <perf.h>
 
-typedef bool (*benchmark_entry_t)(stopwatch_t *, uint64_t,
+/*
+ * So far, a simple wrapper around system stopwatch.
+ * Eventually, we could collection of hardware counters etc. without
+ * modifying signatures of any existing benchmark.
+ */
+typedef struct {
+        stopwatch_t stopwatch;
+} benchmeter_t;
+
+static inline void benchmeter_init(benchmeter_t *meter)
+{
+        stopwatch_init(&meter->stopwatch);
+}
+
+static inline void benchmeter_start(benchmeter_t *meter)
+{
+        stopwatch_start(&meter->stopwatch);
+}
+
+static inline void benchmeter_stop(benchmeter_t *meter)
+{
+        stopwatch_stop(&meter->stopwatch);
+}
+
+typedef bool (*benchmark_entry_t)(benchmeter_t *, uint64_t,
     char *, size_t);
 typedef bool (*benchmark_helper_t)(char *, size_t);
@@ -57 +81 @@
 
 extern errno_t csv_report_open(const char *);
-extern void csv_report_add_entry(stopwatch_t *, int, benchmark_t *, uint64_t);
+extern void csv_report_add_entry(benchmeter_t *, int, benchmark_t *, uint64_t);
 extern void csv_report_close(void);