From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753659AbdEQHI4 (ORCPT ); Wed, 17 May 2017 03:08:56 -0400 Received: from mail-pg0-f66.google.com ([74.125.83.66]:36053 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753211AbdEQHIv (ORCPT ); Wed, 17 May 2017 03:08:51 -0400 From: Namhyung Kim To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Peter Zijlstra , Jiri Olsa , LKML , kernel-team@lge.com, Milian Wolff , Frederic Weisbecker Subject: [PATCH] perf report: Fix invalid warning on callchain param Date: Wed, 17 May 2017 16:08:37 +0900 Message-Id: <20170517070837.6263-1-namhyung@kernel.org> X-Mailer: git-send-email 2.12.2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Currently "perf report -g srcline --stdio" shows two warnings like below: Invalid callchain mode: srcline Invalid callchain order: srcline This is because it always tries to parse callchain mode, order, sort-key and value-type in a row. So even if 'srcline' is a valid sort-key, it shows invalid mode and order warnings. Change it to show a warning only if failed to parse the given token as any of valid parameter. Also code under try_numbers requires the token to be a number, it's good to show the warning there. Cc: Milian Wolff Cc: Frederic Weisbecker Signed-off-by: Namhyung Kim --- tools/perf/util/callchain.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/perf/util/callchain.c b/tools/perf/util/callchain.c index 9ab68682c6d0..89dbfd9e6d24 100644 --- a/tools/perf/util/callchain.c +++ b/tools/perf/util/callchain.c @@ -15,6 +15,7 @@ #include #include #include +#include #include "asm/bug.h" @@ -65,7 +66,6 @@ static int parse_callchain_mode(const char *value) return 0; } - pr_err("Invalid callchain mode: %s\n", value); return -1; } @@ -82,7 +82,6 @@ static int parse_callchain_order(const char *value) return 0; } - pr_err("Invalid callchain order: %s\n", value); return -1; } @@ -105,7 +104,6 @@ static int parse_callchain_sort_key(const char *value) return 0; } - pr_err("Invalid callchain sort key: %s\n", value); return -1; } @@ -124,7 +122,6 @@ static int parse_callchain_value(const char *value) return 0; } - pr_err("Invalid callchain config key: %s\n", value); return -1; } @@ -197,6 +194,9 @@ __parse_callchain_report_opt(const char *arg, bool allow_record_opt) } try_numbers: + if (!isdigit(*tok)) + pr_err("Invalid callchain param: %s\n", tok); + if (try_stack_size) { unsigned long size = 0; -- 2.12.2