Date: Tue, 8 Mar 2016 14:17:13 -0500
From: Steven Rostedt <rostedt@goodmis.org>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: Yang Shi <yang.shi@windriver.com>, tglx@linutronix.de,
        linux-kernel@vger.kernel.org, linux-rt-users@vger.kernel.org
Subject: Re: [v4.4-rt PATCH] rt: trace: add rcuidle version macro for
 preemptirqsoff_hist and hrtimer_interrupt
Message-ID: <20160308141713.30814feb@gandalf.local.home>
In-Reply-To: <56DE88FC.5080500@linutronix.de>
References: <1457394338-11865-1-git-send-email-yang.shi@windriver.com>
	<56DE88FC.5080500@linutronix.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1086
Lines: 37

On Tue, 8 Mar 2016 09:10:36 +0100
Sebastian Andrzej Siewior <bigeasy@linutronix.de> wrote:

> On 03/08/2016 12:45 AM, Yang Shi wrote:
> > diff --git a/include/trace/events/hist.h b/include/trace/events/hist.h
> > index 6122e42..37f6eb8 100644
> > --- a/include/trace/events/hist.h
> > +++ b/include/trace/events/hist.h
> > @@ -9,6 +9,7 @@
> >  
> >  #if !defined(CONFIG_PREEMPT_OFF_HIST) && !defined(CONFIG_INTERRUPT_OFF_HIST)
> >  #define trace_preemptirqsoff_hist(a, b)
> > +#define trace_preemptirqsoff_hist_rcuidle(a, b)
> >  #else
> >  TRACE_EVENT(preemptirqsoff_hist,  
> 
> I have this piece already in my tree.

I don't see it in your git repo.


> 
> >  
> > @@ -33,6 +34,7 @@ TRACE_EVENT(preemptirqsoff_hist,
> >  
> >  #ifndef CONFIG_MISSED_TIMER_OFFSETS_HIST
> >  #define trace_hrtimer_interrupt(a, b, c, d)
> > +#define trace_hrtimer_interrupt_rcuidle(a, b, c, d)
> >  #else
> >  TRACE_EVENT(hrtimer_interrupt,  
> 
> I was thinking about this but then I saw no users of it. Do we have
> any? (haven't seen them in v4.1 either).

I don't see this defined either.

-- Steve