From: Paolo Bonzini <pbonzini@redhat.com>
To: liu ping fan <qemulist@gmail.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
Anthony Liguori <aliguori@us.ibm.com>,
Alex Bligh <alex@alex.org.uk>,
qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@redhat.com>,
MORITA Kazutaka <morita.kazutaka@lab.ntt.co.jp>,
rth@twiddle.net
Subject: Re: [Qemu-devel] [RFC] [PATCHv8 09/30] aio / timers: Add QEMUTimerListGroup and helper functions
Date: Sat, 10 Aug 2013 10:38:12 +0200 [thread overview]
Message-ID: <5205FBF4.9070902@redhat.com> (raw)
In-Reply-To: <CAJnKYQkjppFn=aFBU=NPzL9PuRnAaj-VXGhTKq=K6LcWhjhSVg@mail.gmail.com>
Il 10/08/2013 05:27, liu ping fan ha scritto:
>> > ... except if we get rid of TimerListGroup completely and put it in
>> > AioContext. To do so, we can have _two_ AioContexts in main-loop.c.
>> > One is for the block layer, the other is only used for timers. Later we
> Why can the AioContexts of block layer NOT hold timers? For the
> reason mainloop's iohandler runs after it? Or any reason raised by
> blocklayer itself?
The point of this series is to make the AioContexts of the block layer
able to hold timers.
At the same time, however, the main loop's timers cannot run during a
qemu_aio_wait(), as that would be backwards-incompatible. So I'm
proposing to have two AioContext, and in the future separate
timers-for-the-block-layer from timers-for-everything-else.
Paolo
next prev parent reply other threads:[~2013-08-10 8:39 UTC|newest]
Thread overview: 91+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-08 21:41 [Qemu-devel] [RFC] [PATCHv8 00/30] aio / timers: Add AioContext timers and use ppoll Alex Bligh
2013-08-08 21:41 ` [Qemu-devel] [RFC] [PATCHv8 01/30] aio / timers: Rename qemu_new_clock and expose clock types Alex Bligh
2013-08-08 21:41 ` [Qemu-devel] [RFC] [PATCHv8 02/30] aio / timers: add qemu-timer.c utility functions Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 03/30] aio / timers: Consistent treatment of disabled clocks for deadlines Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 04/30] aio / timers: add ppoll support with qemu_poll_ns Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 05/30] aio / timers: Add prctl(PR_SET_TIMERSLACK, 1, ...) to reduce timer slack Alex Bligh
2013-08-09 8:53 ` Stefan Hajnoczi
2013-08-09 14:10 ` Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 06/30] aio / timers: Make qemu_run_timers and qemu_run_all_timers return progress Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 07/30] aio / timers: Split QEMUClock into QEMUClock and QEMUTimerList Alex Bligh
2013-08-09 10:03 ` Paolo Bonzini
2013-08-09 14:23 ` Alex Bligh
2013-08-09 14:28 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 08/30] aio / timers: Untangle include files Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 09/30] aio / timers: Add QEMUTimerListGroup and helper functions Alex Bligh
2013-08-09 10:02 ` Paolo Bonzini
2013-08-09 14:27 ` Alex Bligh
2013-08-09 14:35 ` Paolo Bonzini
2013-08-09 14:47 ` Alex Bligh
2013-08-09 14:59 ` Paolo Bonzini
2013-08-10 11:05 ` Alex Bligh
2013-08-11 7:53 ` Paolo Bonzini
2013-08-11 8:29 ` Alex Bligh
2013-08-12 17:27 ` Paolo Bonzini
2013-08-12 17:32 ` Alex Bligh
2013-08-12 6:53 ` Wenchao Xia
2013-08-12 7:08 ` Alex Bligh
2013-08-09 15:09 ` Stefan Hajnoczi
2013-08-09 15:15 ` Paolo Bonzini
2013-08-10 3:27 ` liu ping fan
2013-08-10 8:38 ` Paolo Bonzini [this message]
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 10/30] aio / timers: Add QEMUTimerListGroup to AioContext Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 11/30] aio / timers: Add a notify callback to QEMUTimerList Alex Bligh
2013-08-09 9:02 ` Stefan Hajnoczi
2013-08-09 14:16 ` Alex Bligh
2013-08-09 10:03 ` Paolo Bonzini
2013-08-09 14:28 ` Alex Bligh
2013-08-10 11:13 ` Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 12/30] aio / timers: aio_ctx_prepare sets timeout from AioContext timers Alex Bligh
2013-08-09 10:04 ` Paolo Bonzini
2013-08-09 14:30 ` Alex Bligh
2013-08-09 14:41 ` Paolo Bonzini
2013-08-09 14:55 ` Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 13/30] aio / timers: Add aio_timer_new wrapper Alex Bligh
2013-08-09 10:06 ` Paolo Bonzini
2013-08-09 14:32 ` Alex Bligh
2013-08-09 14:36 ` Paolo Bonzini
2013-08-09 14:51 ` Alex Bligh
2013-08-09 14:51 ` Paolo Bonzini
2013-08-09 14:57 ` Alex Bligh
2013-08-09 14:58 ` Paolo Bonzini
2013-08-09 22:57 ` Alex Bligh
2013-08-10 8:36 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 14/30] aio / timers: Convert aio_poll to use AioContext timers' deadline Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 15/30] aio / timers: Convert mainloop to use timeout Alex Bligh
2013-08-09 10:07 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 16/30] aio / timers: On timer modification, qemu_notify or aio_notify Alex Bligh
2013-08-09 10:08 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 17/30] aio / timers: Introduce new API qemu_timer_new and friends Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 18/30] aio / timers: Use all timerlists in icount warp calculations Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 19/30] aio / timers: Add documentation and new format calls Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 20/30] aio / timers: Remove alarm timers Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 21/30] aio / timers: Remove legacy qemu_clock_deadline & qemu_timerlist_deadline Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 22/30] aio / timers: Add qemu_clock_get_ms and qemu_clock_get_ms Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 23/30] aio / timers: Rearrange timer.h & make legacy functions call non-legacy Alex Bligh
2013-08-08 22:12 ` Alex Bligh
2013-08-09 9:23 ` Stefan Hajnoczi
2013-08-09 14:18 ` Alex Bligh
2013-08-09 14:26 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 24/30] aio / timers: Remove main_loop_timerlist Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 25/30] aio / timers: Convert rtc_clock to be a QEMUClockType Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 26/30] aio / timers: convert block_job_sleep_ns and co_sleep_ns to new API Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 27/30] aio / timers: Add test harness for AioContext timers Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 28/30] aio / timers: Add scripts/switch-timer-api Alex Bligh
2013-08-09 10:23 ` Paolo Bonzini
2013-08-09 14:34 ` Alex Bligh
2013-08-09 10:26 ` Paolo Bonzini
2013-08-09 14:35 ` Alex Bligh
2013-08-09 14:37 ` Paolo Bonzini
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 29/30] aio / timers: Switch entire codebase to the new timer API Alex Bligh
2013-08-08 21:42 ` [Qemu-devel] [RFC] [PATCHv8 30/30] aio / timers: Remove legacy interface Alex Bligh
2013-08-08 21:46 ` [Qemu-devel] [RFC] [PATCHv8 00/30] aio / timers: Add AioContext timers and use ppoll Alex Bligh
2013-08-09 8:12 ` Jan Kiszka
2013-08-09 8:24 ` liu ping fan
2013-08-09 8:33 ` Jan Kiszka
2013-08-09 10:35 ` Alex Bligh
2013-08-09 9:41 ` Stefan Hajnoczi
2013-08-09 14:19 ` Alex Bligh
2013-08-09 14:27 ` Paolo Bonzini
2013-08-09 10:27 ` Paolo Bonzini
2013-08-09 11:18 ` Stefan Hajnoczi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5205FBF4.9070902@redhat.com \
--to=pbonzini@redhat.com \
--cc=alex@alex.org.uk \
--cc=aliguori@us.ibm.com \
--cc=kwolf@redhat.com \
--cc=morita.kazutaka@lab.ntt.co.jp \
--cc=qemu-devel@nongnu.org \
--cc=qemulist@gmail.com \
--cc=rth@twiddle.net \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).