On Tue, 27 May 2014, Vince Weaver wrote: > OK, the problem is in: > > kernel/events/core.c find_get_context() > > retry: > perf_lock_task_context() returns NULL. > alloc_perf_context() > task->perf_event_ctxp[ctxn] is *not* NULL > err == EAGAIN so jump to retry > > stuck forever leaking the memory being allocated by alloc_perf_context(). > > I'm not sure what this code should be doing, but it looks like we'll > always leak memory if the EAGAIN case is triggered?
And actually no leak at all, because "put_ctx()" really means "free ctx". Is that some sort of kernel idiom? It would also be nice if it were "perf_put_ctx()" although I guess you don't name functions just so they're easier to enable when using ftrace wildcards. Vince -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
