From mboxrd@z Thu Jan 1 00:00:00 1970 From: Haozhong Zhang Subject: Re: [PATCH v2 06/14] x86/time.c: Scale host TSC in pvclock properly Date: Wed, 30 Dec 2015 09:07:01 +0800 Message-ID: <20151230010701.GA16809@hz-desktop.sh.intel.com> References: <1449435529-12989-1-git-send-email-haozhong.zhang@intel.com> <1449435529-12989-7-git-send-email-haozhong.zhang@intel.com> <5665CC9E.4040302@oracle.com> <20151208015249.GF3547@hz-desktop.sh.intel.com> <56672DCD.60404@oracle.com> <20151210002303.GC4704@hz-desktop.sh.intel.com> <5669685A.9080808@oracle.com> <5682CF74.1070901@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <5682CF74.1070901@oracle.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Joao Martins Cc: Kevin Tian , Keir Fraser , Jan Beulich , Jun Nakajima , Andrew Cooper , xen-devel@lists.xen.org, Aravind Gopalakrishnan , Suravee Suthikulpanit , Boris Ostrovsky List-Id: xen-devel@lists.xenproject.org On 12/29/15 18:22, Joao Martins wrote: > > > On 12/10/2015 11:56 AM, Joao Martins wrote: > > > > > > On 12/10/2015 12:23 AM, Haozhong Zhang wrote: > >> On 12/08/15 14:21, Boris Ostrovsky wrote: > >>> On 12/07/2015 08:52 PM, Haozhong Zhang wrote: > >>>> On 12/07/15 13:14, Boris Ostrovsky wrote: > >>>>> On 12/06/2015 03:58 PM, Haozhong Zhang wrote: > >>>>>> This patch makes the pvclock return the scaled host TSC and > >>>>>> corresponding scaling parameters to HVM domains if guest TSC is not > >>>>>> emulated and TSC scaling is enabled. > >>>>>> > >>>>>> Signed-off-by: Haozhong Zhang > >>>>> +Joao who has been staring at this code. > >>>>> > > Apologies for late response but I was out in the beginning of the week and was > > still catching up. > > > >>>>> Joao, can you run this series through your test with non-native frequency? > >>>>> (http://lists.xenproject.org/archives/html/xen-devel/2015-12/msg00683.html > >>>>> provides an interface to set it in config file). > >>>>> > > OK, I will run it through my time warp tests. > I have using these for a while now and so far I got no issues with a non-native > frequency [I don't have a Skylake processor in my possession to test it > natively]. In addition I rebased my vdso tests on top of your series and no > warps too. But will raise out any flags if I found anything wrong. > Thanks for the test! Haozhong