Re: random() in multi-threaded pgbench - Mailing list pgsql-hackers

From Tom Lane
Subject Re: random() in multi-threaded pgbench
Date
Msg-id 23213.1260563267@sss.pgh.pa.us
Whole thread Raw
In response to random() in multi-threaded pgbench  (Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: random() in multi-threaded pgbench  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp> writes:
> While testing the pgbench setshell command patch with -j option,
> I found all threads use the same sequence of random value.

Were they actually threads, or were you testing the code while it had
the broken configure script that didn't set ENABLE_THREAD_SAFETY?
I think you might have hit the same thing I just ran into, that in
a *non-threaded* build each subprocess will generate the same random
sequence.

> At first, I think we need to call srandom() in each thread,

Each sub-job I think.

> but the manual says we should use random_r() instead of random()
> on multi-threaded programs.
> http://www.kernel.org/doc/man-pages/online/pages/man3/random_r.3.html

It only says that you need those if you want an *independent* random
sequence for each thread.  pgbench never had that before and I doubt
we need it now.  In any case, the same page also says these are a
glibc-ism not a standard API, so we can't really rely on them.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Stephen Smalley
Date:
Subject: Re: Adding support for SE-Linux security
Next
From: Stephen Frost
Date:
Subject: Re: Adding support for SE-Linux security