Re: [RFC] Improved add_timer_randomness for __CRIS__ (instead of rdtsc())

Oliver Xymoron (oxymoron@waste.org)
Tue, 20 Aug 2002 14:34:09 -0500


On Tue, Aug 20, 2002 at 09:20:50PM +0200, johan.adolfsson@axis.com wrote:
> > > > Perhaps something like:
> > > >
> > > > speed=get_timestamp_khz;
> > > > lowbits=get_hires_timestamp();
> > >
> > > But isn't the "num ^= high;" a way to improve the randomness
> > > and the high value doesn't really need to be linear to the time?
> >
> > No, the high order bits aren't very interesting at all. Don't worry
> > about that bit, it's just cuteness.
>
> ok:-)
> But if the the timestamp doesn't have to be entirely accurate
> the name should perhaps reflect that, since at least on the cris arch
> you can save some cycles if you can accept a glitch now and then.

Describe this glitch again? Bear in mind that resolution is a very
different matter than accuracy. Jiffies are not accurate in any
absolute sense, but they are monotonic. And unless you've got
interrupts shut off, even get_cycles can only giving you the time
when it stores the timestamp in the register, which may have little to
do with the time the next instruction that uses it executes.

> We want to be able to separate it from similar functions used
> for high-res timers etc. which need a more accurate function.

Bear in mind that most arches won't have this sort of difficulty so in
the bigger picture, this might be overkill. Put a comment next to the
generic function that says "might not be accurate" and when a user of
the interface comes along that actually cares about accuracy, the
situation can be reevaluated.

-- 
 "Love the dolphins," she advised him. "Write by W.A.S.T.E.." 
-
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/