lkml.org 
[lkml]   [2004]   [Jun]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [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.
    >>>
    >>
    >>Yes, but... The main problem is that this would break all the archs that
    >>don't have the needed arch dependent code yet. By making the high-res part
    >>depend on the arch config turning on a config, we don't break the archs and
    >>they can sign up as they get their act together.
    >
    >
    > if the price is this amount of uglyness then that's the wrong approach imo.
    > Would be nicer to make dummy helpers the arch people can just take during
    > the transition instead.

    I have another solution, connected with the ability to turn it off at boot time.
    Unsupporting archs would just not define a flag.

    --
    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/

    \
     
     \ /
      Last update: 2005-03-22 14:03    [W:4.176 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site