Message ID | 6fbc0bfcbfc1fa2c76fd574f5b6f552b11be7fde.1577088521.git.luto@kernel.org |
---|---|
State | Not Applicable |
Headers | show |
Series | Rework random blocking | expand |
On Mon, Dec 23, 2019 at 12:20:44AM -0800, Andy Lutomirski wrote: > crng_init_wait is only used to wayt for crng_init to be set to 2, so > there's no point to waking it when crng_init is set to 1. Remove the > unnecessary wake_up_interruptible() call. > > Signed-off-by: Andy Lutomirski <luto@kernel.org> Applied with a spelling fix ("wayt->wait"). - Ted
diff --git a/drivers/char/random.c b/drivers/char/random.c index cda12933a17d..c6252a3a4aec 100644 --- a/drivers/char/random.c +++ b/drivers/char/random.c @@ -946,7 +946,6 @@ static int crng_fast_load(const char *cp, size_t len) if (crng_init_cnt >= CRNG_INIT_CNT_THRESH) { invalidate_batched_entropy(); crng_init = 1; - wake_up_interruptible(&crng_init_wait); pr_notice("random: fast init done\n"); } return 1;
crng_init_wait is only used to wayt for crng_init to be set to 2, so there's no point to waking it when crng_init is set to 1. Remove the unnecessary wake_up_interruptible() call. Signed-off-by: Andy Lutomirski <luto@kernel.org> --- drivers/char/random.c | 1 - 1 file changed, 1 deletion(-)