Re: [PATCH] Add support for TAS/S_UNLOCK for aarch64

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH] Add support for TAS/S_UNLOCK for aarch64
Дата
Msg-id 29866.1370370479@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH] Add support for TAS/S_UNLOCK for aarch64  (Mark Salter <msalter@redhat.com>)
Ответы Re: [PATCH] Add support for TAS/S_UNLOCK for aarch64  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Mark Salter <msalter@redhat.com> writes:
> On Tue, 2013-06-04 at 13:40 -0400, Tom Lane wrote:
>> We got no response to the question of whether it couldn't be merged with
>> the existing ARM32 code block.  I'd prefer not to have essentially
>> duplicate sections in s_lock.h if it's not necessary.

> Of course it could be. I don't think it should be. Aarch64 is a 
> completely new architecture with faint resemblance to 32bit arm.

OK, fair enough.

> I went
> back and read the previous thread concerning the problems with builtin
> gcc atomics with certain tool/hardware combinations. Would it be okay to
> add a default implementation based on gcc builtins to be used if no
> arch-specific definitions exist? If so, I could work up a patch for
> review.

We had pretty much rejected that concept in the previous discussion,
I thought.  In the first place, there is no good reason to assume
that somebody on a nonstandard platform is using gcc, and in the second,
our review of the available info suggested that the implementation
quality of gcc's builtins is ... um ... variable.  Blindly falling back
to that on a platform we haven't tested does not seem very safe.
        regards, tom lane



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

Предыдущее
От: Josh Berkus
Дата:
Сообщение: Re: Configurable location for extension .control files
Следующее
От: Kohei KaiGai
Дата:
Сообщение: RFC: ExecNodeExtender