Re: [PATCH] tests: use g_test_rand_int

[Thomas Huth <thuth@redhat.com>]

On 11/12/2019 15.23, Paolo Bonzini wrote:
> g_test_rand_int provides a reproducible random integer number, using a
> different number seed every time but allowing reproduction using the
> --seed command line option.  It is thus better suited to tests than
> g_random_int or random.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  block/vhdx.c         | 2 +-
>  block/vmdk.c         | 4 ++--
>  tests/ivshmem-test.c | 2 +-
>  tests/test-bitmap.c  | 8 ++++----
>  tests/test-qga.c     | 4 ++--
>  5 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/block/vhdx.c b/block/vhdx.c
> index f02d261..36465d5 100644
> --- a/block/vhdx.c
> +++ b/block/vhdx.c
> @@ -1504,7 +1504,7 @@ static int vhdx_create_new_headers(BlockBackend *blk, uint64_t image_size,
>      hdr = g_new0(VHDXHeader, 1);
>  
>      hdr->signature       = VHDX_HEADER_SIGNATURE;
> -    hdr->sequence_number = g_random_int();
> +    hdr->sequence_number = g_test_rand_int();
>      hdr->log_version     = 0;
>      hdr->version         = 1;
>      hdr->log_length      = log_size;
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 20e909d..eba96bf 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -2037,7 +2037,7 @@ static int vmdk_pwritev(BlockDriverState *bs, uint64_t offset,
>          /* update CID on the first write every time the virtual disk is
>           * opened */
>          if (!s->cid_updated) {
> -            ret = vmdk_write_cid(bs, g_random_int());
> +            ret = vmdk_write_cid(bs, g_test_rand_int());
>              if (ret < 0) {
>                  return ret;
>              }
> @@ -2499,7 +2499,7 @@ static int coroutine_fn vmdk_co_do_create(int64_t size,
>  
>      /* generate descriptor file */
>      desc = g_strdup_printf(desc_template,
> -                           g_random_int(),
> +                           g_test_rand_int(),
>                             parent_cid,
>                             BlockdevVmdkSubformat_str(subformat),
>                             parent_desc_line,

Why do you also change this in block/ and not only in tests/ ?

 Thomas