Commit 0cef66a9 authored Oct 22, 2022 by Yang Jihong Committed by Arnaldo Carvalho de Melo Oct 27, 2022

perf config: Add missing newline on pr_warning() call in home_perfconfig()



Add missing newline on pr_warning() call in home_perfconfig().

Before:
  # perf record
  File /home/yangjihong/.perfconfig not owned by current user or root, ignoring it.Couldn't synthesize bpf events.

After:
  # perf record
  File /home/yangjihong/.perfconfig not owned by current user or root, ignoring it.
  Couldn't synthesize bpf events.

Signed-off-by: Yang Jihong <yangjihong1@huawei.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: https://lore.kernel.org/r/20221022092735.114967-4-yangjihong1@huawei.com


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent a87edbec

tools/perf/util/config.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -556,7 +556,7 @@ static char *home_perfconfig(void)

		config = strdup(mkpath("%s/.perfconfig", home));
		if (config == NULL) {
		pr_warning("Not enough memory to process %s/.perfconfig, ignoring it.", home);
		pr_warning("Not enough memory to process %s/.perfconfig, ignoring it.\n", home);
		return NULL;
		}

		@@ -564,7 +564,7 @@ static char *home_perfconfig(void)
		goto out_free;

		if (st.st_uid && (st.st_uid != geteuid())) {
		pr_warning("File %s not owned by current user or root, ignoring it.", config);
		pr_warning("File %s not owned by current user or root, ignoring it.\n", config);
		goto out_free;
		}