OK, I will rename  debug_stack_use_ctr to debug_idt_ctr.
Thanks.

Seiji

> -----Original Message-----
> From: Steven Rostedt [mailto:rost...@goodmis.org]
> Sent: Tuesday, June 04, 2013 4:58 PM
> To: Seiji Aguchi
> Cc: H. Peter Anvin; linux-kernel@vger.kernel.org; x...@kernel.org; 
> t...@linutronix.de; mi...@elte.hu; b...@alien8.de; linux-
> e...@vger.kernel.org; tony.l...@intel.com; dle-deve...@lists.sourceforge.net; 
> Tomoki Sekiyama
> Subject: Re: [PATCH v13 3/3] trace,x86: Add irq vector tracepoints
> 
> On Tue, 2013-06-04 at 20:20 +0000, Seiji Aguchi wrote:
> > Steven,
> >
> > >
> > > Hmm, maybe we can do it. Have two counters, a debug_idt_ctr and a
> > > trace_idt_ctr, then have a function that basically does this:
> > >
> > >   if (this_cpu_read(debug_idt_ctr))
> >
> > I think we can use "debug_stack_use_ctr"  for the checking.
> > Is it correct?
> > Or, do I need to introduce a new debug_idt_ctr?
> >
> 
> No, it's the same variable. I was thinking we should rename it too, as
> debug_stack_use_ctr, doesn't really describe what is happening anymore.
> 
> -- Steve
> 

Reply via email to