* Peter Zijlstra (a.p.zijlstra@chello.nl) wrote:We should really think about what we are doing before we add a marker in the kernel code. The information extracted should be both useful and expected not to change too much between versions. Ideally, implementation details should not be exported. Exporting useless information "just because we can" would indeed put pressure on maintainers. That's where I expect them to be the best persons to tell what is an implementation detail likely to change, and what is a more "conceptually stable" information. e.g. a context switch is a context switch, this does not change with the underlying implementation. I think that whenever we can add a more "generic" marker which solves many special cases, we should do so. In this case, using the system call instrumentation found in my architecture specific instrumentation patchset would comprehend futex instrumentation. By adding extraction of all system call parameters, things such as futexes should be covered. However, we would still need to instrument read() or exec() to extract the file names. Otherwise, we would have to start doing architecture-specific code which would "know" what arguments are passed to each system call. I guess we could do that if it lessens instrumentation intrusiveness, but we would have to deal with a system call tracing infrastructure tied closely to system call parameters. System call audit code seems to already do that, so I guess we could go that way. Then, I think we should turn to inner-kernel instrumentation only when the information extracted from the stable kernel ABI (e.g. system calls) is not complete enough to understand how things work. That would be the case for block I/O tracing for instance. Mathieu -- Mathieu Desnoyers Computer Engineering Ph.D. Student, Ecole Polytechnique de Montreal OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68 --
| Sunil Naidu | Re: Linux 2.6.20-rc6 |
| david | Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3 |
| Jan Engelhardt | intel iommu (Re: -mm merge plans for 2.6.23) |
| Linus Torvalds | Re: init's children list is long and slows reaping children. |
| David Miller | Re: [GIT]: Networking |
| Jarek Poplawski | [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Gerrit Renker | [PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side) |
| Andrew Morton | Re: [BUG] New Kernel Bugs |
git: | |
