* [PATCH v2] tests: use g_test_rand_int
@ 2019-12-12 1:17 Paolo Bonzini
2019-12-12 1:40 ` Richard Henderson
2019-12-12 7:14 ` Thomas Huth
0 siblings, 2 replies; 3+ messages in thread
From: Paolo Bonzini @ 2019-12-12 1:17 UTC (permalink / raw)
To: qemu-devel
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>
---
tests/ivshmem-test.c | 2 +-
tests/test-bitmap.c | 8 ++++----
tests/test-qga.c | 4 ++--
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/tests/ivshmem-test.c b/tests/ivshmem-test.c
index be9aa92..ecda256 100644
--- a/tests/ivshmem-test.c
+++ b/tests/ivshmem-test.c
@@ -443,7 +443,7 @@ static gchar *mktempshm(int size, int *fd)
while (true) {
gchar *name;
- name = g_strdup_printf("/qtest-%u-%u", getpid(), g_random_int());
+ name = g_strdup_printf("/qtest-%u-%u", getpid(), g_test_rand_int());
*fd = shm_open(name, O_CREAT|O_RDWR|O_EXCL,
S_IRWXU|S_IRWXG|S_IRWXO);
if (*fd > 0) {
diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
index 087e02a..2f5b714 100644
--- a/tests/test-bitmap.c
+++ b/tests/test-bitmap.c
@@ -22,10 +22,10 @@ static void check_bitmap_copy_with_offset(void)
bmap2 = bitmap_new(BMAP_SIZE);
bmap3 = bitmap_new(BMAP_SIZE);
- bmap1[0] = random();
- bmap1[1] = random();
- bmap1[2] = random();
- bmap1[3] = random();
+ bmap1[0] = g_test_rand_int();
+ bmap1[1] = g_test_rand_int();
+ bmap1[2] = g_test_rand_int();
+ bmap1[3] = g_test_rand_int();
total = BITS_PER_LONG * 4;
/* Shift 115 bits into bmap2 */
diff --git a/tests/test-qga.c b/tests/test-qga.c
index 1ca49bb..d2b2435 100644
--- a/tests/test-qga.c
+++ b/tests/test-qga.c
@@ -143,7 +143,7 @@ static void qmp_assertion_message_error(const char *domain,
static void test_qga_sync_delimited(gconstpointer fix)
{
const TestFixture *fixture = fix;
- guint32 v, r = g_random_int();
+ guint32 v, r = g_test_rand_int();
unsigned char c;
QDict *ret;
@@ -186,7 +186,7 @@ static void test_qga_sync_delimited(gconstpointer fix)
static void test_qga_sync(gconstpointer fix)
{
const TestFixture *fixture = fix;
- guint32 v, r = g_random_int();
+ guint32 v, r = g_test_rand_int();
QDict *ret;
/*
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] tests: use g_test_rand_int
2019-12-12 1:17 [PATCH v2] tests: use g_test_rand_int Paolo Bonzini
@ 2019-12-12 1:40 ` Richard Henderson
2019-12-12 7:14 ` Thomas Huth
1 sibling, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2019-12-12 1:40 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel
On 12/11/19 5:17 PM, 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>
> ---
> tests/ivshmem-test.c | 2 +-
> tests/test-bitmap.c | 8 ++++----
> tests/test-qga.c | 4 ++--
> 3 files changed, 7 insertions(+), 7 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] tests: use g_test_rand_int
2019-12-12 1:17 [PATCH v2] tests: use g_test_rand_int Paolo Bonzini
2019-12-12 1:40 ` Richard Henderson
@ 2019-12-12 7:14 ` Thomas Huth
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Huth @ 2019-12-12 7:14 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel
On 12/12/2019 02.17, 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>
> ---
> tests/ivshmem-test.c | 2 +-
> tests/test-bitmap.c | 8 ++++----
> tests/test-qga.c | 4 ++--
> 3 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/tests/ivshmem-test.c b/tests/ivshmem-test.c
> index be9aa92..ecda256 100644
> --- a/tests/ivshmem-test.c
> +++ b/tests/ivshmem-test.c
> @@ -443,7 +443,7 @@ static gchar *mktempshm(int size, int *fd)
> while (true) {
> gchar *name;
>
> - name = g_strdup_printf("/qtest-%u-%u", getpid(), g_random_int());
> + name = g_strdup_printf("/qtest-%u-%u", getpid(), g_test_rand_int());
> *fd = shm_open(name, O_CREAT|O_RDWR|O_EXCL,
> S_IRWXU|S_IRWXG|S_IRWXO);
> if (*fd > 0) {
> diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
> index 087e02a..2f5b714 100644
> --- a/tests/test-bitmap.c
> +++ b/tests/test-bitmap.c
> @@ -22,10 +22,10 @@ static void check_bitmap_copy_with_offset(void)
> bmap2 = bitmap_new(BMAP_SIZE);
> bmap3 = bitmap_new(BMAP_SIZE);
>
> - bmap1[0] = random();
> - bmap1[1] = random();
> - bmap1[2] = random();
> - bmap1[3] = random();
> + bmap1[0] = g_test_rand_int();
> + bmap1[1] = g_test_rand_int();
> + bmap1[2] = g_test_rand_int();
> + bmap1[3] = g_test_rand_int();
> total = BITS_PER_LONG * 4;
>
> /* Shift 115 bits into bmap2 */
> diff --git a/tests/test-qga.c b/tests/test-qga.c
> index 1ca49bb..d2b2435 100644
> --- a/tests/test-qga.c
> +++ b/tests/test-qga.c
> @@ -143,7 +143,7 @@ static void qmp_assertion_message_error(const char *domain,
> static void test_qga_sync_delimited(gconstpointer fix)
> {
> const TestFixture *fixture = fix;
> - guint32 v, r = g_random_int();
> + guint32 v, r = g_test_rand_int();
> unsigned char c;
> QDict *ret;
>
> @@ -186,7 +186,7 @@ static void test_qga_sync_delimited(gconstpointer fix)
> static void test_qga_sync(gconstpointer fix)
> {
> const TestFixture *fixture = fix;
> - guint32 v, r = g_random_int();
> + guint32 v, r = g_test_rand_int();
> QDict *ret;
>
> /*
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-12-12 7:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-12 1:17 [PATCH v2] tests: use g_test_rand_int Paolo Bonzini
2019-12-12 1:40 ` Richard Henderson
2019-12-12 7:14 ` Thomas Huth
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).