![]() | ||||||||||
Messages in this thread |
On Sat, 2007-04-07 at 14:30 -0700, Daniel Walker wrote: > On Sat, 2007-04-07 at 22:50 +0200, Thomas Gleixner wrote: > > On Sat, 2007-04-07 at 10:43 -0700, Daniel Walker wrote: > > > Looks like this path , > > > > > > arch/i386/kernel/tsc.c: time_cpufreq_notifier(); <-- takes xtime_lock > > > mark_tsc_unstable(); > > > clocksource_change_rating(&clocksource_tsc, 0); > > > timekeeping_change_clocksource(); <-- takes xtime_lock > > > > > > > > > I'm not sure why the time_cpufreq_notifier is taking the xtime_lock tho . > > > > Simply because it fiddles with variables which are relevant for > > timekeeping. > > loops_per_jiffy perhaps? Oh well, this is a leftover from the days where we tried to use TSC despite of frequency changes. It still modifies the scale factor of the tsc clocksource. I agree that it can be removed as we switch off TSC anyway in that case. tglx - 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: 2007-04-08 08:35 [from the cache] ©2003-2008 | ||||||||||