| 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 |
| 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(). |