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

Re: [Xen-devel] [PATCH v2 08/14] x86/hvm: Collect information of TSC scaling ratio



On 12/10/15 18:19, Tian, Kevin wrote:
> > From: Zhang, Haozhong
> > Sent: Monday, December 07, 2015 4:59 AM
> ratio
> > 
> > Both VMX TSC scaling and SVM TSC ratio use the 64-bit TSC scaling ratio,
> > but the number of fractional bits of the ratio is different between VMX
> > and SVM. This patch adds the architecture code to collect the number of
> > fractional bits and other related information into fields of struct
> > hvm_function_table so that they can be used in the common code.
> > 
> > Signed-off-by: Haozhong Zhang <haozhong.zhang@xxxxxxxxx>
> 
> Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>, with one comment
> 
> 
> > diff --git a/xen/include/asm-x86/hvm/hvm.h b/xen/include/asm-x86/hvm/hvm.h
> > index aba63ab..8b10a67 100644
> > --- a/xen/include/asm-x86/hvm/hvm.h
> > +++ b/xen/include/asm-x86/hvm/hvm.h
> > @@ -100,6 +100,18 @@ struct hvm_function_table {
> >      unsigned int hap_capabilities;
> > 
> >      /*
> > +     * Parameters of hardware-assisted TSC scaling.
> > +     */
> > +    /* is TSC scaling supported? */
> > +    bool_t   tsc_scaling_supported;
> > +    /* number of bits of the fractional part of TSC scaling ratio */
> > +    uint8_t  tsc_scaling_ratio_frac_bits;
> > +    /* default TSC scaling ratio (no scaling) */
> > +    uint64_t default_tsc_scaling_ratio;
> > +    /* maxmimum-allowed TSC scaling ratio */
> 
> maxmimum -> maximum

will fix in the next version

Thanks,
Haozhong

> 
> > +    uint64_t max_tsc_scaling_ratio;
> > +
> > +    /*
> >       * Initialise/destroy HVM domain/vcpu resources
> >       */
> >      int  (*domain_initialise)(struct domain *d);
> > --
> > 2.6.3
> 

_______________________________________________
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®.