(Linus and Andrew Cc:-ed as well) * Frederic Weisbecker <[email protected]> wrote:
> When a task is traced and is in a stopped state, the tracer > may execute a ptrace request to examine the tracee state and > get its task struct. Right after, the tracee can be killed > and thus its breakpoints released. > This can happen concurrently when the tracer is in the middle > of reading or modifying these breakpoints, leading to dereferencing > a freed pointer. > > Hence, to prepare the fix, create a generic breakpoint reference > holding API. When a reference on the breakpoints of a task is > held, the breakpoints won't be released until the last reference > is dropped. After that, no more ptrace request on the task's > breakpoints can be serviced for the tracer. > > Reported-by: Oleg Nesterov <[email protected]> > Signed-off-by: Frederic Weisbecker <[email protected]> Ok, this series looks a bit scary - and this ptrace.h change does not have Oleg's Acked-by. (the arch bits all have maintaner Acked-by's) The changes look a bit ugly as well: beyond the ugly ifdeffery, we have ptrace.h::ptrace_init_task(), which is only used in tracehook.h::tracehook_finish_clone() which is only used in kernel/fork.c::copy_process(). That's two levels of obfuscation to do something rather simple - i think we should get rid of the tracehook.h redirections, it did not work out in the end as a method of capturing events - ftrace TRACE_EVENT() seems better structured and more maintainable. But i guess we could live with this fix for v2.6.39, if neither Oleg nor Linus and Andrew are hating this further complication of the ptrace mess enough to NAK it. Thoughts? Plus, i'd really love it if you did some stress-testing of this change of a mixed ptrace breakpoints and perf breakpoints workload, on some sufficiently SMP box. gdb's hbreak is a very low freq way of testing thus such regressions take ages to be reported. Thanks, Ingo _______________________________________________ stable mailing list [email protected] http://linux.kernel.org/mailman/listinfo/stable
