RE: [PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-22 Thread Andrejczuk, Grzegorz
> > This last define is not used anywhere. I told you before, but addressing my > review comments completely is an unduly burden, or what? > I left to be consistent with the rest of the file. I will remove the line.

RE: [PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-22 Thread Andrejczuk, Grzegorz
> > This last define is not used anywhere. I told you before, but addressing my > review comments completely is an unduly burden, or what? > I left to be consistent with the rest of the file. I will remove the line.

Re: [PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-21 Thread Thomas Gleixner
On Tue, 20 Dec 2016, Grzegorz Andrejczuk wrote: > > diff --git a/arch/x86/include/asm/msr-index.h > b/arch/x86/include/asm/msr-index.h > index 78f3760..55ffae0 100644 > --- a/arch/x86/include/asm/msr-index.h > +++ b/arch/x86/include/asm/msr-index.h > @@ -539,6 +539,12 @@ > #define

Re: [PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-21 Thread Thomas Gleixner
On Tue, 20 Dec 2016, Grzegorz Andrejczuk wrote: > > diff --git a/arch/x86/include/asm/msr-index.h > b/arch/x86/include/asm/msr-index.h > index 78f3760..55ffae0 100644 > --- a/arch/x86/include/asm/msr-index.h > +++ b/arch/x86/include/asm/msr-index.h > @@ -539,6 +539,12 @@ > #define

[PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-20 Thread Grzegorz Andrejczuk
Define new MSR MISC_FEATURE_ENABLES (0x140). On supported CPUs if bit 1 of this new register is set, then calling MONITOR and MWAIT instructions outside of ring 0 will not cause invalid-opcode exception. The MSR MISC_FEATURE_ENABLES is not yet documented in the SDM. Here is the relevant

[PATCH v11 1/5] x86/msr: add MSR_MISC_FEATURE_ENABLES and RING3MWAIT bit

2016-12-20 Thread Grzegorz Andrejczuk
Define new MSR MISC_FEATURE_ENABLES (0x140). On supported CPUs if bit 1 of this new register is set, then calling MONITOR and MWAIT instructions outside of ring 0 will not cause invalid-opcode exception. The MSR MISC_FEATURE_ENABLES is not yet documented in the SDM. Here is the relevant