Re: [BUGS] BUG #2401: spinlocks not available on amd64 - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: [BUGS] BUG #2401: spinlocks not available on amd64
Date
Msg-id 200604280434.k3S4Y5q20868@candle.pha.pa.us
Whole thread Raw
In response to Re: [BUGS] BUG #2401: spinlocks not available on amd64  (Kris Jurka <books@ejurka.com>)
List pgsql-patches
Kris Jurka wrote:
> Bruce Momjian wrote:
> > Great, changes attached and applied.  I removed the solaris_i386 and
> > solaris_x86_64.s files and made just one solaris_x86.s.  I updated the
> > build system to use the new file, updated the macros, and added some
> > documentation on the approach.  Thanks.
> >
> > Would you test current CVS to make sure it still works properly?  Thanks.
> >
>
> The patch adds an extra else in src/template/solaris that means the
> assembly file is not properly picked up.

Sorry, removed.

> Also the claim that Sun's cc understands C preprocessor doesn't hold
> true for me:
>
> /usr/local/SUNWspro/bin/cc -Xa -v -DSUNOS4_CC -g -c tas.s
> Assembler: tas.s
>          "tas.s", line 9 : Illegal mnemonic
>          "tas.s", line 9 : Syntax error
>          "tas.s", line 9 : Illegal mnemonic
>          "tas.s", line 9 : Illegal mnemonic
>
> This is with cc -V
> cc: Sun WorkShop 6 update 2 C 5.3 Patch 111680-09 2003/05/18

Interesting.  It is probably possible to just run the *.s file through
cpp and compile the result, or just go back to having two files.

--
  Bruce Momjian   http://candle.pha.pa.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: Kris Jurka
Date:
Subject: Re: [BUGS] BUG #2401: spinlocks not available on amd64
Next
From: Bruce Momjian
Date:
Subject: Re: plpython tracebacks