Re: [Xen-devel] [PATCH v18 08/10] x86: add CMT related MSRs in allowed list

[Chao Peng <chao.p.peng@xxxxxxxxxxxxxxx>]

On Mon, Oct 06, 2014 at 03:06:21PM +0100, Andrew Cooper wrote:
> On 06/10/14 15:01, Chao Peng wrote:
> > 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
> >>>  
> > We discussed this internally and finally we decide to continue to use
> > the names in current SDM. Which is slightly different with what we used
> > here and also different with what you suggested here.
> >
> > It looks still a little confusing but these MSRs maybe used in features
> > than just CMT so narrowing down to CMT is not OK.
> >
> > I will fix this in a new version and let you know.
> > Thanks,
> > Chao
> 
> Be aware that most of your series has been committed to staging.
> 
> It will be fine to start a brand new series against staging containing
> any outstanding patches, and new changes
> 
> ~Andrew

I'd prefer not to fix this at this time as the patch serial has already
gone into master branch. After all it'just naming issue and current names
are consistent with current CMT code. If other feature coming in the future
makes these names ambiguous then we can change at that time.

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