| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: convert various variables to atomics |
| Date: | 2026-07-23 19:56:16 |
| Message-ID: | amJx4Lwx4nuuExYT@nathan |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Jul 22, 2026 at 09:31:49AM -0400, Nathan Bossart wrote:
> Ah. It looks like we could use 0 as the sentinel and simplify the call
> sites. They subtract one before calling SetStartupBufferPinWaitBufId() and
> add one after calling GetStartupBufferPinWaitBufId().
I added a new prerequisite patch (v2-0004) that does this.
--
nathan
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-convert-SISeg-maxMsgNum-to-an-atomic.patch | text/plain | 5.9 KB |
| v2-0002-convert-ParallelBitmapHeapState-state-to-an-atomi.patch | text/plain | 2.6 KB |
| v2-0003-convert-FixedParallelState-last_xlog_end-to-an-at.patch | text/plain | 2.8 KB |
| v2-0004-use-Buffer-instead-of-buffer-ID-for-startup-s-buf.patch | text/plain | 5.3 KB |
| v2-0005-convert-PROC_HDR-startupBufferPinWaitBuf-to-an-at.patch | text/plain | 2.3 KB |
| v2-0006-convert-Sharedsort-currentWorker-workersFinished-.patch | text/plain | 3.0 KB |
| v2-0007-convert-SharedFileSet-refcnt-to-an-atomic.patch | text/plain | 3.0 KB |
| v2-0008-convert-ParallelBlockTableScanDescData-phs_-start.patch | text/plain | 7.0 KB |
| v2-0009-convert-FastPathStrongRelationLocks-to-atomics.patch | text/plain | 6.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | John Naylor | 2026-07-23 20:26:04 | Re: [PATCH] Use ssup_datum_*_cmp for int2, oid, and oid8 sort support |
| Previous Message | Thom Brown | 2026-07-23 19:17:49 | Skip redundant DISTINCT enforcement (setop branches, provably-unique input) |