[1/2] atomics: do not use __atomic primitives for RCU atomics

Commit Message

Emilio Cota May 24, 2016, 6:42 p.m. UTC

On Sun, May 22, 2016 at 08:58:51 +0100, Alex Bennée wrote:
> For tsan runs you need to re-build with:
> 
>   ./configure --cc=gcc --extra-cflags="-pie -fPIE -fsanitize=thread" --with-coroutine=gthread
> 
> Specifically the coroutine ucontext messing really confuses TSAN.

With your configure args + the appended I can at least compile
arm-softmmu, but I still get:

FATAL: ThreadSanitizer: unexpected memory mapping 0x55d9f389d000-0x55d9f4111000

Am I missing something? Thanks,

		Emilio

Patch

diff --git a/configure b/configure
index ab54f3c..a8903fe 100755
--- a/configure
+++ b/configure
@@ -112,7 +112,7 @@  compile_object() {

 compile_prog() {
   local_cflags="$1"
-  local_ldflags="$2"
+  local_ldflags="$2 -ltsan"
   do_cc $QEMU_CFLAGS $local_cflags -o $TMPE $TMPC $LDFLAGS $local_ldflags
 }

@@ -174,12 +174,12 @@  audio_drv_list=""
 block_drv_rw_whitelist=""
 block_drv_ro_whitelist=""
 host_cc="cc"
-libs_softmmu=""
-libs_tools=""
+libs_softmmu="-ltsan"
+libs_tools="-ltsan"
 audio_pt_int=""
 audio_win_int=""
 cc_i386=i386-pc-linux-gnu-gcc
-libs_qga=""
+libs_qga="-ltsan"
 debug_info="yes"
 stack_protector=""