RE: [ANNOUNCE] high-res-timers patches for 2.6.6

[Dave Hylands]

Please keep in mind that some people just want the VST portion, and
don't want the high-res-timers portion (i.e. my ARM platform doesn't
have or need the hardware side of things). This may not be relevant to
your discussion of the ifdef's, but on the off chance that it was
somehow, I thought I would throw that out.

--
Dave Hylands
Vancouver, BC, Canada
http://www.DaveHylands.com/ 

> -----Original Message-----
> From: high-res-timers-discourse-admin@xxxxxxxxxxxxxxxxxxxxx 
> [mailto:high-res-timers-discourse-admin@xxxxxxxxxxxxxxxxxxxxx]
>  On Behalf Of Arjan van de Ven
> Sent: Thursday, June 10, 2004 3:04 AM
> To: Geoff Levand
> Cc: high-res-timers-discourse@xxxxxxxxxxxxxxxxxxxxx; 
> linux-kernel@xxxxxxxxxxxxxxx; George Anzinger
> Subject: Re: [ANNOUNCE] high-res-timers patches for 2.6.6
> 
> 
> 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.
> 
> 

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/