Messages in this thread | | | Date | Fri, 11 Jun 2004 15:33:38 -0700 | From | George Anzinger <> | Subject | Re: [ANNOUNCE] high-res-timers patches for 2.6.6 |
| |
Arjan van de Ven wrote: > On Thu, Jun 10, 2004 at 05:02:23PM -0700, George Anzinger wrote: > >>Arjan van de Ven wrote: >> >>>On Thu, 2004-06-10 at 03:49, Geoff Levand wrote: >>> >>> >>>>Available at >>>>http://tree.celinuxforum.org/pubwiki/moin.cgi/CELinux_5fPatchArchive >>>> >>>>For those interested, the set of three patches provide POSIX high-res >>>>timer support for linux-2.6.6. The core and i386 patches are updates of >>>>George Anzinger's hrtimers-2.6.5-1.0.patch available on SourceForge >>>><http://sourceforge.net/projects/high-res-timers/>. The ppc32 port is >>>>not available on SourceForge yet. >>> >>> >>>My first impression is that it has WAAAAAAAAAAAY too many ifdefs. I >>>would strongly suggest to not make this a config option and just >>>mandatory, it's a core feature that has no point in being optional. If >>>you accept that, the code also becomes a *LOT* cleaner. >>> Can I be so bold as to ask about the changed to the timer list code? Assuming we scrapped all the ifdefs, that is.
I have been thinking of a major rewrite which would leave this code alone, but would introduce an additional list and, of course, overhead for high-res timers. This will take some time and be sub optimal, so I wonder if it is needed.
-- George Anzinger george@mvista.com High-res-timers: http://sourceforge.net/projects/high-res-timers/ Preemption patch: http://www.kernel.org/pub/linux/kernel/people/rml
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |