![]() | |||||||||||||
Messages in this thread |
* Andi Kleen <andi@firstfloor.org> wrote: > > if (!f) > > f = cpufreq_get(freq->cpu); > > if (!f) > > f = tsc_khz; > > > > ? > > > > Something's not quite right here :-) > > What do you think is wrong? cpufreq_get can return 0. > > Admittedly the second test could be inside a block of the first, but > then it would make the code more ugly and this code is not performance > critical. moving it inside the block makes the code _cleaner_ in fact :-) Ingo - 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-05-25 07:45 [from the cache] ©2003-2008 | |||||||||||||