Re: PATCH? trace_remove_event_call() should fail if call is active

From: Oleg Nesterov
Date: Wed Jul 03 2013 - 13:59:14 EST


On 07/03, Oleg Nesterov wrote:
>
> IOW. So far _I think_ we just need the additional changes in
> trace_remove_event_call() if it succeeds (with the patch I sent)
> to prevent the races like above, but I didn't even try to think
> about this problem.

And I guess greatly underestimated the problem(s). When I look at
this code now, it seems that, say, event_enable_write() will use
the already freed ftrace_event_file in this case.

Still I think this is another (although closely related) problem.

> Or I missed your point completely?

Yes?

Oleg.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/