diff options
| author | Emilio G. Cota | 2016-05-24 22:06:13 +0200 |
|---|---|---|
| committer | Paolo Bonzini | 2016-05-29 09:11:11 +0200 |
| commit | c983895258a771f8a5e4a53950bfb7fd2216651c (patch) | |
| tree | b383a83403d84c2d8247143783f3f700772dcaf8 /include/exec | |
| parent | docs/atomics: update atomic_read/set comparison with Linux (diff) | |
| download | qemu-c983895258a771f8a5e4a53950bfb7fd2216651c.tar.gz qemu-c983895258a771f8a5e4a53950bfb7fd2216651c.tar.xz qemu-c983895258a771f8a5e4a53950bfb7fd2216651c.zip | |
atomics: emit an smp_read_barrier_depends() barrier only for Alpha and Thread Sanitizer
For correctness, smp_read_barrier_depends() is only required to
emit a barrier on Alpha hosts. However, we are currently emitting
a consume fence unconditionally, and most compilers currently treat
consume and acquire fences as equivalent.
Fix it by keeping the consume fence if we're compiling with Thread
Sanitizer, since this might help prevent false warnings. Otherwise,
only emit the barrier for Alpha hosts. Note that we still guarantee
that smp_read_barrier_depends() is a compiler barrier.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Message-Id: <1464120374-8950-3-git-send-email-cota@braap.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'include/exec')
0 files changed, 0 insertions, 0 deletions