All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Jan Beulich <JBeulich@novell.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: syncing wall clock time from Dom0 to hypervisor
Date: Wed, 13 Jul 2011 09:52:35 -0400	[thread overview]
Message-ID: <20110713135235.GB8466@dumpdata.com> (raw)
In-Reply-To: <4E0C6911020000780004B45B@nat28.tlf.novell.com>

On Thu, Jun 30, 2011 at 11:16:17AM +0100, Jan Beulich wrote:
> While in the upstream kernel I'm unable to find any use of XENPF_settime
> (and the DOM0_SETTIME alias of it) at all, in the 2.6.18 tree (and the
> forward ports of it) the function gets used only when ntp_synced()
> returns true (and - that's minor - when independent_wallclock is not
> set).
> 
> It would however seem to me that this doesn't cover the case where
> the host clock gets adjusted by the Dom0 admin. Is there perhaps
> someone who remembers whether this was implemented the way it
> is intentionally?

<groan> I think we just forgot.

Looking at xen_set_wallclock it just looks as we need to make it do
a similar hypercall as the sync_xen_wallclock to update it?

> 
> There's no such restriction in Jeremy's tree, but that tree also doesn't
> seem to be doing updates at regular intervals (for NTP), nor does it
> look like the CMOS clock would get updated here at all (i.e. I can't
> really take this as a proper reference either).

I think the 'sync_cmos_clock' looks to be doing that. And the hook
to the Xen is via 'update_persistent_clock' call?

  reply	other threads:[~2011-07-13 13:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-30 10:16 syncing wall clock time from Dom0 to hypervisor Jan Beulich
2011-07-13 13:52 ` Konrad Rzeszutek Wilk [this message]
2011-07-13 15:17   ` Ian Campbell
2011-07-13 14:37 Jan Beulich

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=20110713135235.GB8466@dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=JBeulich@novell.com \
    --cc=xen-devel@lists.xensource.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.