From mboxrd@z Thu Jan 1 00:00:00 1970 From: Namhyung Kim Subject: Re: [RFC][PATCH 12/21] tracing: Account for variables in named trigger compatibility Date: Mon, 13 Feb 2017 15:04:39 +0900 Message-ID: <20170213060439.GG14705@sejong> References: <88d17e7197f809ade0dd150712ae1d6b86b26b6b.1486569306.git.tom.zanussi@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: rostedt@goodmis.org, tglx@linutronix.de, mhiramat@kernel.org, linux-kernel@vger.kernel.org, linux-rt-users@vger.kernel.org To: Tom Zanussi Return-path: Content-Disposition: inline In-Reply-To: <88d17e7197f809ade0dd150712ae1d6b86b26b6b.1486569306.git.tom.zanussi@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-rt-users.vger.kernel.org On Wed, Feb 08, 2017 at 11:25:08AM -0600, Tom Zanussi wrote: > Named triggers must also have the same set of variables in order to be > considered compatible - update the trigger match test to account for > that. > > The reason for this requirement is that named triggers with variables > are meant to allow one or more events to set the same variable. > > Signed-off-by: Tom Zanussi > --- > kernel/trace/trace_events_hist.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_hist.c > index e707577..889455e 100644 > --- a/kernel/trace/trace_events_hist.c > +++ b/kernel/trace/trace_events_hist.c > @@ -1576,6 +1576,10 @@ static bool hist_trigger_match(struct event_trigger_data *data, > return false; > if (key_field->is_signed != key_field_test->is_signed) > return false; > + if ((key_field->var_name && !key_field_test->var_name) || > + (!key_field->var_name && key_field_test->var_name) || > + strcmp(key_field->var_name, key_field_test->var_name) != 0) > + return false; What if key_field->var_name and key_field_test->var_name are both NULL? Thanks, Namhyung > } > > for (i = 0; i < hist_data->n_sort_keys; i++) { > -- > 1.9.3 >