Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c549aca5 authored by David Ahern's avatar David Ahern Committed by Arnaldo Carvalho de Melo
Browse files

perf tests: Make terms a stack variable in test_term 



No need to malloc the memory for it.

Signed-off-by: default avatarDavid Ahern <dsahern@gmail.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1372793245-4136-6-git-send-email-dsahern@gmail.com


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 0142dab0

tools/perf/tests/parse-events.c

+5 −9
Original line number Original line Diff line number Diff line
@@ -1246,24 +1246,20 @@ static int test_events(struct evlist_test *events, unsigned cnt) 




static int test_term(struct terms_test *t)

static int test_term(struct terms_test *t)

{

{

	struct list_head *terms;

	struct list_head terms;

	int ret;

	int ret;





	terms = malloc(sizeof(*terms));

	INIT_LIST_HEAD(&terms);

	if (!terms)

		return -ENOMEM;



	INIT_LIST_HEAD(terms);





	ret = parse_events_terms(terms, t->str);

	ret = parse_events_terms(&terms, t->str);

	if (ret) {

	if (ret) {

		pr_debug("failed to parse terms '%s', err %d\n",

		pr_debug("failed to parse terms '%s', err %d\n",

			 t->str , ret);

			 t->str , ret);

		return ret;

		return ret;

	}

	}





	ret = t->check(terms);

	ret = t->check(&terms);

	parse_events__free_terms(terms);

	parse_events__free_terms(&terms);





	return ret;

	return ret;

}

}