PERFORCE change 58403 for review

John Baldwin jhb at FreeBSD.org
Thu Jul 29 11:01:40 PDT 2004


On Wednesday 28 July 2004 06:57 pm, Justin T. Gibbs wrote:
> > http://perforce.freebsd.org/chv.cgi?CH=58403
> >
> > Change 58403 by jhb at 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 at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org


More information about the p4-projects mailing list