[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 0/3] tsc adjust implementation for hvm


  • To: Jan Beulich <JBeulich@xxxxxxxx>, Keir Fraser <keir@xxxxxxx>
  • From: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
  • Date: Thu, 20 Sep 2012 08:48:26 +0000
  • Accept-language: en-US
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxx>
  • Delivery-date: Thu, 20 Sep 2012 08:50:52 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>
  • Thread-index: AQHNlwtlVpsyDniHZEaRwosPbRL+3peS6wDw
  • Thread-topic: [Xen-devel] [PATCH 0/3] tsc adjust implementation for hvm

Jan Beulich wrote:
>>>> On 20.09.12 at 10:16, Keir Fraser <keir@xxxxxxx> wrote:
>> On 20/09/2012 09:06, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
>> 
>>> Intel recently release a new tsc adjust feature at latest SDM
>>> 17.13.3. CPUID.7.0.EBX[1]=1 indicates TSC_ADJUST MSR 0x3b is
>>> supported. 
>>> 
>>> Basically it is used to simplify TSC synchronization, operation of
>>> IA32_TSC_ADJUST MSR is as follows:
>>> 1). On RESET, the value of the IA32_TSC_ADJUST MSR is 0;
>>> 2). If an execution of WRMSR to the IA32_TIME_STAMP_COUNTER MSR
>>>     adds (or subtracts) value X from the TSC, the logical processor
>>>     also adds (or subtracts) value X from the IA32_TSC_ADJUST MSR;
>>> 3). If an execution of WRMSR to the IA32_TSC_ADJUST MSR adds (or
>>>     subtracts) value X from that MSR, the logical processor also
>>>     adds (or subtracts) value X from the TSC;
>>> 
>>> With it, OS would be easier when sync tsc.
>> 
>> Actually it appears to strictly add code to, and hence complicate,
>> the hypervisor. So how exactly is it beneficial?
> 
> It's beneficial to the guest if I'm not mistaken, for precisely the
> purpose that patch 2 doesn't address for the hypervisor in spite
> of its title (see my response there).
> 
> Jan

Yes, updated accordingly w/ more clear comments and would send later.

Thanks,
Jinsong
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.