Re: [PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-11 Thread Peter Zijlstra
On Mon, Mar 10, 2014 at 11:47:06PM -0400, Steven Rostedt wrote: > On Mon, 10 Mar 2014 16:17:18 +0900 > Fernando Luis Vázquez Cao wrote: > > > > Thank you four your review, Peter, Steven. > > > > By the way, who is going to pick this patch? Do you want > > me to resend with Steven's Acked-by add

Re: [PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-10 Thread Steven Rostedt
On Mon, 10 Mar 2014 16:17:18 +0900 Fernando Luis Vázquez Cao wrote: > Thank you four your review, Peter, Steven. > > By the way, who is going to pick this patch? Do you want > me to resend with Steven's Acked-by added? I don't think you have to. I believe Peter got it. I know there's a bunch o

Re: [PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-10 Thread Fernando Luis Vázquez Cao
On 03/06/2014 07:51 PM, Steven Rostedt wrote: On Thu, 06 Mar 2014 14:25:28 +0900 Fernando Luis Vázquez Cao wrote: From: Fernando Luis Vazquez Cao Prevent tracing of preempt_disable/enable() in sched_clock_cpu(). When CONFIG_DEBUG_PREEMPT is enabled, preempt_disable/enable() are traced and th

Re: [PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-06 Thread Peter Zijlstra
On Thu, Mar 06, 2014 at 02:25:28PM +0900, Fernando Luis Vázquez Cao wrote: > From: Fernando Luis Vazquez Cao > > Prevent tracing of preempt_disable/enable() in sched_clock_cpu(). > When CONFIG_DEBUG_PREEMPT is enabled, preempt_disable/enable() are > traced and this causes trace_clock() users (and

Re: [PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-06 Thread Steven Rostedt
On Thu, 06 Mar 2014 14:25:28 +0900 Fernando Luis Vázquez Cao wrote: > From: Fernando Luis Vazquez Cao > > Prevent tracing of preempt_disable/enable() in sched_clock_cpu(). > When CONFIG_DEBUG_PREEMPT is enabled, preempt_disable/enable() are > traced and this causes trace_clock() users (and prob

[PATCH] sched/clock: prevent tracing recursion in sched_clock_cpu()

2014-03-05 Thread Fernando Luis Vázquez Cao
From: Fernando Luis Vazquez Cao Prevent tracing of preempt_disable/enable() in sched_clock_cpu(). When CONFIG_DEBUG_PREEMPT is enabled, preempt_disable/enable() are traced and this causes trace_clock() users (and probably others) to go into an infinite recursion. Systems with a stable sched_clock