Re: [HACKERS] Configure with thread sanitizer fails the thread test

2015-08-18 Thread Robert Haas
On Mon, Aug 17, 2015 at 3:02 PM, Andres Freund wrote: > On 2015-08-17 14:31:24 -0300, Alvaro Herrera wrote: >> The postmaster process in particular runs in a rather unusual >> arrangement, where most of the interesting stuff does happen in signal >> handlers. > > FWIW, I think it might be worthwhi

Re: [HACKERS] Configure with thread sanitizer fails the thread test

2015-08-17 Thread Andres Freund
On 2015-08-17 14:31:24 -0300, Alvaro Herrera wrote: > The postmaster process in particular runs in a rather unusual > arrangement, where most of the interesting stuff does happen in signal > handlers. FWIW, I think it might be worthwhile to convert postmaster into a loop over a process local latch

Re: [HACKERS] Configure with thread sanitizer fails the thread test

2015-08-17 Thread Andres Freund
On 2015-08-17 07:37:45 +, Ewan Higgs wrote: > So I changed volatile to _Atomic and continued (patch is in > thread_test_atomic.patch). I then ran it against sqlsmith. The good > news: I didn't happen to find any problems in normal use. The bad > news: I did find a lot of warnings about improper

Re: [HACKERS] Configure with thread sanitizer fails the thread test

2015-08-17 Thread Alvaro Herrera
Ewan Higgs wrote: > So I changed volatile to _Atomic and continued (patch is in > thread_test_atomic.patch). I then ran it against sqlsmith. The good > news: I didn't happen to find any problems in normal use. The bad > news: I did find a lot of warnings about improper use of functions > like mall