Re: [PATCH v6 2/7] random, timekeeping: Collect timekeeping entropy in the timekeeping code

From: Andy Lutomirski
Date: Thu Aug 14 2014 - 13:35:24 EST


On Wed, Aug 13, 2014 at 10:43 PM, Andy Lutomirski <luto@xxxxxxxxxxxxxx> wrote:
> Currently, init_std_data calls ktime_get_real(). This imposes
> awkward constraints on when init_std_data can be called, and
> init_std_data is unlikely to collect the full unpredictable data
> available to the timekeeping code, especially after resume.
>
> Remove this code from random.c and add the appropriate
> add_device_randomness calls to timekeeping.c instead.

*sigh* this is buggy:


> + add_device_randomness(tk, sizeof(tk));

sizeof(*tk)

> + add_device_randomness(tk, sizeof(tk));

ditto.

I'll fix this for v7, but I'll wait awhile for other comments to reduce spam.

--Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/