Skip to content

Commit 7c144bf

Browse files
author
Ingo Molnar
committed
Merge tag 'perf-urgent-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/urgent
Pull two 'perf bench' fixes from Arnaldo: * Make 'perf bench mem' (i.e. no args) mean 'run all tests' so that we can run all tests, not stopping at the numa ones. (Arnaldo Carvalho de Melo) * Fix NULL pointer dereference after last test in in "perf bench all" (Patrick Palka) Signed-off-by: Arnaldo Carvalho de Melo <[email protected]> Signed-off-by: Ingo Molnar <[email protected]>
2 parents dcb99fd + 6eeefcc commit 7c144bf

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

tools/perf/bench/numa.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1593,6 +1593,7 @@ static void init_params(struct params *p, const char *name, int argc, const char
15931593
p->data_rand_walk = true;
15941594
p->nr_loops = -1;
15951595
p->init_random = true;
1596+
p->run_all = argc == 1;
15961597
}
15971598

15981599
static int run_bench_numa(const char *name, const char **argv)

tools/perf/builtin-bench.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ static struct collection collections[] = {
7676

7777
/* Iterate over all benchmarks within a collection: */
7878
#define for_each_bench(coll, bench) \
79-
for (bench = coll->benchmarks; bench->name; bench++)
79+
for (bench = coll->benchmarks; bench && bench->name; bench++)
8080

8181
static void dump_benchmarks(struct collection *coll)
8282
{

0 commit comments

Comments
 (0)