Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 29 Jul 2004 14:01:33 -0400
From:      John Baldwin <jhb@FreeBSD.org>
To:        "Justin T. Gibbs" <gibbs@scsiguy.com>
Cc:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   Re: PERFORCE change 58403 for review
Message-ID:  <200407291401.33538.jhb@FreeBSD.org>
In-Reply-To: <A437589FD438D4483200878E@aslan.scsiguy.com>
References:  <200407282050.i6SKo9ci010595@repoman.freebsd.org> <A437589FD438D4483200878E@aslan.scsiguy.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday 28 July 2004 06:57 pm, Justin T. Gibbs wrote:
> > http://perforce.freebsd.org/chv.cgi?CH=58403
> >
> > Change 58403 by jhb@jhb_slimer on 2004/07/28 20:49:40
> >
> > 	Prompted by some questions on IRC from gibbs@, scottl@, and others:
> > 	atomic_store_rel() on ia32 does not actually need a lock
>
> ...
>
> > Differences ...
> >
> > ==== //depot/projects/smpng/sys/i386/include/atomic.h#13 (text+ko) ====
> >
> > @@ -180,7 +180,7 @@
> >   * XXX: This is _NOT_ safe on a P6 or higher because it does not
> > guarantee * memory ordering.  These should only be used on a 386.
> >   */
>
> Is the above comment still accurate?

For the load_acq, yes.

-- 
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?200407291401.33538.jhb>