This patch prevents duplicated error output. per_session__new function calls perf_data_file__open which will print error message if it can not open the file and we will get duplicated output of the error message:
failed to open perf.data.old: No such file or directory Failed to open perf.data.old Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- tools/perf/builtin-diff.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index 74aada5..a18bf98 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c @@ -742,7 +742,6 @@ static int __cmd_diff(void) data__for_each_file(i, d) { d->session = perf_session__new(&d->file, false, &tool); if (!d->session) { - pr_err("Failed to open %s\n", d->file.path); ret = -1; goto out_delete; } -- 2.3.1.291.g7a9409c -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/