From: Jamie Lokier <jamie@shareable.org>
To: qemu-devel@nongnu.org
Cc: kvm-devel@lists.sourceforge.net, Luca Tettamanti <kronos.it@gmail.com>
Subject: Re: [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2
Date: Sun, 19 Aug 2007 14:10:42 +0100 [thread overview]
Message-ID: <20070819131042.GA22798@mail.shareable.org> (raw)
In-Reply-To: <46C7FE32.4050309@qumranet.com>
Avi Kivity wrote:
> >>>In this case the dyn-tick minimum res will be 1msec. I believe it should
> >>>work ok since this is the case without any dyn-tick.
> >>>
> >>Actually minimum resolution depends on host HZ setting, but - yes -
> >>essentially you have the same behaviour of the "unix" timer, plus the
> >>overhead of reprogramming the timer.
> >>
> >
> >Is this significant? At a high guest HZ, this is could be quite a lot
> >of additional syscalls right?
> >
> At HZ=1000, this adds a small multiple of 1000 syscalls, which is a
> fairly small overhead.
Small, but maybe measurable.
That overhead could be removed if the dyn-tick code were to
predictively set the host timer into a repeating mode when guests do
actually require a regular tick.
I'm thinking when it detects it needed a tick a small number of times
in a row, with the same interval, it could set the host timer to
trigger repeatedly at that interval. Then it wouldn't need to reprogram
if that stayed the same (except maybe to correct for drift?)
If a tick then _wasn't_ required for that interval (i.e. it was
required for less, more, or not at all), then it would have to
reprogram the host timer. If it really mattered, it wouldn't have to
reprogram the host timer when the next required tick is further in the
future or not at all; it would simply be a redundant SIGALRM. In
weird cases that's worthwhile, but I suspect it generally isn't.
-- Jamie
next prev parent reply other threads:[~2007-08-19 13:10 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-17 23:11 [Qemu-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take 2 Luca Tettamanti
2007-08-17 23:11 ` [Qemu-devel] [PATCH 1/4] Rework alarm timer infrastrucure Luca Tettamanti
2007-08-17 23:11 ` [Qemu-devel] [PATCH 2/4] Add -clock option Luca Tettamanti
2007-08-17 23:11 ` [Qemu-devel] [PATCH 3/4] Add support for HPET periodic timer Luca Tettamanti
2007-08-21 19:24 ` Matthew Kent
2007-08-21 19:40 ` Luca
2007-08-21 20:15 ` Matthew Kent
2007-08-22 6:48 ` [kvm-devel] " Dan Kenigsberg
2007-08-22 7:03 ` Avi Kivity
2007-08-22 12:34 ` Andi Kleen
2007-08-22 21:11 ` Dan Kenigsberg
2007-08-22 22:09 ` Andi Kleen
2007-08-23 7:02 ` Dan Kenigsberg
2007-08-24 20:18 ` Luca
2007-08-25 8:24 ` Dan Kenigsberg
2007-09-03 8:40 ` GUERRAZ Francois
2007-08-17 23:11 ` [Qemu-devel] [PATCH 4/4] Add support for dynamic ticks Luca Tettamanti
2007-08-17 23:48 ` [Qemu-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take 2 Christian MICHON
2007-08-18 0:10 ` [kvm-devel] " Luca
2007-08-18 15:17 ` [Qemu-devel] Re: [kvm-devel] " Anthony Liguori
2007-08-18 16:53 ` [Qemu-devel] RE: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2 Dor Laor
2007-08-18 22:02 ` [Qemu-devel] " Luca Tettamanti
2007-08-18 23:58 ` Anthony Liguori
2007-08-19 7:36 ` [Qemu-devel] RE: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure -take2 Dor Laor
2007-08-19 8:24 ` [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2 Avi Kivity
2007-08-19 13:10 ` Jamie Lokier [this message]
2007-08-19 13:48 ` [kvm-devel] [Qemu-devel] " Avi Kivity
2007-08-19 13:57 ` Paul Brook
2007-08-19 14:07 ` Avi Kivity
2007-08-19 14:27 ` Dor Laor
2007-08-20 9:25 ` Avi Kivity
2007-08-19 17:15 ` Jamie Lokier
2007-08-19 19:29 ` [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarmtimer " Dor Laor
2007-08-19 19:30 ` [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer " Avi Kivity
2007-08-19 16:52 ` [Qemu-devel] Re: [kvm-devel] " Luca
2007-08-19 19:31 ` Avi Kivity
2007-08-20 21:20 ` Luca Tettamanti
2007-08-20 21:55 ` malc
2007-08-20 22:49 ` [kvm-devel] [Qemu-devel] " Luca
2007-08-21 12:09 ` [Qemu-devel] Re: [kvm-devel] " Avi Kivity
2007-08-21 19:38 ` Luca Tettamanti
2007-08-21 19:44 ` malc
2007-08-22 5:02 ` Avi Kivity
2007-08-22 16:12 ` Luca Tettamanti
2007-08-22 16:21 ` Avi Kivity
2007-08-22 16:38 ` Luca
2007-08-22 16:45 ` Avi Kivity
2007-08-22 17:23 ` Luca
2007-08-22 17:39 ` Luca
2007-08-22 19:21 ` Luca
2007-08-22 21:35 ` [Qemu-devel] " Dor Laor
2007-08-22 22:07 ` [Qemu-devel] " Luca
2007-08-22 20:42 ` Dan Kenigsberg
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=20070819131042.GA22798@mail.shareable.org \
--to=jamie@shareable.org \
--cc=kronos.it@gmail.com \
--cc=kvm-devel@lists.sourceforge.net \
--cc=qemu-devel@nongnu.org \
/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).