Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752673AbdIFT6D (ORCPT ); Wed, 6 Sep 2017 15:58:03 -0400 Received: from mail.kernel.org ([198.145.29.99]:44156 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752187AbdIFT6C (ORCPT ); Wed, 6 Sep 2017 15:58:02 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1FE5B21E93 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=goodmis.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=rostedt@goodmis.org Date: Wed, 6 Sep 2017 15:57:58 -0400 From: Steven Rostedt To: Tom Zanussi 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 Subject: Re: [PATCH v2 07/40] tracing: Apply absolute timestamps to instance max buffer Message-ID: <20170906155758.18e4a683@gandalf.local.home> In-Reply-To: References: X-Mailer: Claws Mail 3.14.0 (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1862 Lines: 48 On Tue, 5 Sep 2017 16:57:19 -0500 Tom Zanussi wrote: > From: Baohong Liu > > Currently absolute timestamps are applied to both regular and max > buffers only for global trace. For instance trace, absolute > timestamps are applied only to regular buffer. But, regular and max > buffers can be swapped, for example, following a snapshot. So, for > instance trace, bad timestamps can be seen following a snapshot. > Let's apply absolute timestamps to instance max buffer as well. > > Similarly, buffer clock change is applied to instance max buffer > as well. Hmm, this is a bug fix in its own right. I'll pull this in for the current merge window and slap a stable tag on it too. -- Steve > > Signed-off-by: Baohong Liu > --- > kernel/trace/trace.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c > index 66d465e..719e4c1 100644 > --- a/kernel/trace/trace.c > +++ b/kernel/trace/trace.c > @@ -6223,7 +6223,7 @@ static int tracing_set_clock(struct trace_array *tr, const char *clockstr) > tracing_reset_online_cpus(&tr->trace_buffer); > > #ifdef CONFIG_TRACER_MAX_TRACE > - if (tr->flags & TRACE_ARRAY_FL_GLOBAL && tr->max_buffer.buffer) > + if (tr->max_buffer.buffer) > ring_buffer_set_clock(tr->max_buffer.buffer, trace_clocks[i].func); > tracing_reset_online_cpus(&tr->max_buffer); > #endif > @@ -6307,7 +6307,7 @@ int tracing_set_time_stamp_abs(struct trace_array *tr, bool abs) > tracing_reset_online_cpus(&tr->trace_buffer); > > #ifdef CONFIG_TRACER_MAX_TRACE > - if (tr->flags & TRACE_ARRAY_FL_GLOBAL && tr->max_buffer.buffer) > + if (tr->max_buffer.buffer) > ring_buffer_set_time_stamp_abs(tr->max_buffer.buffer, abs); > tracing_reset_online_cpus(&tr->max_buffer); > #endif