From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35698) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0461-0001Rz-1Y for qemu-devel@nongnu.org; Fri, 19 Jul 2013 02:23:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V0460-0004xN-2L for qemu-devel@nongnu.org; Fri, 19 Jul 2013 02:23:16 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14176) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V045z-0004xC-Q1 for qemu-devel@nongnu.org; Fri, 19 Jul 2013 02:23:16 -0400 Message-ID: <51E8DB3D.8010805@redhat.com> Date: Fri, 19 Jul 2013 08:22:53 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <51E4E54A.10908@redhat.com> <51E4F77C.2090509@redhat.com> <794E19D97CCC267CCBFA8397@Ximines.local> <51E56A1A.50502@redhat.com> <19631228D7B62545DC6A2928@Ximines.local> <51E57AF3.1050409@redhat.com> <20130717030230.GA27807@stefanha-thinkpad.redhat.com> <51A6D731C894ECC429EA2171@nimrod.local> <20130719015850.GA29671@stefanha-thinkpad.redhat.com> In-Reply-To: <20130719015850.GA29671@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] [RFC] aio/async: Add timed bottom-halves List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Kevin Wolf , Anthony Liguori , Stefan Hajnoczi , qemu-devel@nongnu.org, Alex Bligh , rth@twiddle.net Il 19/07/2013 03:58, Stefan Hajnoczi ha scritto: >> Options: >> >> a) restore alarm timers (at least for the time being). Make all >> alarm timers do qemu_notify_event. However, only run the AioContext >> clock's timers within aio_poll. This is the least intrusive change. >> >> b) calculate the timeout in aio_poll with respect to the minimum >> deadline across all clocks, not just AioContext's clock. Use the >> same logic in mainloop. >> >> I'd go for (b), except for the millisecond accuracy thing. So my >> temptation (sadly) is (a). > > I think this is a non-issue because host_alarm_handler() can only be > called from the main loop: > > main-loop.c:qemu_signal_init() sets up signalfd to monitor SIGALRM. > Therefore we do not asynchronously invoke the SIGALRM signals handler. > It is only invoked from main-loop.c:sigfd_handler() when the main loop > runs. > > That's how I read the code. I haven't checked a running process to be > sure. I think you're right. It was not strictly needed even with alarm timers, because host_alarm_handler() is called always before qemu_run_all_timers. But it made the code more robust. With your change to delete alarm timers and move the deadline to poll, the next poll call will have a timeout of 0 and all will be well. As to millisecond accuracy, as discussed earlier we can use ppoll on Linux. This of course should be introduced before alarm timers are deleted, to avoid breaking bisection. Paolo >> 2. If we do delete alarm timers, I'll need to delete the -clock option. > > I noticed this too because I think we should stub it out for > compatibility. Accept existing options but ignore them, update > documentation to state that they are kept for compatibility. > > Stefan >