diff options
author | nwhitehorn <nwhitehorn@FreeBSD.org> | 2012-05-04 16:00:22 +0000 |
---|---|---|
committer | nwhitehorn <nwhitehorn@FreeBSD.org> | 2012-05-04 16:00:22 +0000 |
commit | 68e9eabdbf1a8b9a1461c0a249d893b2c5223f45 (patch) | |
tree | ecee53979b0f999ca24600814860dd57b6ca0ca3 /sys/powerpc/include/pio.h | |
parent | 4ee7205a6e0fa4716b49b495f88a3f51579b9ab0 (diff) | |
download | FreeBSD-src-68e9eabdbf1a8b9a1461c0a249d893b2c5223f45.zip FreeBSD-src-68e9eabdbf1a8b9a1461c0a249d893b2c5223f45.tar.gz |
Fix final bugs in memory barriers on PowerPC:
- Use isync/lwsync unconditionally for acquire/release. Use of isync
guarantees a complete memory barrier, which is important for serialization
of bus space accesses with mutexes on multi-processor systems.
- Go back to using sync as the I/O memory barrier, which solves the same
problem as above with respect to mutex release using lwsync, while not
penalizing non-I/O operations like a return to sync on the atomic release
operations would.
- Place an acquisition barrier around thread lock acquisition in
cpu_switchin().
Diffstat (limited to 'sys/powerpc/include/pio.h')
-rw-r--r-- | sys/powerpc/include/pio.h | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/sys/powerpc/include/pio.h b/sys/powerpc/include/pio.h index 0994ed1..b09e68a 100644 --- a/sys/powerpc/include/pio.h +++ b/sys/powerpc/include/pio.h @@ -39,7 +39,12 @@ * I/O macros. */ -#define powerpc_iomb() __asm __volatile("eieio" : : : "memory") +/* + * Use sync so that bus space operations cannot sneak out the bottom of + * mutex-protected sections (mutex release does not guarantee completion of + * accesses to caching-inhibited memory on some systems) + */ +#define powerpc_iomb() __asm __volatile("sync" : : : "memory") static __inline void __outb(volatile u_int8_t *a, u_int8_t v) |