Re: Trying out <stdatomic.h>

From: Greg Burd <greg(at)burd(dot)me>
To: shihao zhong <zhong950419(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Trying out <stdatomic.h>
Date: 2026-10-23 16:30:24
Message-ID: 7gJmxtbunjsr2_04YEK5pN3XH0ZqYY8E2HQxFZFEsCUxScihXgxQyL5KNQ6nCO_X1bOLgsiCcPX1iy7sB9BSQH8MoqdB4-wgRTYAYXo0UR4=@burd.me
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On Thursday, October 22nd, 2026 at 9:17 PM, shihao zhong <zhong950419(at)gmail(dot)com> wrote:

> Hi Greg,
> I did some fuzz testing using Opus today and did not find anything.
>
> I also did some manual experiments and I think everything is working fine in
> mac.
>
> V10 looks good to me.
>
> Thanks,
> Shihao

Hi Shihao,

Thank you for the testing, and for the three real bugs you found along
the way. I've added you as Reviewed-by.

v11 attached, rebased on master (1910daf9831). The code is unchanged
from v10; this round only corrects the commit messages, which had
drifted from the code (they still mentioned pg_atomic_uint8/16, which
were removed earlier, a test_atomics module that no longer exists, and
a pg_atomic_flag-based spinlock, when slock_t is a pg_atomic_uint32).

As before, 0003 is named .patch_ so cfbot tests 0001 and 0002 only.

best.

-greg

Attachment Content-Type Size
v11-0002-NOT-FOR-MERGE-Add-atomics-benchmark-harness.patch text/x-patch 88.5 KB
v11-0003-NOT-FOR-MERGE.-yet.-Remove-traditional-atomics-u.patch_ application/octet-stream 99.9 KB
v11-0001-Add-an-opt-in-C11-stdatomic.h-implementation-of-.patch text/x-patch 66.5 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Burd 2026-10-23 17:00:08 Re: [PATCH] Corruption Issue: Fix missing tts_tid in ExecForceStoreHeapTuple
Previous Message Heikki Linnakangas 2026-10-23 16:17:50 Re: Return pg_control from pg_backup_stop().