Re: postgres has no spinlock support on riscv rv64imafdc

Поиск
Список
Период
Сортировка
От Merlin Moncure
Тема Re: postgres has no spinlock support on riscv rv64imafdc
Дата
Msg-id CAHyXU0zwtb_79=-BPSF5CTLET1cYYBX=EUsNqGDGUWo-6SXdVw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: postgres has no spinlock support on riscv rv64imafdc  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: postgres has no spinlock support on riscv rv64imafdc  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-bugs
On Saturday, October 19, 2019, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Andres Freund <andres@anarazel.de> writes:
> On 2019-10-18 09:00:23 +0200, Tom Lane wrote:
>> TBH, though, my preference would be for some assembly code rather than
>> relying on GCC builtins as Richard's patch did.

> -1. I think there's good arguments for using inline assembly on
> platforms where we've done so historically, and where we have to support
> old compilers without good support for intrinsics/builtins. But I see
> very little reason for adding more such cases for newer platforms -
> doing so correctly and efficiently is substantial work and fragile.

The reason I'm skeptical of that line of argument is that gcc's track
record for support of these intrinsics on non-mainstream architectures
is just sucky.  Now maybe, somebody was careful and it all works great
on RISC-V.  But IMO, the burden of proof is to show that the intrinsics
work, not to show that they don't.

I recall Noah's recent argument in a related context that with an
asm implementation, anybody with a copy of the architecture manual
can review/verify the code; and such a verification doesn't depend
on which compiler version you're using.  If we depend on gcc intrinsics,
we've basically got zero confidence about anything except from testing.

Do our regression tests exercise the quality of the intrinsic implementation? 

merlin

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

Предыдущее
От: Jeff Janes
Дата:
Сообщение: Re: BUG #16162: create index using gist_trgm_ops leads to panic
Следующее
От: PG Bug reporting form
Дата:
Сообщение: BUG #16165: Query Works fine but spam so many LOG: unrecognized win32 error code: 38