Messages in this thread | | | Date | Mon, 17 Jan 2022 18:31:16 +0100 | From | Dominik Brodowski <> | Subject | Re: [PATCH 2/4] random: cleanup fractional entropy shift constants |
| |
Am Sun, Jan 16, 2022 at 05:35:45PM +0100 schrieb Jason A. Donenfeld: > The entropy estimator is calculated in terms of 1/8 bits, which means > there are various constants where things are shifted by 3. Move these > into our pool info enum with the other relevant constants. While we're > at it, move an English assertion about sizes into a proper BUILD_BUG_ON > so that the compiler can ensure this invariant. > > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> > --- > drivers/char/random.c | 28 +++++++++++++--------------- > 1 file changed, 13 insertions(+), 15 deletions(-) > > diff --git a/drivers/char/random.c b/drivers/char/random.c > index de1c14787ae8..7343bff086c5 100644 > --- a/drivers/char/random.c > +++ b/drivers/char/random.c > @@ -358,16 +358,6 @@ > > /* #define ADD_INTERRUPT_BENCH */ > > -/* > - * To allow fractional bits to be tracked, the entropy_count field is > - * denominated in units of 1/8th bits. > - * > - * 2*(POOL_ENTROPY_SHIFT + poolbitshift) must <= 31, or the multiply in > - * credit_entropy_bits() needs to be 64 bits wide. > - */ > -#define POOL_ENTROPY_SHIFT 3 > -#define POOL_ENTROPY_BITS() (input_pool.entropy_count >> POOL_ENTROPY_SHIFT) > - > /* > * If the entropy count falls under this number of bits, then we > * should wake up processes which are selecting or polling on write > @@ -425,8 +415,13 @@ enum poolinfo { > POOL_WORDMASK = POOL_WORDS - 1, > POOL_BYTES = POOL_WORDS * sizeof(u32), > POOL_BITS = POOL_BYTES * 8, > - POOL_BITSHIFT = ilog2(POOL_WORDS) + 5, > - POOL_FRACBITS = POOL_WORDS << (POOL_ENTROPY_SHIFT + 5), > + POOL_BITSHIFT = ilog2(POOL_BITS), > + > + /* To allow fractional bits to be tracked, the entropy_count field is > + * denominated in units of 1/8th bits. */ > + POOL_ENTROPY_SHIFT = 3, > +#define POOL_ENTROPY_BITS() (input_pool.entropy_count >> POOL_ENTROPY_SHIFT) > + POOL_FRACBITS = POOL_BITS << POOL_ENTROPY_SHIFT,
the #define here confuses me a bit, as it is optically breaking the POOL enum. But that relates to coding style preferences only, so
Reviewed-by: Dominik Brodowski <linux@dominikbrodowski.net>
| |