[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 19/23] hw/misc/exynos4210_rng: Use qemu_guest
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 19/23] hw/misc/exynos4210_rng: Use qemu_guest_getrandom |
Date: |
Thu, 11 Apr 2019 11:53:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 3/15/19 4:26 AM, Richard Henderson wrote:
> The random number is intended for use by the guest. As such, we should
> honor the -seed argument for reproducibility.
>
> Cc: address@hidden
> Cc: Igor Mitsyanko <address@hidden>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> hw/misc/exynos4210_rng.c | 11 ++++-------
> 1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/hw/misc/exynos4210_rng.c b/hw/misc/exynos4210_rng.c
> index 4ecbebd2d7..0e70ffb404 100644
> --- a/hw/misc/exynos4210_rng.c
> +++ b/hw/misc/exynos4210_rng.c
> @@ -18,10 +18,10 @@
> */
>
> #include "qemu/osdep.h"
> -#include "crypto/random.h"
> #include "hw/sysbus.h"
> #include "qapi/error.h"
> #include "qemu/log.h"
> +#include "qemu/guest-random.h"
>
> #define DEBUG_EXYNOS_RNG 0
>
> @@ -109,7 +109,6 @@ static void exynos4210_rng_set_seed(Exynos4210RngState
> *s, unsigned int i,
> static void exynos4210_rng_run_engine(Exynos4210RngState *s)
> {
> Error *err = NULL;
> - int ret;
>
> /* Seed set? */
> if ((s->reg_status & EXYNOS4210_RNG_STATUS_SEED_SETTING_DONE) == 0) {
> @@ -127,13 +126,11 @@ static void
> exynos4210_rng_run_engine(Exynos4210RngState *s)
> }
>
> /* Get randoms */
> - ret = qcrypto_random_bytes((uint8_t *)s->randr_value,
> - sizeof(s->randr_value), &err);
> - if (!ret) {
> + if (qemu_guest_getrandom(s->randr_value, sizeof(s->randr_value), &err)) {
> + error_report_err(err);
> + } else {
> /* Notify that PRNG is ready */
> s->reg_status |= EXYNOS4210_RNG_STATUS_PRNG_DONE;
> - } else {
> - error_report_err(err);
> }
>
> out:
>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 19/23] hw/misc/exynos4210_rng: Use qemu_guest_getrandom,
Philippe Mathieu-Daudé <=