cvs commit: src/sys/sparc64/include atomic.h

John Baldwin jhb at FreeBSD.org
Fri May 28 10:40:43 PDT 2004


On Friday 28 May 2004 04:38 am, Daniel Eischen wrote:
> On Fri, 28 May 2004, Mike Makonnen wrote:
> > On Sat, May 22, 2004 at 02:58:06AM +0200, Marius Strobl wrote:
> > > On Fri, May 21, 2004 at 05:52:16PM -0700, Marius Strobl wrote:
> > > > marius      2004/05/21 17:52:16 PDT
> > > >
> > > >   FreeBSD src repository
> > > >
> > > >   Modified files:
> > > >     sys/sparc64/include  atomic.h
> > > >   Log:
> > > >   Use unsigned types for the arguments of the atomic(9) operations,
> > > >   like described in the man page and done on all other architectures.
> > >
> > > This fixes the current build problems of libthr on sparc64 but the
> > > libthr code shouldn't rely on the integer-type of enums.
> >
> > Thanks. I'll change those enums to integers soon (others have suggested
> > it as well). It's just that I find them so convenient when debugging.
>
> And the use of atomic_cmpset_foo() isn't going to work on
> 80386 since it doesn't have cmpxchgl.  <machine/atomic.h>
> will try to use cli instead.

I'd say just punt and suggest that 80386 machines use libc_r.

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org


More information about the cvs-all mailing list