From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54204) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bxdlu-0003H4-J7 for qemu-devel@nongnu.org; Fri, 21 Oct 2016 13:38:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bxdlr-0000FY-G5 for qemu-devel@nongnu.org; Fri, 21 Oct 2016 13:38:22 -0400 Received: from mail-wm0-x235.google.com ([2a00:1450:400c:c09::235]:34931) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1bxdlr-0000Ed-7R for qemu-devel@nongnu.org; Fri, 21 Oct 2016 13:38:19 -0400 Received: by mail-wm0-x235.google.com with SMTP id c78so7892368wme.0 for ; Fri, 21 Oct 2016 10:38:16 -0700 (PDT) References: <1476107947-31430-1-git-send-email-pbonzini@redhat.com> From: Alex =?utf-8?Q?Benn=C3=A9e?= In-reply-to: <1476107947-31430-1-git-send-email-pbonzini@redhat.com> Date: Fri, 21 Oct 2016 18:38:13 +0100 Message-ID: <877f91fvqy.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 0/5] More thread sanitizer fixes and atomic.h improvements List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org, cota@braap.org Paolo Bonzini writes: > See each patch. My attempt at fixing whatever I did when I obviously > didn't know enough^W about the C11 memory model, and at setting a > better example for future generations... Are you posting a v2 of this? Alternatively I can slurp them into my next patch set as I've got a whole bunch more tsan related fixes that are not directly MTTCG related currently in my MTTCG tree: https://github.com/stsquad/qemu/commits/mttcg/base-patches-v5 > > Paolo Bonzini (5): > atomic: introduce smp_mb_acquire and smp_mb_release > cpus: use atomic_read to read seqlock-protected variables > qemu-thread: use acquire/release to clarify semantics of QemuEvent > rcu: simplify memory barriers > atomic: base mb_read/mb_set on load-acquire and store-release > > cpus.c | 9 +-- > docs/atomics.txt | 84 ++++++++++++++++------------ > include/qemu/atomic.h | 139 ++++++++++++++++++++--------------------------- > util/qemu-thread-posix.c | 15 ++++- > util/qemu-thread-win32.c | 15 ++++- > util/rcu.c | 11 ++-- > 6 files changed, 143 insertions(+), 130 deletions(-) -- Alex Bennée