From owner-cvs-all Mon Oct 9 23:27:23 2000 Delivered-To: cvs-all@freebsd.org Received: from echunga.lemis.com (freebie.lemis.com [192.109.197.137]) by hub.freebsd.org (Postfix) with ESMTP id BE10137B66F; Mon, 9 Oct 2000 23:27:15 -0700 (PDT) Received: (from grog@localhost) by wantadilla.lemis.com (8.11.0/8.9.3) id e973IeQ28433; Sat, 7 Oct 2000 12:48:40 +0930 (CST) (envelope-from grog) Date: Sat, 7 Oct 2000 12:48:40 +0930 From: Greg Lehey To: Bruce Evans Cc: Mike Smith , cvs-all@FreeBSD.ORG, cvs-committers@FreeBSD.ORG Subject: Re: cvs commit: src/sys/alpha/include atomic.h src/sys/i386/incl Message-ID: <20001007124840.A28301@wantadilla.lemis.com> References: <200010060352.e963qPh04244@mass.osd.bsdi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 1.0i In-Reply-To: ; from bde@zeta.org.au on Fri, Oct 06, 2000 at 08:36:03PM +1100 Organization: LEMIS, PO Box 460, Echunga SA 5153, Australia Phone: +61-8-8388-8286 Fax: +61-8-8388-8725 Mobile: +61-418-838-708 WWW-Home-Page: http://www.lemis.com/~grog X-PGP-Fingerprint: 6B 7B C3 8C 61 CD 54 AF 13 24 52 F8 6D A4 95 EF Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Friday, 6 October 2000 at 20:36:03 +1100, Bruce Evans wrote: > On Thu, 5 Oct 2000, Mike Smith wrote: >> [omitted attribution to Bruce Evans] >>> The instruction to do the load is even named "ld" on alphas. I will >>> remove the suffixes on the atomic functions soon. It should go without >>> saying that operations like load and clear, test and set, and compare and >>> swap, etc., are atomic. >> >> It doesn't go without saying, and I think it's important from the >> perspective of general code readability that we make it extremely clear >> that these methods are explicitly atomic. > > Verbose names make code unreadable. Cryptic names make code unintelligible. Agreed, the "atomic_" prefix may have been overdoing it, but at least the intention is clear, and I don't see any reason to change it to something that loses information. Greg -- Finger grog@lemis.com for PGP public key See complete headers for address and phone numbers To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message