Re: [PATCH] ath9k: use hw_random API instead of directly dumping into random.c

From: Dominik Brodowski
Date: Wed Feb 16 2022 - 02:32:29 EST


Am Wed, Feb 16, 2022 at 12:52:15AM +0100 schrieb Jason A. Donenfeld:
> On 2/16/22, Toke Høiland-Jørgensen <toke@xxxxxxx> wrote:
> >> @@ -22,9 +22,6 @@
> >> #include "hw.h"
> >> #include "ar9003_phy.h"
> >>
> >> -#define ATH9K_RNG_BUF_SIZE 320
> >> -#define ATH9K_RNG_ENTROPY(x) (((x) * 8 * 10) >> 5) /* quality: 10/32 */
> >
> > So this comment says "quality: 10/32" but below you're setting "quality"
> > as 320. No idea what the units are supposed to be, but is this right?
>
> I think the unit is supposed to be how many entropic bits there are
> out of 1024 bits? These types of estimates are always BS, so keeping
> it on the lower end as before seemed right. Herbert can jump in here
> if he has a better idea; that's his jig.

10/32 = 320/1024, so that change is correct.

Dominik