Philippe Gerum wrote: > Jan Kiszka wrote: >> Tracing of IRQ on/off paths on x86_64 currently suffers from heavy >> over-instrumentation. > > I understand the point of grouping ipipe_trace_begin/ipipe_trace_end > statements inside the .interrupt macros and using a lightweight thunk > code since we are already covered by the SAVE_ARGS prologue, but I find > the following hunk suspicious, since unlike i386, we do not virtualize > inline sti/cli ops for x86_64. My concern is that removing this > instrumentation would leave us naked in the cold the day some subtle > upstream change introduces a hw masked section we don't immediately > notice; such trace points would precisely help us spotting it.
The situation is not that different compared to i386: - we do not add or remove enabling/disabled points - we rely on the correctness of mainline here, so we are not on our own anyway - if things actually change in mainline, the tracer will probably be the last indicator for it (patching breaks and/or leaking clis will fairly visible impact) On the other side, there was a good reason not to instrument cli/sti in the assembly exit/entry code on i386, just like it is on x86_64: The effort to get worthwhile outputs is noticeable. As you see, the current approach is not very helpful due to loosing the caller context (thanks to the thunks). And those points only mark uninteresting micro paths, thus create a lot of noise in normal traces. So, I see not good reason for fixing this instrumentation and still vote for killing it. Jan
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Adeos-main mailing list [email protected] https://mail.gna.org/listinfo/adeos-main
