On Fri, 15 Mar 2019 at 03:26, Richard Henderson <richard.hender...@linaro.org> 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: qemu-...@nongnu.org > Cc: Joel Stanley <j...@jms.id.au> > Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
Reviewed-by: Joel Stanley <j...@jms.id.au> > --- > 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 >