Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Joao Martins
On 03/21/2016 03:27 PM, Andrew Cooper wrote: > On 21/03/16 15:10, Jan Beulich wrote: > On 17.03.16 at 17:12, wrote: >>> --- a/xen/include/public/xen.h >>> +++ b/xen/include/public/xen.h >>> @@ -614,10 +614,14 @@ struct vcpu_time_info { >>> */ >>> uint32_t tsc_to_system_mul; >>>

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Andrew Cooper
On 21/03/16 15:10, Jan Beulich wrote: On 17.03.16 at 17:12, wrote: >> --- a/xen/include/public/xen.h >> +++ b/xen/include/public/xen.h >> @@ -614,10 +614,14 @@ struct vcpu_time_info { >> */ >> uint32_t tsc_to_system_mul; >> int8_t tsc_shift; >> -int8_t pad1[3]; >> +

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Jan Beulich
>>> On 17.03.16 at 17:12, wrote: > --- a/xen/include/public/xen.h > +++ b/xen/include/public/xen.h > @@ -614,10 +614,14 @@ struct vcpu_time_info { > */ > uint32_t tsc_to_system_mul; > int8_t tsc_shift; > -int8_t pad1[3]; > +int8_t flags; For use as flags I'm sure thi

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Joao Martins
On 03/21/2016 11:43 AM, Andrew Cooper wrote: > On 21/03/16 11:42, Joao Martins wrote: >> >> On 03/18/2016 08:12 PM, Andrew Cooper wrote: >>> On 17/03/16 16:12, Joao Martins wrote: This field has two possible flags (as of latest pvclock ABI shared with KVM). flags: bits in this

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Andrew Cooper
On 21/03/16 11:42, Joao Martins wrote: > > On 03/18/2016 08:12 PM, Andrew Cooper wrote: >> On 17/03/16 16:12, Joao Martins wrote: >>> This field has two possible flags (as of latest pvclock ABI >>> shared with KVM). >>> >>> flags: bits in this field indicate extended capabilities >>> coordinated be

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-21 Thread Joao Martins
On 03/18/2016 08:12 PM, Andrew Cooper wrote: > On 17/03/16 16:12, Joao Martins wrote: >> This field has two possible flags (as of latest pvclock ABI >> shared with KVM). >> >> flags: bits in this field indicate extended capabilities >> coordinated between the guest and the hypervisor. Specifical

[Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-19 Thread Joao Martins
This field has two possible flags (as of latest pvclock ABI shared with KVM). flags: bits in this field indicate extended capabilities coordinated between the guest and the hypervisor. Specifically on KVM, availability of specific flags has to be checked in 0x4001 cpuid leaf. On Xen, we don't

Re: [Xen-devel] [PATCH 1/5] public/xen.h: add flags field to vcpu_time_info

2016-03-19 Thread Andrew Cooper
On 17/03/16 16:12, Joao Martins wrote: > This field has two possible flags (as of latest pvclock ABI > shared with KVM). > > flags: bits in this field indicate extended capabilities > coordinated between the guest and the hypervisor. Specifically > on KVM, availability of specific flags has to be