Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 28 Mar 2006 10:32:07 -0500
From:      John Baldwin <jhb@freebsd.org>
To:        Dag-Erling Smorgrav <des@freebsd.org>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/i386/include atomic.h
Message-ID:  <200603281032.10624.jhb@freebsd.org>
In-Reply-To: <200603281434.k2SEYmaR031447@repoman.freebsd.org>
References:  <200603281434.k2SEYmaR031447@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday 28 March 2006 09:34, Dag-Erling Smorgrav wrote:
> des         2006-03-28 14:34:48 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/i386/include     atomic.h 
>   Log:
>   Use wrapper macros for atomic pointer operations in order to perform the
>   correct casts.  This should probably be merged to other architectures.

It should really be uintptr_t rather than u_int.  You can merge it to other
arch's if you wish.  You'd complicate sparc64's atomic.h, but not too much.
It would be nice if you could take the time to make changes to all archs at
once in the future.

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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200603281032.10624.jhb>