From owner-p4-projects@FreeBSD.ORG  Wed Jul 28 22:57:09 2004
Return-Path: <owner-p4-projects@FreeBSD.ORG>
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id 076F916A4D0; Wed, 28 Jul 2004 22:57:09 +0000 (GMT)
Delivered-To: perforce@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id B091916A4CE; Wed, 28 Jul 2004 22:57:08 +0000 (GMT)
Received: from aslan.scsiguy.com (aslan.scsiguy.com [63.229.232.106])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 1C99243D39; Wed, 28 Jul 2004 22:57:08 +0000 (GMT)
	(envelope-from gibbs@scsiguy.com)
Received: from aslan.scsiguy.com (aslan.scsiguy.com [63.229.232.106])
	by aslan.scsiguy.com (8.12.11/8.12.11) with ESMTP id i6SMv1CU027572;
	Wed, 28 Jul 2004 16:57:01 -0600 (MDT)
	(envelope-from gibbs@scsiguy.com)
Date: Wed, 28 Jul 2004 16:57:01 -0600
From: "Justin T. Gibbs" <gibbs@scsiguy.com>
To: John Baldwin <jhb@freebsd.org>,
	Perforce Change Reviews <perforce@freebsd.org>
Message-ID: <A437589FD438D4483200878E@aslan.scsiguy.com>
In-Reply-To: <200407282050.i6SKo9ci010595@repoman.freebsd.org>
References: <200407282050.i6SKo9ci010595@repoman.freebsd.org>
X-Mailer: Mulberry/3.1.6 (Linux/x86)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Subject: Re: PERFORCE change 58403 for review
X-BeenThere: p4-projects@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
Reply-To: "Justin T. Gibbs" <gibbs@scsiguy.com>
List-Id: p4 projects tree changes <p4-projects.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/p4-projects>
List-Post: <mailto:p4-projects@freebsd.org>
List-Help: <mailto:p4-projects-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 28 Jul 2004 22:57:09 -0000

> 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?

--
Justin