GNU bug report logs - #6658
[PATCH] randread: don't require -lrt

Previous Next

Full log

Message #8 received at 6658 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> CS.UCLA.EDU>
Cc: 6658 <at> debbugs.gnu.org
Subject: Re: bug#6658: [PATCH] randread: don't require -lrt
Date: Sat, 17 Jul 2010 00:34:49 +0200

Paul Eggert wrote:
> In looking at the random part of coreutils some more, I see some
> issues.
>
> 1.  Apps that use random numbers typically must link to -lrt,
>     for a very small benefit (nanosecond resolution rather than
>     microsecond resolution time stamp for random seed).  Often
>     this "benefit" is illusory as the time stamps really are not
>     nanosecond resolution.
>
> 2.  If /dev/urandom is available, it should be used to seed
>     the ISAAC generator.  This will cost more than invoking
>     gettimeofday() but it's far more random.
>
> 3.  We could get about 2X CPU performance on 64-bit machines by using
>     ISAAC64 instead of ISAAC.
>
> The patch below implements (1); I haven't installed it.  I'd like to
> do (2) and (3) too, but thought I'd ask for feedback first.

These are all welcome changes, and the patch for (1) looks fine.
Thanks!

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.