[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v18 08/10] x86: add CMT related MSRs in allowed list
On Tue, Sep 30, 2014 at 02:26:35PM +0100, Jan Beulich wrote: > >>> On 30.09.14 at 12:49, <chao.p.peng@xxxxxxxxxxxxxxx> wrote: > > --- a/xen/include/asm-x86/msr-index.h > > +++ b/xen/include/asm-x86/msr-index.h > > @@ -324,6 +324,8 @@ > > #define MSR_IA32_ENERGY_PERF_BIAS 0x000001b0 > > > > /* Platform Shared Resource MSRs */ > > +#define MSR_IA32_QOSEVTSEL 0x00000c8d > > Is this still an appropriate name with then name change "QOS" -> "CMT"? > > > +#define MSR_IA32_QMC 0x00000c8e > > The Q here may similarly be questionable now. > > > #define MSR_IA32_PQR_ASSOC 0x00000c8f > > And maybe even the one here? > > As the SDM still use the old names, I left these names unchanged so that no surprise for someone don't know the history to look into the spec. So it's more like a spec issue. On the other hand, I also feel the code here looks ugly. So maybe we can still change them to 'correct' ones? Chao > > _______________________________________________ > Xen-devel mailing list > Xen-devel@xxxxxxxxxxxxx > http://lists.xen.org/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |