| From: Jiri Olsa <jolsa@kernel.org> |
| Date: Tue, 6 Feb 2018 19:17:58 +0100 |
| Subject: perf record: Put new line after target override warning |
| |
| commit c3dec27b7f70a9ad5f777d943d51ecdfcd9824d0 upstream. |
| |
| There's no new-line after target-override warning, now: |
| |
| $ perf record -a --per-thread |
| Warning: |
| SYSTEM/CPU switch overriding PER-THREAD^C[ perf record: Woken up 1 times to write data ] |
| [ perf record: Captured and wrote 0.705 MB perf.data (2939 samples) ] |
| |
| with patch: |
| |
| $ perf record -a --per-thread |
| Warning: |
| SYSTEM/CPU switch overriding PER-THREAD |
| ^C[ perf record: Woken up 1 times to write data ] |
| [ perf record: Captured and wrote 0.705 MB perf.data (2939 samples) ] |
| |
| Signed-off-by: Jiri Olsa <jolsa@kernel.org> |
| Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> |
| Cc: David Ahern <dsahern@gmail.com> |
| Cc: Namhyung Kim <namhyung@kernel.org> |
| Cc: Peter Zijlstra <peterz@infradead.org> |
| Fixes: 16ad2ffb822c ("perf tools: Introduce perf_target__strerror()") |
| Link: http://lkml.kernel.org/r/20180206181813.10943-3-jolsa@kernel.org |
| Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Signed-off-by: Ben Hutchings <ben@decadent.org.uk> |
| --- |
| tools/perf/builtin-record.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/tools/perf/builtin-record.c |
| +++ b/tools/perf/builtin-record.c |
| @@ -942,7 +942,7 @@ int cmd_record(int argc, const char **ar |
| err = target__validate(&rec->opts.target); |
| if (err) { |
| target__strerror(&rec->opts.target, err, errbuf, BUFSIZ); |
| - ui__warning("%s", errbuf); |
| + ui__warning("%s\n", errbuf); |
| } |
| |
| err = target__parse_uid(&rec->opts.target); |