From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:36903)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1V3jes-0007sm-5q
	for qemu-devel@nongnu.org; Mon, 29 Jul 2013 05:22:34 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1V3jej-0005w9-L2
	for qemu-devel@nongnu.org; Mon, 29 Jul 2013 05:22:26 -0400
Received: from mail-wg0-x22d.google.com ([2a00:1450:400c:c00::22d]:54470)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1V3jej-0005w4-F9
	for qemu-devel@nongnu.org; Mon, 29 Jul 2013 05:22:17 -0400
Received: by mail-wg0-f45.google.com with SMTP id x12so4704491wgg.12
	for <qemu-devel@nongnu.org>; Mon, 29 Jul 2013 02:22:16 -0700 (PDT)
Date: Mon, 29 Jul 2013 11:22:13 +0200
From: Stefan Hajnoczi <stefanha@gmail.com>
Message-ID: <20130729092213.GA1585@stefanha-thinkpad.redhat.com>
References: <1375067768-11342-1-git-send-email-pingfank@linux.vnet.ibm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1375067768-11342-1-git-send-email-pingfank@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [RFC v2 0/5] arm AioContext with its own timer
	stuff
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Liu Ping Fan <qemulist@gmail.com>
Cc: Kevin Wolf <kwolf@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Jan Kiszka <jan.kiszka@siemens.com>, qemu-devel@nongnu.org, Alex Bligh <alex@alex.org.uk>, Anthony Liguori <anthony@codemonkey.ws>, Paolo Bonzini <pbonzini@redhat.com>

On Mon, Jul 29, 2013 at 11:16:03AM +0800, Liu Ping Fan wrote:
> summary of the model:
>   Three qemu-wide clock source allowed in system. And each AioContext has
> three corresponding timer list to run timer against clocksource.
> 
> rfcv2:
>    drop patches about alarm-timer(if timeout of poll will not satisfy, will come back to it)
>    fix qemu_clock_enable sync problem (Thanks for Jan and Stefan)
>    fix process=true when aio_poll run timers (Thanks for Alex)
> 
> Liu Ping Fan (5):
>   timer: protect timers_state with lock
>   timer: pick out timer list info from QemuClock
>   timer: make qemu_clock_enable sync between disable and timer's cb
>   timer: associate three timerlists with AioContext
>   timer: run timers on aio_poll
> 
>  aio-posix.c          |   2 +
>  async.c              |   9 +++
>  cpus.c               |  26 ++++++--
>  include/block/aio.h  |  13 ++++
>  include/qemu/timer.h |  24 ++++++-
>  main-loop.c          |   2 -
>  qemu-timer.c         | 184 ++++++++++++++++++++++++++++++++++++---------------
>  7 files changed, 198 insertions(+), 62 deletions(-)

The potential for overlap with Alex Bligh's series is large.  Can you
base your patches on his v4?

It seems the difference is that you make clock sources to be available
globally while Alex's series uses rt_clock (no synchronization
necessary).

Stefan