All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Lutomirski <luto@amacapital.net>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>,
	Owen Hofmann <osh@google.com>, KVM General <kvm@vger.kernel.org>,
	Peter Hornyack <peterhornyack@google.com>,
	Joao Martins <joao.m.martins@oracle.com>
Subject: Re: What time is it kvm-clock?
Date: Wed, 24 Feb 2016 11:52:10 -0800	[thread overview]
Message-ID: <CALCETrXWWskniE1X2hdgqxidYZMVhYstyjrKDJwVHYW0M0WaOw@mail.gmail.com> (raw)
In-Reply-To: <56CE082B.3020102@redhat.com>

On Wed, Feb 24, 2016 at 11:44 AM, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
>
> On 24/02/2016 20:38, Andy Lutomirski wrote:
>> If the master clock accurately exposed CLOCK_MONOTONIC_RAW or
>> CLOCK_MONOTONIC (I much prefer the latter), then it would be fine
>> across suspend/resume.
>
> Here we already have a conflict... Owen says he prefers the master clock
> to expose the (stable) TSC, you say you prefer CLOCK_MONOTONIC.
>
> I for one _thought_ CLOCK_MONOTONIC would have been my choice, but I'm
> not so sure about it and I'm also not sure it's possible to do it
> efficiently.

I think it should be configurable.

>  The mult/shift/offset tuple potentially can change every
> tick, and it would be bad to do such an update #vms times per tick (or
> worse, #vcpus times per tick).

If the interface were sane, it would be a single update per tick that
would update a data structure that all guests would share.  Arguably
we ought to expose all three useful clocks (MONOTONIC, MONOTONIC_RAW,
and REALTIME) in the same page.  The vclock code already does this for
MONOTONIC and REALTIME.)

Each vm would have a *different* page that had information like the
guest-boot-to-monotonic offset.

>
> Paolo



-- 
Andy Lutomirski
AMA Capital Management, LLC

  reply	other threads:[~2016-02-24 19:52 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-24  2:31 What time is it kvm-clock? Owen Hofmann
2016-02-24  3:57 ` Marcelo Tosatti
2016-02-24 17:35   ` Peter Hornyack
2016-02-24 20:17     ` Radim Krčmář
2016-02-24 20:24       ` Andy Lutomirski
2016-02-24 20:53         ` Radim Krčmář
2016-02-25 11:13           ` Radim Krčmář
2016-02-25 11:22           ` Marcelo Tosatti
2016-02-24 23:35     ` Marcelo Tosatti
2016-02-24 23:36       ` Marcelo Tosatti
2016-02-25  1:19       ` Andy Lutomirski
2016-02-25  3:50         ` Owen Hofmann
2016-02-25 12:20           ` Radim Krčmář
2016-02-26 17:02             ` Andy Lutomirski
2016-02-26 19:30               ` Marcelo Tosatti
2016-02-27  0:00                 ` Andy Lutomirski
2016-02-25 11:36         ` Radim Krčmář
2016-02-25 12:12         ` Marcelo Tosatti
2016-02-24  3:59 ` Marcelo Tosatti
2016-02-24 14:14 ` Paolo Bonzini
2016-02-24 16:44   ` Andy Lutomirski
2016-02-24 17:38     ` Marcelo Tosatti
2016-02-24 19:38       ` Andy Lutomirski
2016-02-24 19:44         ` Paolo Bonzini
2016-02-24 19:52           ` Andy Lutomirski [this message]
2016-02-24 19:55         ` Owen Hofmann
2016-02-25 12:22           ` Joao Martins
2016-02-25 12:22           ` Joao Martins
2016-02-26 15:04 ` Marcelo Tosatti

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=CALCETrXWWskniE1X2hdgqxidYZMVhYstyjrKDJwVHYW0M0WaOw@mail.gmail.com \
    --to=luto@amacapital.net \
    --cc=joao.m.martins@oracle.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    --cc=osh@google.com \
    --cc=pbonzini@redhat.com \
    --cc=peterhornyack@google.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.