Re: [PATCH] simplify do_signal_stop() utrace_report_jctl() interaction

2009-03-19 Thread Roland McGrath
Roland, I think it better to change tracehook definition more, please see below. I don't really object to this in principle. But this touches signal.c a lot more in less obviously-trivial ways than my tracehook patch. That is more of an issue at the outset than some extra fiddling in the

Re: PATCH? tracehook_notify_jctl SIGCONT

2009-03-19 Thread Roland McGrath
Now we send SIGCHLD with si_code = CLD_STOPPED twice. By T1 from do_signal_stop(), and by T1 or T2 from get_signal_to_deliver() which checks SIGNAL_CLD_MASK. Yes, I considered this problem. It's just not so big a deal as to worry overmuch about this corner case in the first version. What

Re: [RFC][PATCH 1/2] tracing/ftrace: syscall tracing infrastructure

2009-03-19 Thread Roland McGrath
The utrace API itself is not a good fit for global tracing, since its purpose is tracing and control of individual user threads. There is no reason to allocate its per-task data structures when you are going to treat all tasks the same anyway. The points that I think are being missed are about