Commit d82257d7 authored by Ian Rogers's avatar Ian Rogers Committed by Namhyung Kim
Browse files

perf symbol: Remove now unused symbol_conf.sort_by_name 



Previously used to specify symbol_name_rb_node was in use.

Signed-off-by: default avatarIan Rogers <irogers@google.com>
Acked-by: default avatarNamhyung Kim <namhyung@kernel.org>
Cc: Carsten Haitzler <carsten.haitzler@arm.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Jason Wang <wangborong@cdjrlc.com>
Cc: Changbin Du <changbin.du@huawei.com>
Cc: Yang Jihong <yangjihong1@huawei.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Athira Rajeev <atrajeev@linux.vnet.ibm.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/20230623054520.4118442-4-irogers@google.com


Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
parent 259dce91

tools/perf/builtin-kallsyms.c

+0 −1
Original line number Diff line number Diff line
@@ -62,7 +62,6 @@ int cmd_kallsyms(int argc, const char **argv) 
	if (argc < 1)

		usage_with_options(kallsyms_usage, options);



	symbol_conf.sort_by_name = true;

	symbol_conf.try_vmlinux_path = (symbol_conf.vmlinux_name == NULL);

	if (symbol__init(NULL) < 0)

		return -1;

tools/perf/builtin-lock.c

+0 −2
Original line number Diff line number Diff line
@@ -1774,7 +1774,6 @@ static int __cmd_report(bool display_info) 
	}



	/* for lock function check */

	symbol_conf.sort_by_name = true;

	symbol_conf.allow_aliases = true;

	symbol__init(&session->header.env);
@@ -1904,7 +1903,6 @@ static int __cmd_contention(int argc, const char **argv) 
		con.save_callstack = true;



	/* for lock function check */

	symbol_conf.sort_by_name = true;

	symbol_conf.allow_aliases = true;

	symbol__init(&session->header.env);

tools/perf/builtin-report.c

+0 −1
Original line number Diff line number Diff line
@@ -1676,7 +1676,6 @@ int cmd_report(int argc, const char **argv) 
			 * See symbol__browser_index.

			 */

			symbol_conf.priv_size += sizeof(u32);

			symbol_conf.sort_by_name = true;

		}

		annotation_config__init(&report.annotation_opts);

	}

tools/perf/tests/builtin-test.c

+0 −1
Original line number Diff line number Diff line
@@ -542,7 +542,6 @@ int cmd_test(int argc, const char **argv) 
		return run_workload(workload, argc, argv);



	symbol_conf.priv_size = sizeof(int);

	symbol_conf.sort_by_name = true;

	symbol_conf.try_vmlinux_path = true;



	if (symbol__init(NULL) < 0)

tools/perf/util/probe-event.c

+0 −1
Original line number Diff line number Diff line
@@ -74,7 +74,6 @@ int init_probe_symbol_maps(bool user_only) 
{

	int ret;



	symbol_conf.sort_by_name = true;

	symbol_conf.allow_aliases = true;

	ret = symbol__init(NULL);

	if (ret < 0) {