Re: Fwd: [draft] Tracing multibuffer support concurrency issues

2013-07-01 Thread Alexander Lam
On Mon, Jul 1, 2013 at 6:35 PM, Steven Rostedt wrote: > On Mon, 2013-07-01 at 15:33 -0700, Alexander Lam wrote: > >> To fix this we could go through the ftrace_trace_arrays list and use >> addresses to check if a particular pointer to a trace_array is still >> valid, but t

Re: [PATCH 3/3] tracing: Protect ftrace_trace_arrays list in trace_events.c

2013-07-01 Thread Alexander Lam
Oh, sorry, that is an incomplete patch; some bits are in a patch I dropped. I'll send you a new one in about 20 minutes. - Alex On Mon, Jul 1, 2013 at 6:25 PM, Steven Rostedt wrote: > On Mon, 2013-07-01 at 15:31 -0700, Alexander Z Lam wrote: >> There are multiple places where the ftrace_trace_ar

Fwd: [draft] Tracing multibuffer support concurrency issues

2013-07-01 Thread Alexander Lam
Hi all, I noticed that a695cb58 "tracing: Prevent deleting instances when they are being read" [1] still leaves open the possibility of the trace_array being deleted before the reference counter is incremented. Thread A creates a new instance "foo", then tries to open "foo/trace" for writing, whi

Re: thermal governor: does it actually work??

2013-02-18 Thread Alexander Lam
do end up applying Peter's fix. > > > > Thanks, > > rui > > > > From: Alexander Lam [mailto:a...@andrew.cmu.edu] > Sent: Sunday, February 17, 2013 4:56 AM > To: Andreas Mohr > Cc: Zhang, Rui; linux-kernel@vger.kernel.org; R, Durgadoss; pe...@piie.net;

Re: thermal governor: does it actually work??

2013-02-16 Thread Alexander Lam
uot; in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- Alexander Lam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body

[PATCH 1/1] thermal: Check cooling device's current state before declaring it inactive

2012-12-30 Thread Alexander Lam
again, so checking the current state of the cooling device is required. Signed-off-by: Alexander Lam --- drivers/thermal/step_wise.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/step_wise.c b/drivers/thermal/step_wise.c index 0cd5e9f..0eeb709