From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40661) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK4wD-0003Iz-Kx for qemu-devel@nongnu.org; Thu, 12 Sep 2013 07:20:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VK4w0-0005Ta-Jk for qemu-devel@nongnu.org; Thu, 12 Sep 2013 07:19:53 -0400 Received: from mail-ee0-x230.google.com ([2a00:1450:4013:c00::230]:60256) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK4w0-0005TK-DM for qemu-devel@nongnu.org; Thu, 12 Sep 2013 07:19:40 -0400 Received: by mail-ee0-f48.google.com with SMTP id l10so5242154eei.35 for ; Thu, 12 Sep 2013 04:19:39 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <5231A353.8070105@redhat.com> Date: Thu, 12 Sep 2013 13:19:47 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1377573663-16727-1-git-send-email-pingfank@linux.vnet.ibm.com> In-Reply-To: <1377573663-16727-1-git-send-email-pingfank@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/4] timers thread-safe stuff List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Liu Ping Fan Cc: Kevin Wolf , Jan Kiszka , Stefan Hajnoczi , qemu-devel@nongnu.org, Alex Bligh Il 27/08/2013 05:20, Liu Ping Fan ha scritto: > Saw the Alex's patches has been merged, rebase mine onto his. > > v3: > 1. rename seqlock_read_check as seqlock_read_retry > 2. Document timerlist were protected by BQL, and discard private lock around "qemu_event_wait(tl->ev)". > > v2: > 1. fix comment in commit and code > 2. fix race issue for qemu_clock_enable(foo,disable) > > > Liu Ping Fan (2): > timer: protect timers_state's clock with seqlock > timer: make qemu_clock_enable sync between disable and timer's cb > > Paolo Bonzini (2): > seqlock: introduce read-write seqlock > qemu-thread: add QemuEvent > > cpus.c | 36 +++++++++++--- > include/qemu/seqlock.h | 72 +++++++++++++++++++++++++++ > include/qemu/thread-posix.h | 8 +++ > include/qemu/thread-win32.h | 4 ++ > include/qemu/thread.h | 7 +++ > include/qemu/timer.h | 4 ++ > qemu-timer.c | 20 +++++++- > util/qemu-thread-posix.c | 116 ++++++++++++++++++++++++++++++++++++++++++++ > util/qemu-thread-win32.c | 26 ++++++++++ > 9 files changed, 286 insertions(+), 7 deletions(-) > create mode 100644 include/qemu/seqlock.h > Stefan, could you pick up these four patches as well? Paolo