Commit 3dfc01fe authored by Namhyung Kim's avatar Namhyung Kim Committed by Arnaldo Carvalho de Melo

perf test: Add 'datasym' test workload

The datasym workload is to check if perf mem command gets the data
addresses precisely.  This is needed for data symbol test.

  $ perf test -w datasym

I had to keep the buf1 in the data section, otherwise it could end
up in the BSS and was mmaped as a separate //anon region, then it
was not symbolized at all.  It needs to be fixed separately.

Committer notes:

Add a -U _FORTIFY_SOURCE to the datasym CFLAGS, as the main perf flags
set it and it requires building with optimization, and this new test has
a -O0.
Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Athira Jajeev <atrajeev@linux.vnet.ibm.com>
Cc: German Gomez <german.gomez@arm.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: James Clark <james.clark@arm.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Leo Yan <leo.yan@linaro.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Zhengjun Xing <zhengjun.xing@linux.intel.com>
Link: https://lore.kernel.org/r/20221116233854.1596378-12-namhyung@kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 7bc1dd96
...@@ -124,6 +124,7 @@ static struct test_workload *workloads[] = { ...@@ -124,6 +124,7 @@ static struct test_workload *workloads[] = {
&workload__leafloop, &workload__leafloop,
&workload__sqrtloop, &workload__sqrtloop,
&workload__brstack, &workload__brstack,
&workload__datasym,
}; };
static int num_subtests(const struct test_suite *t) static int num_subtests(const struct test_suite *t)
......
...@@ -205,5 +205,6 @@ DECLARE_WORKLOAD(thloop); ...@@ -205,5 +205,6 @@ DECLARE_WORKLOAD(thloop);
DECLARE_WORKLOAD(leafloop); DECLARE_WORKLOAD(leafloop);
DECLARE_WORKLOAD(sqrtloop); DECLARE_WORKLOAD(sqrtloop);
DECLARE_WORKLOAD(brstack); DECLARE_WORKLOAD(brstack);
DECLARE_WORKLOAD(datasym);
#endif /* TESTS_H */ #endif /* TESTS_H */
...@@ -5,7 +5,9 @@ perf-y += thloop.o ...@@ -5,7 +5,9 @@ perf-y += thloop.o
perf-y += leafloop.o perf-y += leafloop.o
perf-y += sqrtloop.o perf-y += sqrtloop.o
perf-y += brstack.o perf-y += brstack.o
perf-y += datasym.o
CFLAGS_sqrtloop.o = -g -O0 -fno-inline -U_FORTIFY_SOURCE CFLAGS_sqrtloop.o = -g -O0 -fno-inline -U_FORTIFY_SOURCE
CFLAGS_leafloop.o = -g -O0 -fno-inline -fno-omit-frame-pointer -U_FORTIFY_SOURCE CFLAGS_leafloop.o = -g -O0 -fno-inline -fno-omit-frame-pointer -U_FORTIFY_SOURCE
CFLAGS_brstack.o = -g -O0 -fno-inline -U_FORTIFY_SOURCE CFLAGS_brstack.o = -g -O0 -fno-inline -U_FORTIFY_SOURCE
CFLAGS_datasym.o = -g -O0 -fno-inline -U_FORTIFY_SOURCE
#include <linux/compiler.h>
#include "../tests.h"
typedef struct _buf {
char data1;
char reserved[55];
char data2;
} buf __attribute__((aligned(64)));
static buf buf1 = {
/* to have this in the data section */
.reserved[0] = 1,
};
static int datasym(int argc __maybe_unused, const char **argv __maybe_unused)
{
for (;;) {
buf1.data1++;
buf1.data2 += buf1.data1;
}
return 0;
}
DEFINE_WORKLOAD(datasym);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment