Date: Sat, 7 Oct 2000 12:48:40 +0930 From: Greg Lehey <grog@lemis.com> To: Bruce Evans <bde@zeta.org.au> Cc: Mike Smith <msmith@FreeBSD.ORG>, 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> In-Reply-To: <Pine.BSF.4.21.0010062021030.2593-100000@besplex.bde.org>; from bde@zeta.org.au on Fri, Oct 06, 2000 at 08:36:03PM %2B1100 References: <200010060352.e963qPh04244@mass.osd.bsdi.com> <Pine.BSF.4.21.0010062021030.2593-100000@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20001007124840.A28301>