Re: [PATCHES] Win32 CHECK_FOR_INTERRUPTS() performance tweak - Mailing list pgsql-hackers

From Qingqing Zhou
Subject Re: [PATCHES] Win32 CHECK_FOR_INTERRUPTS() performance tweak
Date
Msg-id Pine.LNX.4.58.0510211602440.23248@josh.db
Whole thread Raw
In response to Re: [PATCHES] Win32 CHECK_FOR_INTERRUPTS() performance tweak  ("Magnus Hagander" <mha@sollentuna.net>)
List pgsql-hackers

On Fri, 21 Oct 2005, Magnus Hagander wrote:

> > > Shall we add "volatile" quanlifier to at least pg_signal_queue?
> >
> > If that's changed by a separate thread, "volatile" seems essential.
> > What about the mask variable?
>
> Yes, that does seem right. Previously it would never be concurrently
> modified, because it was always locked by the critical section, but now
> we read it without locking, and we certainly don't want that optimized
> away.
>
> The mask is only ever written by the main thread, never by the signal
> dispatching thread. So I think that one could do without.
>

Agreed.

Regards,
Qingqing


pgsql-hackers by date:

Previous
From: "Merlin Moncure"
Date:
Subject: Re: [PATCHES] Win32 CHECK_FOR_INTERRUPTS() performance tweak
Next
From: Tom Lane
Date:
Subject: Re: [PATCHES] Win32 CHECK_FOR_INTERRUPTS() performance tweak