Messages in this thread | | | Date | Thu, 13 Jan 2005 23:34:37 +0100 | From | Andries Brouwer <> | Subject | Re: [PATCH 4/5] random periodicity detection fix |
| |
On Wed, Jan 12, 2005 at 10:46:29PM -0800, Matt Mackall wrote:
> The input layer is now sending us a bunch of events in a row for each > actual event. This shows up weaknesses in the periodicity detector and > using the high clock rate from get_clock: each keystroke is getting > accounted as 10 different tmaximal-entropy events. > > A brief touch on a trackpad will generate as much as 2000 maximal > entropy events which is more than 2k of /dev/random output. IOW, we're > WAY overestimating input entropy.
Yes, indeed. I muttered about this long ago - let me see, yes, http://marc.theaimsgroup.com/?l=linux-kernel&m=106271659930542&w=3
My patch did the opposite of your patch: I removed the add entropy call in input.c.
I don't recall the details, but I think my reason was that some events are synthetic, and I wanted all entropy calls to be derived from actual outside sources.
Also, when there are several sources, all constant or almost constant, then merging the streams might cause one to see variation where there isn't really any. Also that is a reason not to do the adding so far away from the source.
Andries - 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/
| |