Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754366AbdIEWDS (ORCPT ); Tue, 5 Sep 2017 18:03:18 -0400 Received: from mga14.intel.com ([192.55.52.115]:45771 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754079AbdIEV71 (ORCPT ); Tue, 5 Sep 2017 17:59:27 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,481,1498546800"; d="scan'208";a="1011287689" From: Tom Zanussi To: rostedt@goodmis.org Cc: tglx@linutronix.de, mhiramat@kernel.org, namhyung@kernel.org, vedang.patel@intel.com, bigeasy@linutronix.de, joel.opensrc@gmail.com, joelaf@google.com, mathieu.desnoyers@efficios.com, baohong.liu@intel.com, linux-kernel@vger.kernel.org, linux-rt-users@vger.kernel.org, Tom Zanussi Subject: [PATCH v2 31/40] tracing: Allow whitespace to surround hist trigger filter Date: Tue, 5 Sep 2017 16:57:43 -0500 Message-Id: <9e6d9f17bcd71f29dec9039ab00d316ca2965c43.1504642143.git.tom.zanussi@linux.intel.com> X-Mailer: git-send-email 1.9.3 In-Reply-To: References: In-Reply-To: References: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1636 Lines: 55 The existing code only allows for one space before and after the 'if' specifying the filter for a hist trigger. Add code to make that more permissive as far as whitespace goes. Signed-off-by: Tom Zanussi --- kernel/trace/trace_events_hist.c | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_hist.c index 0a398f3..4f66f2e 100644 --- a/kernel/trace/trace_events_hist.c +++ b/kernel/trace/trace_events_hist.c @@ -4819,7 +4819,7 @@ static int event_hist_trigger_func(struct event_command *cmd_ops, struct synth_event *se; const char *se_name; bool remove = false; - char *trigger; + char *trigger, *p; int ret = 0; if (!param) @@ -4829,9 +4829,19 @@ static int event_hist_trigger_func(struct event_command *cmd_ops, remove = true; /* separate the trigger from the filter (k:v [if filter]) */ - trigger = strsep(¶m, " \t"); - if (!trigger) - return -EINVAL; + trigger = param; + p = strstr(param, " if"); + if (!p) + p = strstr(param, "\tif"); + if (p) { + if (p == trigger) + return -EINVAL; + param = p + 1; + param = strstrip(param); + *p = '\0'; + trigger = strstrip(trigger); + } else + param = NULL; attrs = parse_hist_trigger_attrs(trigger); if (IS_ERR(attrs)) @@ -4889,6 +4899,7 @@ static int event_hist_trigger_func(struct event_command *cmd_ops, } ret = cmd_ops->reg(glob, trigger_ops, trigger_data, file); + /* * The above returns on success the # of triggers registered, * but if it didn't register any it returns zero. Consider no -- 1.9.3