Re: + clocksource-initialize-list-value.patch added to -mm tree

[Arjan van de Ven]

On Sun, 2006-10-08 at 07:42 -0700, Daniel Walker wrote:
> On Sun, 2006-10-08 at 11:50 +0200, Thomas Gleixner wrote:
> > On Fri, 2006-10-06 at 18:53 -0700, akpm@xxxxxxxx wrote:
> > > A change to clocksource initialization.  It's optional for new clocksources,
> > > but prefered.  If the list field is initialized it allows clocksource_register
> > > to complete faster since it doesn't have the scan the list of clocks doing
> > > strcmp on each.
> > 
> > Either make it required or not.
> 
> This is for compatability with all those clocksources people are writing
> that just haven't been released (assuming they will be eventually
> released). I would love to make this required.

so do it. The ones that are in development are easily fixed and
consistency is worth a LOT in terms of API. Bonus points if the kernel
printk's a helpful message on abuse.



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