[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.1.10.0809061356270.3117@nehalem.linux-foundation.org>
Date: Sat, 6 Sep 2008 13:59:20 -0700 (PDT)
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Thomas Gleixner <tglx@...utronix.de>
cc: Alok Kataria <akataria@...are.com>,
Alan Cox <alan@...rguk.ukuu.org.uk>,
LKML <linux-kernel@...r.kernel.org>,
Arjan van de Veen <arjan@...radead.org>,
"H. Peter Anvin" <hpa@...or.com>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Dan Hecht <dhecht@...are.com>,
Garrett Smith <garrett@...are.com>
Subject: Re: [RFC patch 0/4] TSC calibration improvements
On Sat, 6 Sep 2008, Thomas Gleixner wrote:
>
> If Alok has the second check in place and is actually worried about
> that 288us impact, then we can add the following (untested), which
> does not impact the speed of the check.
Guys, please.
Show some _taste_.
Dammit, stop adding random crap to "native_calibrate_tsc()" and make it
look like total and utter SHIT.
If you want to do that
tsc1 = tsc_read_refs(&ref1, hpet);
..
tsc2 = tsc_read_refs(&ref1, hpet);
around calibration and comparing it, then do it *once*. Do it over the
whole thing. Do it in a function of its own, instead of making this
horrible and unreadable mess.
This patch may be fine as a "let's check if it works" thing, but please
don't send out total SH*T to public lists.
Some _tasted_ in programming, please!
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists