Code changes look fine to me.
>
> Usually, Random Number Generator is abbreviated to RNG/rng.
> so replacing RndRandom with RngRandom seems more reasonable
> and keep consistent with RngBackend.
Reviewed-by: Pankaj Gupta <pagu...@redhat.com>
>
> Signed-off-by: Wei Jiangang <weijg.f...@cn.fujitsu.com>
> ---
> backends/rng-random.c | 18 +++++++++---------
> include/hw/virtio/virtio-rng.h | 2 +-
> include/sysemu/rng-random.h | 4 ++--
> 3 files changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/backends/rng-random.c b/backends/rng-random.c
> index 2e44e25..e2a49b0 100644
> --- a/backends/rng-random.c
> +++ b/backends/rng-random.c
> @@ -17,7 +17,7 @@
> #include "qapi/qmp/qerror.h"
> #include "qemu/main-loop.h"
>
> -struct RndRandom
> +struct RngRandom
> {
> RngBackend parent;
>
> @@ -34,7 +34,7 @@ struct RndRandom
>
> static void entropy_available(void *opaque)
> {
> - RndRandom *s = RNG_RANDOM(opaque);
> + RngRandom *s = RNG_RANDOM(opaque);
>
> while (!QSIMPLEQ_EMPTY(&s->parent.requests)) {
> RngRequest *req = QSIMPLEQ_FIRST(&s->parent.requests);
> @@ -57,7 +57,7 @@ static void entropy_available(void *opaque)
>
> static void rng_random_request_entropy(RngBackend *b, RngRequest *req)
> {
> - RndRandom *s = RNG_RANDOM(b);
> + RngRandom *s = RNG_RANDOM(b);
>
> if (QSIMPLEQ_EMPTY(&s->parent.requests)) {
> /* If there are no pending requests yet, we need to
> @@ -68,7 +68,7 @@ static void rng_random_request_entropy(RngBackend *b,
> RngRequest *req)
>
> static void rng_random_opened(RngBackend *b, Error **errp)
> {
> - RndRandom *s = RNG_RANDOM(b);
> + RngRandom *s = RNG_RANDOM(b);
>
> if (s->filename == NULL) {
> error_setg(errp, QERR_INVALID_PARAMETER_VALUE,
> @@ -83,7 +83,7 @@ static void rng_random_opened(RngBackend *b, Error **errp)
>
> static char *rng_random_get_filename(Object *obj, Error **errp)
> {
> - RndRandom *s = RNG_RANDOM(obj);
> + RngRandom *s = RNG_RANDOM(obj);
>
> return g_strdup(s->filename);
> }
> @@ -92,7 +92,7 @@ static void rng_random_set_filename(Object *obj, const char
> *filename,
> Error **errp)
> {
> RngBackend *b = RNG_BACKEND(obj);
> - RndRandom *s = RNG_RANDOM(obj);
> + RngRandom *s = RNG_RANDOM(obj);
>
> if (b->opened) {
> error_setg(errp, QERR_PERMISSION_DENIED);
> @@ -105,7 +105,7 @@ static void rng_random_set_filename(Object *obj, const
> char *filename,
>
> static void rng_random_init(Object *obj)
> {
> - RndRandom *s = RNG_RANDOM(obj);
> + RngRandom *s = RNG_RANDOM(obj);
>
> object_property_add_str(obj, "filename",
> rng_random_get_filename,
> @@ -118,7 +118,7 @@ static void rng_random_init(Object *obj)
>
> static void rng_random_finalize(Object *obj)
> {
> - RndRandom *s = RNG_RANDOM(obj);
> + RngRandom *s = RNG_RANDOM(obj);
>
> if (s->fd != -1) {
> qemu_set_fd_handler(s->fd, NULL, NULL, NULL);
> @@ -139,7 +139,7 @@ static void rng_random_class_init(ObjectClass *klass,
> void *data)
> static const TypeInfo rng_random_info = {
> .name = TYPE_RNG_RANDOM,
> .parent = TYPE_RNG_BACKEND,
> - .instance_size = sizeof(RndRandom),
> + .instance_size = sizeof(RngRandom),
> .class_init = rng_random_class_init,
> .instance_init = rng_random_init,
> .instance_finalize = rng_random_finalize,
> diff --git a/include/hw/virtio/virtio-rng.h b/include/hw/virtio/virtio-rng.h
> index 3f07de7..2bc1ee5 100644
> --- a/include/hw/virtio/virtio-rng.h
> +++ b/include/hw/virtio/virtio-rng.h
> @@ -26,7 +26,7 @@ struct VirtIORNGConf {
> RngBackend *rng;
> uint64_t max_bytes;
> uint32_t period_ms;
> - RndRandom *default_backend;
> + RngRandom *default_backend;
> };
>
> typedef struct VirtIORNG {
> diff --git a/include/sysemu/rng-random.h b/include/sysemu/rng-random.h
> index 4332772..38186fe 100644
> --- a/include/sysemu/rng-random.h
> +++ b/include/sysemu/rng-random.h
> @@ -15,8 +15,8 @@
> #include "qom/object.h"
>
> #define TYPE_RNG_RANDOM "rng-random"
> -#define RNG_RANDOM(obj) OBJECT_CHECK(RndRandom, (obj), TYPE_RNG_RANDOM)
> +#define RNG_RANDOM(obj) OBJECT_CHECK(RngRandom, (obj), TYPE_RNG_RANDOM)
>
> -typedef struct RndRandom RndRandom;
> +typedef struct RngRandom RngRandom;
>
> #endif
> --
> 1.9.3
>
>
>
>
>
