Re: Add spin_delay() implementation for Arm in s_lock.h

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Add spin_delay() implementation for Arm in s_lock.h
Дата
Msg-id 3189791.1649286416@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: Add spin_delay() implementation for Arm in s_lock.h  ("Blake, Geoff" <blakgeof@amazon.com>)
Ответы Re: Add spin_delay() implementation for Arm in s_lock.h
Список pgsql-hackers
"Blake, Geoff" <blakgeof@amazon.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

[1] https://jira.mariadb.org/browse/MDEV-25807



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Mark Dilger
Дата:
Сообщение: Re: How about a psql backslash command to show GUCs?
Следующее
От: Andres Freund
Дата:
Сообщение: Re: shared-memory based stats collector - v70