Re: [PATCH RESEND] random: remove dead code left over from blocking pool
Eric Biggers wrote: > From: Eric Biggers > > Remove some dead code that was left over following commit 90ea1c6436d2 > ("random: remove the blocking pool"). > > Cc: linux-cry...@vger.kernel.org > Cc: Andy Lutomirski > Cc: Jann Horn > Cc: Theodore Ts'o > Reviewed-by: Andy Lutomirski > Acked-by: Ard Biesheuvel > Signed-off-by: Eric Biggers > --- > drivers/char/random.c | 17 ++- > include/trace/events/random.h | 83 --- > 2 files changed, 3 insertions(+), 97 deletions(-) Patch applied. Thanks. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
[PATCH RESEND] random: remove dead code left over from blocking pool
From: Eric Biggers Remove some dead code that was left over following commit 90ea1c6436d2 ("random: remove the blocking pool"). Cc: linux-cry...@vger.kernel.org Cc: Andy Lutomirski Cc: Jann Horn Cc: Theodore Ts'o Reviewed-by: Andy Lutomirski Acked-by: Ard Biesheuvel Signed-off-by: Eric Biggers --- drivers/char/random.c | 17 ++- include/trace/events/random.h | 83 --- 2 files changed, 3 insertions(+), 97 deletions(-) diff --git a/drivers/char/random.c b/drivers/char/random.c index 5d6acfecd919b..605969ed0f965 100644 --- a/drivers/char/random.c +++ b/drivers/char/random.c @@ -500,7 +500,6 @@ struct entropy_store { unsigned short add_ptr; unsigned short input_rotate; int entropy_count; - unsigned int initialized:1; unsigned int last_data_init:1; __u8 last_data[EXTRACT_SIZE]; }; @@ -660,7 +659,7 @@ static void process_random_ready_list(void) */ static void credit_entropy_bits(struct entropy_store *r, int nbits) { - int entropy_count, orig, has_initialized = 0; + int entropy_count, orig; const int pool_size = r->poolinfo->poolfracbits; int nfrac = nbits << ENTROPY_SHIFT; @@ -717,23 +716,14 @@ static void credit_entropy_bits(struct entropy_store *r, int nbits) if (cmpxchg(&r->entropy_count, orig, entropy_count) != orig) goto retry; - if (has_initialized) { - r->initialized = 1; - kill_fasync(&fasync, SIGIO, POLL_IN); - } - trace_credit_entropy_bits(r->name, nbits, entropy_count >> ENTROPY_SHIFT, _RET_IP_); if (r == &input_pool) { int entropy_bits = entropy_count >> ENTROPY_SHIFT; - if (crng_init < 2) { - if (entropy_bits < 128) - return; + if (crng_init < 2 && entropy_bits >= 128) crng_reseed(&primary_crng, r); - entropy_bits = ENTROPY_BITS(r); - } } } @@ -1372,8 +1362,7 @@ static size_t account(struct entropy_store *r, size_t nbytes, int min, } /* - * This function does the actual extraction for extract_entropy and - * extract_entropy_user. + * This function does the actual extraction for extract_entropy. * * Note: we assume that .poolwords is a multiple of 16 words. */ diff --git a/include/trace/events/random.h b/include/trace/events/random.h index 9570a10cb949b..3d7b432ca5f31 100644 --- a/include/trace/events/random.h +++ b/include/trace/events/random.h @@ -85,28 +85,6 @@ TRACE_EVENT(credit_entropy_bits, __entry->entropy_count, (void *)__entry->IP) ); -TRACE_EVENT(push_to_pool, - TP_PROTO(const char *pool_name, int pool_bits, int input_bits), - - TP_ARGS(pool_name, pool_bits, input_bits), - - TP_STRUCT__entry( - __field( const char *, pool_name ) - __field( int, pool_bits ) - __field( int, input_bits ) - ), - - TP_fast_assign( - __entry->pool_name = pool_name; - __entry->pool_bits = pool_bits; - __entry->input_bits = input_bits; - ), - - TP_printk("%s: pool_bits %d input_pool_bits %d", - __entry->pool_name, __entry->pool_bits, - __entry->input_bits) -); - TRACE_EVENT(debit_entropy, TP_PROTO(const char *pool_name, int debit_bits), @@ -161,35 +139,6 @@ TRACE_EVENT(add_disk_randomness, MINOR(__entry->dev), __entry->input_bits) ); -TRACE_EVENT(xfer_secondary_pool, - TP_PROTO(const char *pool_name, int xfer_bits, int request_bits, -int pool_entropy, int input_entropy), - - TP_ARGS(pool_name, xfer_bits, request_bits, pool_entropy, - input_entropy), - - TP_STRUCT__entry( - __field( const char *, pool_name ) - __field( int, xfer_bits ) - __field( int, request_bits) - __field( int, pool_entropy) - __field( int, input_entropy ) - ), - - TP_fast_assign( - __entry->pool_name = pool_name; - __entry->xfer_bits = xfer_bits; - __entry->request_bits = request_bits; - __entry->pool_entropy = pool_entropy; - __entry->input_entropy = input_entropy; - ), - - TP_printk("pool %s xfer_bits %d request_bits %d pool_entropy %d " - "input_entropy %d", __entry->pool_name, __entry->xfer_bits, - __entry->request_bits, __entry->pool_entropy, - __entry->input_entropy) -); - DECLARE_EVENT_CLASS(random__get_random_bytes, TP_PROTO(int nbytes, unsigned long
Re: [PATCH RESEND] random: remove dead code left over from blocking pool
On Tue, Jan 12, 2021 at 11:29:38AM -0800, Eric Biggers wrote: > From: Eric Biggers > > Remove some dead code that was left over following commit 90ea1c6436d2 > ("random: remove the blocking pool"). > > Cc: linux-cry...@vger.kernel.org > Cc: Andy Lutomirski > Cc: Jann Horn > Cc: Theodore Ts'o > Reviewed-by: Andy Lutomirski > Signed-off-by: Eric Biggers > --- > > Andrew, please consider taking this patch since the maintainer has been > ignoring it for 4 months > (https://lkml.kernel.org/lkml/20200916043652.96640-1-ebigg...@kernel.org/T/#u). > Ping.
Re: [PATCH RESEND] random: remove dead code left over from blocking pool
On Tue, 12 Jan 2021 at 20:30, Eric Biggers wrote: > > From: Eric Biggers > > Remove some dead code that was left over following commit 90ea1c6436d2 > ("random: remove the blocking pool"). > > Cc: linux-cry...@vger.kernel.org > Cc: Andy Lutomirski > Cc: Jann Horn > Cc: Theodore Ts'o > Reviewed-by: Andy Lutomirski > Signed-off-by: Eric Biggers Acked-by: Ard Biesheuvel > --- > > Andrew, please consider taking this patch since the maintainer has been > ignoring it for 4 months > (https://lkml.kernel.org/lkml/20200916043652.96640-1-ebigg...@kernel.org/T/#u). > > > drivers/char/random.c | 17 ++- > include/trace/events/random.h | 83 --- > 2 files changed, 3 insertions(+), 97 deletions(-) > > diff --git a/drivers/char/random.c b/drivers/char/random.c > index a894c0559a8cf..bbc5098b1a81f 100644 > --- a/drivers/char/random.c > +++ b/drivers/char/random.c > @@ -500,7 +500,6 @@ struct entropy_store { > unsigned short add_ptr; > unsigned short input_rotate; > int entropy_count; > - unsigned int initialized:1; > unsigned int last_data_init:1; > __u8 last_data[EXTRACT_SIZE]; > }; > @@ -660,7 +659,7 @@ static void process_random_ready_list(void) > */ > static void credit_entropy_bits(struct entropy_store *r, int nbits) > { > - int entropy_count, orig, has_initialized = 0; > + int entropy_count, orig; > const int pool_size = r->poolinfo->poolfracbits; > int nfrac = nbits << ENTROPY_SHIFT; > > @@ -717,23 +716,14 @@ static void credit_entropy_bits(struct entropy_store > *r, int nbits) > if (cmpxchg(&r->entropy_count, orig, entropy_count) != orig) > goto retry; > > - if (has_initialized) { > - r->initialized = 1; > - kill_fasync(&fasync, SIGIO, POLL_IN); > - } > - > trace_credit_entropy_bits(r->name, nbits, > entropy_count >> ENTROPY_SHIFT, _RET_IP_); > > if (r == &input_pool) { > int entropy_bits = entropy_count >> ENTROPY_SHIFT; > > - if (crng_init < 2) { > - if (entropy_bits < 128) > - return; > + if (crng_init < 2 && entropy_bits >= 128) > crng_reseed(&primary_crng, r); > - entropy_bits = ENTROPY_BITS(r); > - } > } > } > > @@ -1385,8 +1375,7 @@ static size_t account(struct entropy_store *r, size_t > nbytes, int min, > } > > /* > - * This function does the actual extraction for extract_entropy and > - * extract_entropy_user. > + * This function does the actual extraction for extract_entropy. > * > * Note: we assume that .poolwords is a multiple of 16 words. > */ > diff --git a/include/trace/events/random.h b/include/trace/events/random.h > index 9570a10cb949b..3d7b432ca5f31 100644 > --- a/include/trace/events/random.h > +++ b/include/trace/events/random.h > @@ -85,28 +85,6 @@ TRACE_EVENT(credit_entropy_bits, > __entry->entropy_count, (void *)__entry->IP) > ); > > -TRACE_EVENT(push_to_pool, > - TP_PROTO(const char *pool_name, int pool_bits, int input_bits), > - > - TP_ARGS(pool_name, pool_bits, input_bits), > - > - TP_STRUCT__entry( > - __field( const char *, pool_name ) > - __field( int, pool_bits ) > - __field( int, input_bits ) > - ), > - > - TP_fast_assign( > - __entry->pool_name = pool_name; > - __entry->pool_bits = pool_bits; > - __entry->input_bits = input_bits; > - ), > - > - TP_printk("%s: pool_bits %d input_pool_bits %d", > - __entry->pool_name, __entry->pool_bits, > - __entry->input_bits) > -); > - > TRACE_EVENT(debit_entropy, > TP_PROTO(const char *pool_name, int debit_bits), > > @@ -161,35 +139,6 @@ TRACE_EVENT(add_disk_randomness, > MINOR(__entry->dev), __entry->input_bits) > ); > > -TRACE_EVENT(xfer_secondary_pool, > - TP_PROTO(const char *pool_name, int xfer_bits, int request_bits, > -int pool_entropy, int input_entropy), > - > - TP_ARGS(pool_name, xfer_bits, request_bits, pool_entropy, > - input_entropy), > - > - TP_STRUCT__entry( > - __field( const char *, pool_name ) > - __field( int, xfer_bits ) > - __field( int, request_bits) > - __field( int, pool_entropy) > - __field( int, input_entropy ) > - ), > - > - TP_fast_assign( > - __entry->pool_name = pool_name; > - __entry->xfer_bits = xfer_bits; > - __entry->request_bits = request_bi
[PATCH RESEND] random: remove dead code left over from blocking pool
From: Eric Biggers Remove some dead code that was left over following commit 90ea1c6436d2 ("random: remove the blocking pool"). Cc: linux-cry...@vger.kernel.org Cc: Andy Lutomirski Cc: Jann Horn Cc: Theodore Ts'o Reviewed-by: Andy Lutomirski Signed-off-by: Eric Biggers --- Andrew, please consider taking this patch since the maintainer has been ignoring it for 4 months (https://lkml.kernel.org/lkml/20200916043652.96640-1-ebigg...@kernel.org/T/#u). drivers/char/random.c | 17 ++- include/trace/events/random.h | 83 --- 2 files changed, 3 insertions(+), 97 deletions(-) diff --git a/drivers/char/random.c b/drivers/char/random.c index a894c0559a8cf..bbc5098b1a81f 100644 --- a/drivers/char/random.c +++ b/drivers/char/random.c @@ -500,7 +500,6 @@ struct entropy_store { unsigned short add_ptr; unsigned short input_rotate; int entropy_count; - unsigned int initialized:1; unsigned int last_data_init:1; __u8 last_data[EXTRACT_SIZE]; }; @@ -660,7 +659,7 @@ static void process_random_ready_list(void) */ static void credit_entropy_bits(struct entropy_store *r, int nbits) { - int entropy_count, orig, has_initialized = 0; + int entropy_count, orig; const int pool_size = r->poolinfo->poolfracbits; int nfrac = nbits << ENTROPY_SHIFT; @@ -717,23 +716,14 @@ static void credit_entropy_bits(struct entropy_store *r, int nbits) if (cmpxchg(&r->entropy_count, orig, entropy_count) != orig) goto retry; - if (has_initialized) { - r->initialized = 1; - kill_fasync(&fasync, SIGIO, POLL_IN); - } - trace_credit_entropy_bits(r->name, nbits, entropy_count >> ENTROPY_SHIFT, _RET_IP_); if (r == &input_pool) { int entropy_bits = entropy_count >> ENTROPY_SHIFT; - if (crng_init < 2) { - if (entropy_bits < 128) - return; + if (crng_init < 2 && entropy_bits >= 128) crng_reseed(&primary_crng, r); - entropy_bits = ENTROPY_BITS(r); - } } } @@ -1385,8 +1375,7 @@ static size_t account(struct entropy_store *r, size_t nbytes, int min, } /* - * This function does the actual extraction for extract_entropy and - * extract_entropy_user. + * This function does the actual extraction for extract_entropy. * * Note: we assume that .poolwords is a multiple of 16 words. */ diff --git a/include/trace/events/random.h b/include/trace/events/random.h index 9570a10cb949b..3d7b432ca5f31 100644 --- a/include/trace/events/random.h +++ b/include/trace/events/random.h @@ -85,28 +85,6 @@ TRACE_EVENT(credit_entropy_bits, __entry->entropy_count, (void *)__entry->IP) ); -TRACE_EVENT(push_to_pool, - TP_PROTO(const char *pool_name, int pool_bits, int input_bits), - - TP_ARGS(pool_name, pool_bits, input_bits), - - TP_STRUCT__entry( - __field( const char *, pool_name ) - __field( int, pool_bits ) - __field( int, input_bits ) - ), - - TP_fast_assign( - __entry->pool_name = pool_name; - __entry->pool_bits = pool_bits; - __entry->input_bits = input_bits; - ), - - TP_printk("%s: pool_bits %d input_pool_bits %d", - __entry->pool_name, __entry->pool_bits, - __entry->input_bits) -); - TRACE_EVENT(debit_entropy, TP_PROTO(const char *pool_name, int debit_bits), @@ -161,35 +139,6 @@ TRACE_EVENT(add_disk_randomness, MINOR(__entry->dev), __entry->input_bits) ); -TRACE_EVENT(xfer_secondary_pool, - TP_PROTO(const char *pool_name, int xfer_bits, int request_bits, -int pool_entropy, int input_entropy), - - TP_ARGS(pool_name, xfer_bits, request_bits, pool_entropy, - input_entropy), - - TP_STRUCT__entry( - __field( const char *, pool_name ) - __field( int, xfer_bits ) - __field( int, request_bits) - __field( int, pool_entropy) - __field( int, input_entropy ) - ), - - TP_fast_assign( - __entry->pool_name = pool_name; - __entry->xfer_bits = xfer_bits; - __entry->request_bits = request_bits; - __entry->pool_entropy = pool_entropy; - __entry->input_entropy = input_entropy; - ), - - TP_printk("pool %s xfer_bits %d request_bits %d pool_entropy %d " - "input_entropy %d", __entry->pool_name, __entry->xfer_bits, - __entry->request_bits, __entr