agora inbox for pgsql-hackers@postgresql.org
help / color / mirror / Atom feedFrom: Nathan Bossart <nathandbossart@gmail.com>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-hackers@postgresql.org
Subject: Re: convert various variables to atomics
Date: Wed, 22 Jul 2026 09:06:44 -0400
Message-ID: <amDAZAVob4DJjUfW@nathan> (raw)
In-Reply-To: <9d8c317d-d933-46c7-b675-4b9308eaca2b@eisentraut.org>
References: <alAJeRRzehDjLaF1@nathan>
<9d8c317d-d933-46c7-b675-4b9308eaca2b@eisentraut.org>
On Wed, Jul 22, 2026 at 02:31:49PM +0200, Peter Eisentraut wrote:
> Here is an instance that seems obviously wrong:
>
> /* Buffer id of the buffer that Startup process waits for pin on, or -1 */
> - int startupBufferPinWaitBufId;
> + pg_atomic_uint32 startupBufferPinWaitBufId;
I may just be undercaffeinated, but what is wrong with this case? AFAICT
the casting should work as expected, and I see other examples that do
something similar, like avLauncherProc.
--
nathan
view thread (22+ messages) latest in thread
Message-ID: <amDAZAVob4DJjUfW@nathan>
Permalink: ../amDAZAVob4DJjUfW@nathan/
Also on: postgresql.org/message-id/amDAZAVob4DJjUfW@nathan
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: nathandbossart@gmail.com, peter@eisentraut.org
Subject: Re: convert various variables to atomics
In-Reply-To: <amDAZAVob4DJjUfW@nathan>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox