qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange@redhat.com>
To: Avi Kivity <avi@qumranet.com>
Cc: kvm-devel@lists.sourceforge.net,
	Luca Tettamanti <kronos.it@gmail.com>,
	qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [kvm-devel] [PATCH] Add support for HPET periodic timer.
Date: Mon, 13 Aug 2007 12:50:30 +0100	[thread overview]
Message-ID: <20070813115030.GB14348@redhat.com> (raw)
In-Reply-To: <46C0109E.3090800@qumranet.com>

On Mon, Aug 13, 2007 at 11:04:46AM +0300, Avi Kivity wrote:
> Luca Tettamanti wrote:
> > Linux operates the HPET timer in legacy replacement mode, which means that the
> > periodic interrupt of the CMOS RTC is not delivered (qemu won't be able to use
> > /dev/rtc). Add support for HPET (/dev/hpet) as a replacement for the RTC; the
> > periodic interrupt is delivered via SIGIO and is handled in the same way as the
> > RTC timer.
> > HPET must be explicitly enabled with -use-hpet.
> >
> >   
> 
> Are there any downsides to using HPET?  If not, I suggest making it the 
> default, in order to reduce complexity for the user.

Only downside is that not all machines have HPET, and many which do have it
disabled in BIOS for 'compatability' with Windows, but if you always fallback
to trying RTC too, that's not an issue.

> Something like:
> 
> - try to use HPET (unless -no-rtc selected)
> - try to use RTC  (unless -no-rtc selected)
> - fallback to normal unix facilities

If we're going to add command line args it probably makes sense  to be a
little more generic than -no-rtc or -use-hpet. Have a list of preferred
clock sources eg

  -clock hpet,rtc,unix

If -clock is omitted, then default to trying all in the priority you
describe.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

  reply	other threads:[~2007-08-13 11:51 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-10 20:32 [Qemu-devel] [PATCH] Add support for HPET periodic timer Luca Tettamanti
2007-08-13  8:04 ` [Qemu-devel] Re: [kvm-devel] " Avi Kivity
2007-08-13 11:50   ` Daniel P. Berrange [this message]
2007-08-13 17:03     ` Luca

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=20070813115030.GB14348@redhat.com \
    --to=berrange@redhat.com \
    --cc=avi@qumranet.com \
    --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).