| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | "Blake, Geoff" <blakgeof(at)amazon(dot)com> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Add spin_delay() implementation for Arm in s_lock.h |
| Date: | 2022-04-06 23:06:56 |
| Message-ID: | 3189791.1649286416@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
"Blake, Geoff" <blakgeof(at)amazon(dot)com> writes:
> Hi Tom, Andres,
> Any additional feedback for this patch?
I did some more research and testing:
* Using a Mac with the M1 Pro chip (marginally beefier than the M1
I was testing on before), I think I can see some benefit in the
test case I proposed upthread. It's marginal though.
* On a Raspberry Pi 3B+, there's no outside-the-noise difference.
* ISB doesn't exist in pre-V7 ARM, so it seems prudent to restrict
the patch to ARM64. I doubt any flavor of ARM32 would be able to
benefit anyway. (Googling finds that MariaDB made this same
choice not long ago [1].)
So what we've got is that there seems to be benefit at high
core counts, and it at least doesn't hurt at lower ones.
That's good enough for me, so pushed.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2022-04-06 23:12:09 | Re: shared-memory based stats collector - v70 |
| Previous Message | Mark Dilger | 2022-04-06 22:36:08 | Re: How about a psql backslash command to show GUCs? |