Changeset 3bd74758 in mainline for uspace/app/perf/perf.h

Timestamp:

2018-12-28T09:32:11Z (5 years ago)

Author:

Vojtech Horky <vojtech.horky@…>

Branches:

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

Children:

c7de81b

Parents:

8ee106b

Message:

perf: huge refactoring

The overall aim of this refactoring was to remove duplicate code in all
benchmarks that was responsible (a) for computing proper workload size
and (b) for computing final statistics.

After the refactoring, the actual benchmark code is quite short and
takes care of the actual work only.

The harness code was factored out into perf.c that is now responsible
for computing the workload size and then runs the actual benchmark.

As an extra feature, we pass stopwatch_t into the benchmark code that is
only responsible for starting/stopping. Duration is then queried outside
the benchmark code.

File:

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

uspace/app/perf/perf.h

@@ -37 +37 @@
 
 #include <stdbool.h>
+#include <perf.h>
 
-typedef const char *(*benchmark_entry_t)(void);
+typedef bool (*benchmark_entry_t)(stopwatch_t *, uint64_t,
+    char *, size_t);
+typedef bool (*benchmark_helper_t)(char *, size_t);
 
 typedef struct {
@@ -44 +47 @@
         const char *desc;
         benchmark_entry_t entry;
+        benchmark_helper_t setup;
+        benchmark_helper_t teardown;
 } benchmark_t;
 
-extern const char *bench_malloc1(void);
-extern const char *bench_malloc2(void);
-extern const char *bench_ns_ping(void);
-extern const char *bench_ping_pong(void);
+extern bool bench_malloc1(stopwatch_t *, uint64_t, char *, size_t);
+extern bool bench_malloc2(stopwatch_t *, uint64_t, char *, size_t);
+extern bool bench_ns_ping(stopwatch_t *, uint64_t, char *, size_t);
+extern bool bench_ping_pong(stopwatch_t *, uint64_t, char *, size_t);
+extern bool bench_ping_pong_setup(char *, size_t);
+extern bool bench_ping_pong_teardown(char *, size_t);
 
 extern benchmark_t benchmarks[];