From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59041) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK7mN-0001Yh-Sy for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:22:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VK7mH-0002E8-Sx for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:21:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:16056) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK7mH-0002Dz-K8 for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:21:49 -0400 Date: Thu, 12 Sep 2013 16:21:37 +0200 From: Stefan Hajnoczi Message-ID: <20130912142137.GH18070@stefanha-thinkpad.redhat.com> References: <1377573663-16727-1-git-send-email-pingfank@linux.vnet.ibm.com> <5231A353.8070105@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5231A353.8070105@redhat.com> 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: Paolo Bonzini Cc: Kevin Wolf , Jan Kiszka , Liu Ping Fan , Alex Bligh , qemu-devel@nongnu.org On Thu, Sep 12, 2013 at 01:19:47PM +0200, Paolo Bonzini wrote: > 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? This week Kevin is merging patches into the block tree. If it's still on the list next week I'll pick these patches up. Stefan