svn commit: r208332 - in head/sys: amd64/include i386/include

Poul-Henning Kamp phk at phk.freebsd.dk
Sun May 23 12:36:19 UTC 2010


In message <20100523183502.C13588 at delplex.bde.org>, Bruce Evans writes:
>On Thu, 20 May 2010, Poul-Henning Kamp wrote:

><machine/atomic.h> should be kernel-only.

No, it absolutely should not, if anything, it should be pushed as
international standards (with <sys/endian.h>, and <sys/queue.h>).

Atomic operations and write barriers are necessary when multiple
threads or processes cooperate using shared memory. and if anything
the bug is that POSIX has not updated pthreads to have these and
other necessary primitives.  (pthread_mutex_assert_held() for instance).

I agree that FlexeLint is a bit hysterical at times, but when the
issue can be avioded at a "cost" of increased clarity of intention,
I don't see any reason to not do so.

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.


More information about the svn-src-all mailing list