Re: [Xen-devel] [PATCH v2 3/3] x86/vlapic: Apply change to TDCR right away to the timer

2017-08-04 Thread Anthony PERARD
On Thu, Aug 03, 2017 at 09:29:02AM -0600, Jan Beulich wrote: > >>> Anthony PERARD 07/18/17 7:10 PM >>> > >@@ -701,6 +702,13 @@ static void vlapic_update_timer(struct vlapic *vlapic, > >uint32_t lvtt); > >delta = period - time_passed; > >} > > > >+if (

Re: [Xen-devel] [PATCH v2 3/3] x86/vlapic: Apply change to TDCR right away to the timer

2017-08-03 Thread Jan Beulich
>>> Anthony PERARD 07/18/17 7:10 PM >>> >--- a/xen/arch/x86/hvm/vlapic.c >+++ b/xen/arch/x86/hvm/vlapic.c >@@ -671,12 +671,13 @@ static void vlapic_tdt_pt_cb(struct vcpu *v, void *data) > >/* >* This function is used when a register related to the APIC timer is

[Xen-devel] [PATCH v2 3/3] x86/vlapic: Apply change to TDCR right away to the timer

2017-07-18 Thread Anthony PERARD
The description in the Intel SDM of how the divide configuration register is used: "The APIC timer frequency will be the processor's bus clock or core crystal clock frequency divided by the value specified in the divide configuration register." Observation of baremetal shown that when the TDCR is