qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
	Anthony Liguori <aliguori@us.ibm.com>,
	liu ping fan <qemulist@gmail.com>,
	qemu-devel@nongnu.org, Alex Bligh <alex@alex.org.uk>,
	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: Fri, 09 Aug 2013 17:15:29 +0200	[thread overview]
Message-ID: <52050791.60409@redhat.com> (raw)
In-Reply-To: <20130809150919.GB9270@stefanha-thinkpad.redhat.com>

Il 09/08/2013 17:09, Stefan Hajnoczi ha scritto:
>> Perhaps, but do we really need it to work outside AioContexts?  Using a
>> second AioContext for bottom halves has been mentioned for a while.  It
>> would fix issues where a bottom half runs spuriously before QEMU starts,
>> just because something uses qemu_aio_wait().  Extending the same
>> approach to timers is natural.
>>
>> At some point Stefan was planning to add a per-BlockDriverState
>> AioContext.  Once that is done, using two different AioContext for
>> "default block layer context" and "default main loop" context would be
>> very easily done.
> 
> The model I'm going for is that the user can configure how many event
> loop threads to use.  Then they can bind a -drive to a specific event
> loop thread and it will operate in that AioContext.

Yes, this is really what I meant.  I wasn't sure exactly how to describe
it.  In any case, it is not really related to nested event loops.  It
even seems like an extension of what I'm asking Alex to do.  In
principle you could have many AioContexts (plus the one for
qemu_new_timer aka qemu_timer_new) but put all of them in the main iothread.

Paolo

> So it isn't quite a per-BDS AioContext model but perhaps I'll hit
> something which does require per-BDS AioContexts.

  reply	other threads:[~2013-08-09 15:16 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 [this message]
2013-08-10  3:27     ` liu ping fan
2013-08-10  8:38       ` Paolo Bonzini
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=52050791.60409@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).