[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/4] timers thread-safe stuff
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/4] timers thread-safe stuff |
Date: |
Wed, 18 Sep 2013 15:54:42 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Aug 27, 2013 at 11:20:59AM +0800, Liu Ping Fan wrote:
> 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
Ping Fan: Can you send a final version that addresses Alex's request for
documentation?
Otherwise we're ready to go.
Stefan