Re: pgsql: Remove --disable-spinlocks.

2024-07-30 Thread Thomas Munro
On Tue, Jul 30, 2024 at 11:09 PM Heikki Linnakangas wrote: > The SpinLockSemaArray field should be completely completely, nut just > the #ifndef guard around it. Looks like I had that right in the first version, but got confused while rebasing over a conflict. Fixed, thanks!

Re: pgsql: Remove --disable-spinlocks.

2024-07-30 Thread Heikki Linnakangas
On 30/07/2024 14:03, Thomas Munro wrote: Remove --disable-spinlocks. Sorry, spotted this only now: --- a/src/backend/postmaster/launch_backend.c +++ b/src/backend/postmaster/launch_backend.c @@ -108,9 +108,7 @@ typedef struct #ifdef USE_INJECTION_POINTS struct InjectionPointsCtl *ActiveI

pgsql: Remove --disable-spinlocks.

2024-07-30 Thread Thomas Munro
Remove --disable-spinlocks. A later change will require atomic support, so it wouldn't make sense for a hypothetical new system not to be able to implement spinlocks. Reviewed-by: Heikki Linnakangas Reviewed-by: Tom Lane (concept, not the patch) Reviewed-by: Andres Freund (concept, not the pat