Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCH 09/14] perf ftrace: Add 'record' sub-command | Date | Wed, 08 May 2013 14:27:40 +0900 |
| |
Hi Jiri,
On Mon, 6 May 2013 14:04:11 +0200, Jiri Olsa wrote: > On Mon, May 06, 2013 at 10:44:56AM +0900, Namhyung Kim wrote: >> Hi Jiri, >> >> Sorry for late reply. I've been busy these days. >> >> On Thu, 25 Apr 2013 15:24:18 +0900, Namhyung Kim wrote: >> > But it looks there's a race between cond_wait() and cond_broatcast(). >> > I'll take a look at that. >> >> I think the problem is that a recorder can be blocked if it received a >> broadcast signal before calling cond_wait(). So I modified the code to >> place signaling ready_cond and waiting start_cond in the same cs. >> >> Does the patch below fix your problem? > > yes.. without the patch I get lockup in like 10 minutes > with the patch the test is now ok for almost an hour..
Many thanks for your test!
Namhyung
| |