From owner-cvs-src@FreeBSD.ORG Fri May 28 01:39:31 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D5A3216A4CE; Fri, 28 May 2004 01:39:31 -0700 (PDT) Received: from mail.pcnet.com (mail.pcnet.com [204.213.232.4]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9328743D39; Fri, 28 May 2004 01:39:31 -0700 (PDT) (envelope-from eischen@vigrid.com) Received: from mail.pcnet.com (mail.pcnet.com [204.213.232.4]) by mail.pcnet.com (8.12.10/8.12.1) with ESMTP id i4S8cU8Q016311; Fri, 28 May 2004 04:38:30 -0400 (EDT) Date: Fri, 28 May 2004 04:38:30 -0400 (EDT) From: Daniel Eischen X-Sender: eischen@pcnet5.pcnet.com To: Mike Makonnen In-Reply-To: <20040528082228.GA997@rogue.acs-et.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: Marius Strobl Subject: Re: cvs commit: src/sys/sparc64/include atomic.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 28 May 2004 08:39:32 -0000 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. will try to use cli instead. -- Dan Eischen