[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v3 17/23] hw/misc/nrf51_rng: Use qemu_guest_getran
From: |
Joel Stanley |
Subject: |
Re: [Qemu-arm] [PATCH v3 17/23] hw/misc/nrf51_rng: Use qemu_guest_getrandom_nofail |
Date: |
Fri, 15 Mar 2019 03:52:18 +0000 |
On Fri, 15 Mar 2019 at 03:26, Richard Henderson
<address@hidden> wrote:
>
> The random number is intended for use by the guest. As such, we should
> honor the -seed argument for reproducibility. Use the *_nofail routine
> instead of error_abort directly.
>
> Cc: address@hidden
> Cc: Joel Stanley <address@hidden>
> Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Joel Stanley <address@hidden>
> ---
> hw/misc/nrf51_rng.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/misc/nrf51_rng.c b/hw/misc/nrf51_rng.c
> index d188f044f4..3400e90a9b 100644
> --- a/hw/misc/nrf51_rng.c
> +++ b/hw/misc/nrf51_rng.c
> @@ -14,7 +14,7 @@
> #include "qapi/error.h"
> #include "hw/arm/nrf51.h"
> #include "hw/misc/nrf51_rng.h"
> -#include "crypto/random.h"
> +#include "qemu/guest-random.h"
>
> static void update_irq(NRF51RNGState *s)
> {
> @@ -145,7 +145,7 @@ static void nrf51_rng_timer_expire(void *opaque)
> {
> NRF51RNGState *s = NRF51_RNG(opaque);
>
> - qcrypto_random_bytes(&s->value, 1, &error_abort);
> + qemu_guest_getrandom_nofail(&s->value, 1);
>
> s->event_valrdy = 1;
> qemu_set_irq(s->eep_valrdy, 1);
> --
> 2.17.2
>