mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
synced 2025-01-17 22:05:08 +00:00
72b4ca7e99
The current use of atomics can lead to test failures, as tests (such as
tests/shell/record.sh) search for samples with "test_loop" as the
top-most stack frame, but find frames related to the atomic operation
(e.g. __aarch64_ldadd4_relax).
This change simply removes the "count" variable, as it is not necessary.
Fixes: 1962ab6f6e0b39e4 ("perf test workload thloop: Make count increments atomic")
Reviewed-by: James Clark <james.clark@arm.com>
Signed-off-by: Nick Forrington <nick.forrington@arm.com>
Acked-by: Leo Yan <leo.yan@linaro.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: https://lore.kernel.org/r/20231102162225.50028-1-nick.forrington@arm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
52 lines
813 B
C
52 lines
813 B
C
/* SPDX-License-Identifier: GPL-2.0 */
|
|
#include <pthread.h>
|
|
#include <stdlib.h>
|
|
#include <signal.h>
|
|
#include <unistd.h>
|
|
#include <linux/compiler.h>
|
|
#include "../tests.h"
|
|
|
|
static volatile sig_atomic_t done;
|
|
|
|
/* We want to check this symbol in perf report */
|
|
noinline void test_loop(void);
|
|
|
|
static void sighandler(int sig __maybe_unused)
|
|
{
|
|
done = 1;
|
|
}
|
|
|
|
noinline void test_loop(void)
|
|
{
|
|
while (!done);
|
|
}
|
|
|
|
static void *thfunc(void *arg)
|
|
{
|
|
void (*loop_fn)(void) = arg;
|
|
|
|
loop_fn();
|
|
return NULL;
|
|
}
|
|
|
|
static int thloop(int argc, const char **argv)
|
|
{
|
|
int sec = 1;
|
|
pthread_t th;
|
|
|
|
if (argc > 0)
|
|
sec = atoi(argv[0]);
|
|
|
|
signal(SIGINT, sighandler);
|
|
signal(SIGALRM, sighandler);
|
|
alarm(sec);
|
|
|
|
pthread_create(&th, NULL, thfunc, test_loop);
|
|
test_loop();
|
|
pthread_join(th, NULL);
|
|
|
|
return 0;
|
|
}
|
|
|
|
DEFINE_WORKLOAD(thloop);
|