perf session: Add missing newlines to some pr_err() calls

acmel · acmel · commit e87b49116ded · 2015-11-11T18:41:31.000-03:00
Before: [acme@zoo linux]$ perf evlist WARNING: The perf.data file's data size field is 0 which is unexpected. Was the 'perf record' command properly terminated? non matching sample_type[acme@zoo linux]$ After: [acme@zoo linux]$ perf evlist WARNING: The perf.data file's data size field is 0 which is unexpected. Was the 'perf record' command properly terminated? non matching sample_type [acme@zoo linux]$ Cc: Adrian Hunter <adrian.hunter@intel.com> Cc: David Ahern <dsahern@gmail.com> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Wang Nan <wangnan0@huawei.com> Link: http://lkml.kernel.org/n/tip-wscok3a2s7yrj8156oc2r6qe@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
@@ -29,25 +29,25 @@ static int perf_session__open(struct perf_session *session)
 	struct perf_data_file *file = session->file;
 
 	if (perf_session__read_header(session) < 0) {
-		pr_err("incompatible file format (rerun with -v to learn more)");
+		pr_err("incompatible file format (rerun with -v to learn more)\n");
 		return -1;
 	}
 
 	if (perf_data_file__is_pipe(file))
 		return 0;
 
 	if (!perf_evlist__valid_sample_type(session->evlist)) {
-		pr_err("non matching sample_type");
+		pr_err("non matching sample_type\n");
 		return -1;
 	}
 
 	if (!perf_evlist__valid_sample_id_all(session->evlist)) {
-		pr_err("non matching sample_id_all");
+		pr_err("non matching sample_id_all\n");
 		return -1;
 	}
 
 	if (!perf_evlist__valid_read_format(session->evlist)) {
-		pr_err("non matching read_format");
+		pr_err("non matching read_format\n");
 		return -1;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -29,25 +29,25 @@ static int perf_session__open(struct perf_session *session)`
`29`	`29`	`struct perf_data_file *file = session->file;`
`30`	`30`
`31`	`31`	`if (perf_session__read_header(session) < 0) {`
`32`		`- pr_err("incompatible file format (rerun with -v to learn more)");`
	`32`	`+ pr_err("incompatible file format (rerun with -v to learn more)\n");`
`33`	`33`	`return -1;`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`if (perf_data_file__is_pipe(file))`
`37`	`37`	`return 0;`
`38`	`38`
`39`	`39`	`if (!perf_evlist__valid_sample_type(session->evlist)) {`
`40`		`- pr_err("non matching sample_type");`
	`40`	`+ pr_err("non matching sample_type\n");`
`41`	`41`	`return -1;`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`if (!perf_evlist__valid_sample_id_all(session->evlist)) {`
`45`		`- pr_err("non matching sample_id_all");`
	`45`	`+ pr_err("non matching sample_id_all\n");`
`46`	`46`	`return -1;`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`if (!perf_evlist__valid_read_format(session->evlist)) {`
`50`		`- pr_err("non matching read_format");`
	`50`	`+ pr_err("non matching read_format\n");`
`51`	`51`	`return -1;`
`52`	`52`	`}`
`53`	`53`