Messages in this thread | | | Date | Wed, 5 Jan 2011 14:56:12 -0500 | From | Mathieu Desnoyers <> | Subject | Re: [RFC patch 3/5] ftrace trace event add missing semicolumn |
| |
* Frederic Weisbecker (fweisbec@gmail.com) wrote: [...] > Looks good! > > I might be missing corner things but it seems this would reduce the code > footprint (one function less) and turn more rw into ro datas. > > So it seems to be a very valuable reason to change the semicolon requirement > all over the place. > > If you come up with this feature along the massive semicolon requirement > change, we will probably happily apply the whole. > > But coming with only the semicolon change is more like an empty shell.
My proposal here is to incrementally improve the tracing code, starting by cleaning up what is already there. I cannot do this if you keep asking me for larger changes to both Ftrace and Perf before any of the prerequisite cleanups can make their way in.
In this thread, I demonstrated that the TRACE_EVENT cleanup I proposed opens a lot of code/data size reduction cleanups for Ftrace and Perf. But let's get the cleanup in there first (it does not break the current way Ftrace and Perf are working), and once all the code-base has moved to the semicolumn-less semantic, then we can start improving Ftrace and Perf.
As we say in French, "il ne faut pas mettre la charrue devant les boeufs" (roughly: don't put the cart before the horse)
Thanks,
Mathieu
-- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com
| |