On Wed, 2007-03-07 at 14:17 -0800, Zachary Amsden wrote: > Thomas Gleixner wrote: > > Simply because you _ABUSE_ timer_init_hook() to set it up. Keep it self > > contained and do not impose restrictions on the kernel core code, which > > we have to maintain. > > > > But time_init_hook is supposed to be abused. That is its purpose - to > be a hook for different time devices on SGI Visual Workstation and > Voyager. And we don't actually abuse it anymore, we just bypass it > because the default timer init path wants to setup the PIT or the HPET, > neither of which should be used in paravirt.
It is there for those hardware platforms, but using it inside your clock event device is _JUST_ wrong. Please make these things self contained and not relying on whatever time_init_hook() contains. tglx - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/