All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olaf Hering <olaf@aepfle.de>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: "Stefano Stabellini" <sstabellini@kernel.org>,
	"Wei Liu" <wei.liu2@citrix.com>,
	"Konrad Rzeszutek Wilk" <konrad.wilk@oracle.com>,
	"George Dunlap" <George.Dunlap@eu.citrix.com>,
	"Tim Deegan" <tim@xen.org>,
	"Ian Jackson" <ian.jackson@eu.citrix.com>,
	"Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>,
	xen-devel@lists.xen.org, "Julien Grall" <julien.grall@arm.com>,
	"Jan Beulich" <jbeulich@suse.com>
Subject: Re: [PATCH v9] new config option vtsc_tolerance_khz to avoid TSC emulation
Date: Tue, 9 Oct 2018 18:42:14 +0200	[thread overview]
Message-ID: <20181009184214.7754dc5f.olaf@aepfle.de> (raw)
In-Reply-To: <ba4e6e5f-b342-10e5-971e-1e71d50b3073@citrix.com>


[-- Attachment #1.1: Type: text/plain, Size: 1872 bytes --]

Thanks for all the replies. I will work through them.
One remark now:

Am Mon, 1 Oct 2018 13:39:51 +0100
schrieb Andrew Cooper <andrew.cooper3@citrix.com>:

> > With the new option the host admin can decide how a domU should behave
> > when it is migrated across systems of the same class. Since there is
> > always some jitter when Xen calibrates the cpu_khz value, all hosts of
> > the same class will most likely have slightly different values. As a
> > result vTSC emulation is unavoidable. Data collected during the incident
> > which triggered this change showed a jitter of up to 200 KHz across
> > systems of the same class.  
> Do you have any further details of the systems involved?  If they are
> identical systems, they should all have the same real TSC frequency, and
> its a known issue that Xen isn't very good at working out the
> frequency.

If "identical" systems really have an identical frequency, but Xen is
unable to figure out the exact value of that frequency that it is going
to use for its calculations, why would it be a good idea in the first place
to rely on such an estimated value? If there is drift expected, shouldn't
that be handled by running ntpd inside dom0 and the domUs?

If I did that math correctly: a domU that is started with the estimated
cpu_khz value of 2666766, and is migrated to another host with the
estimated cpu_khz value of 2666755, would see 950400K fewer ticks during
a day. How many ticks actually happen, noone knows. But since that amount
of ticks represents a time span of ca. 0.3 seconds I would assume that
ntpd can handle such drift.

With larger differences, like the 200 KHz mentioned above, the drift will
be larger.


What I mean to say is: should the comparison of both hosts be based
on different metrics, instead of a plain estimated value in 'cpu_khz'?


Olaf

[-- Attachment #1.2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 157 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

  parent reply	other threads:[~2018-10-09 16:42 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-07 13:08 [PATCH v9] new config option vtsc_tolerance_khz to avoid TSC emulation Olaf Hering
2018-06-07 13:31 ` Jan Beulich
2018-06-07 13:47   ` Olaf Hering
2018-06-07 14:44     ` Jan Beulich
2018-06-07 14:49       ` Olaf Hering
2018-06-07 14:55         ` Jan Beulich
2018-06-07 21:05           ` Olaf Hering
2018-06-26 17:11 ` Olaf Hering
2018-08-01 16:17   ` Olaf Hering
2018-09-13  7:39     ` Olaf Hering
2018-09-28 13:50       ` Olaf Hering
2018-10-01 10:52         ` Ian Jackson
2018-10-01 11:25           ` Jan Beulich
2018-10-01 13:38             ` George Dunlap
2018-10-01 14:00               ` Jan Beulich
2018-10-01 14:25                 ` George Dunlap
2018-10-01 16:01                   ` Wei Liu
2018-10-01 12:39 ` Andrew Cooper
2018-10-01 13:24   ` [PATCH v9] new config option vtsc_tolerance_khz to avoid TSC emulation [and 1 more messages] Ian Jackson
2018-10-01 13:58     ` Andrew Cooper
2018-10-01 13:29   ` [PATCH v9] new config option vtsc_tolerance_khz to avoid TSC emulation Jan Beulich
2018-10-01 15:17     ` Andrew Cooper
2018-10-01 15:28       ` Jan Beulich
2018-10-09 16:42   ` Olaf Hering [this message]
2018-11-29 13:21   ` Olaf Hering

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=20181009184214.7754dc5f.olaf@aepfle.de \
    --to=olaf@aepfle.de \
    --cc=George.Dunlap@eu.citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=julien.grall@arm.com \
    --cc=konrad.wilk@oracle.com \
    --cc=marmarek@invisiblethingslab.com \
    --cc=sstabellini@kernel.org \
    --cc=tim@xen.org \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.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 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.