* [PATCH] x86, amd: rename vmmu support capability
@ 2012-07-13 18:02 Davidlohr Bueso
2012-07-14 10:19 ` Borislav Petkov
0 siblings, 1 reply; 6+ messages in thread
From: Davidlohr Bueso @ 2012-07-13 18:02 UTC (permalink / raw)
To: H. Peter Anvin, Ingo Molnar; +Cc: lkml, kvm list, x86
From: Davidlohr Bueso <dave@gnu.org>
AMD has renamed nested page table technology to rapid virtualization indexing,
reflect this change in the kernel.
Signed-off-by: Davidlohr Bueso <dave@gnu.org>
---
arch/x86/include/asm/cpufeature.h | 2 +-
arch/x86/kernel/cpu/scattered.c | 2 +-
arch/x86/kvm/svm.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h
index f91e80f..a6fa778 100644
--- a/arch/x86/include/asm/cpufeature.h
+++ b/arch/x86/include/asm/cpufeature.h
@@ -185,7 +185,7 @@
#define X86_FEATURE_FLEXPRIORITY (8*32+ 2) /* Intel FlexPriority */
#define X86_FEATURE_EPT (8*32+ 3) /* Intel Extended Page Table */
#define X86_FEATURE_VPID (8*32+ 4) /* Intel Virtual Processor ID */
-#define X86_FEATURE_NPT (8*32+ 5) /* AMD Nested Page Table support */
+#define X86_FEATURE_RVI (8*32+ 5) /* AMD Rapid Virtualization Indexing support */
#define X86_FEATURE_LBRV (8*32+ 6) /* AMD LBR Virtualization support */
#define X86_FEATURE_SVML (8*32+ 7) /* "svm_lock" AMD SVM locking MSR */
#define X86_FEATURE_NRIPS (8*32+ 8) /* "nrip_save" AMD SVM next_rip save */
diff --git a/arch/x86/kernel/cpu/scattered.c b/arch/x86/kernel/cpu/scattered.c
index ee8e9ab..78ec9e6 100644
--- a/arch/x86/kernel/cpu/scattered.c
+++ b/arch/x86/kernel/cpu/scattered.c
@@ -41,7 +41,7 @@ void __cpuinit init_scattered_cpuid_features(struct cpuinfo_x86 *c)
{ X86_FEATURE_XSAVEOPT, CR_EAX, 0, 0x0000000d, 1 },
{ X86_FEATURE_CPB, CR_EDX, 9, 0x80000007, 0 },
{ X86_FEATURE_HW_PSTATE, CR_EDX, 7, 0x80000007, 0 },
- { X86_FEATURE_NPT, CR_EDX, 0, 0x8000000a, 0 },
+ { X86_FEATURE_RVI, CR_EDX, 0, 0x8000000a, 0 },
{ X86_FEATURE_LBRV, CR_EDX, 1, 0x8000000a, 0 },
{ X86_FEATURE_SVML, CR_EDX, 2, 0x8000000a, 0 },
{ X86_FEATURE_NRIPS, CR_EDX, 3, 0x8000000a, 0 },
diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
index f75af40..6863898 100644
--- a/arch/x86/kvm/svm.c
+++ b/arch/x86/kvm/svm.c
@@ -900,7 +900,7 @@ static __init int svm_hardware_setup(void)
goto err;
}
- if (!boot_cpu_has(X86_FEATURE_NPT))
+ if (!boot_cpu_has(X86_FEATURE_RVI))
npt_enabled = false;
if (npt_enabled && !npt) {
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] x86, amd: rename vmmu support capability
2012-07-13 18:02 [PATCH] x86, amd: rename vmmu support capability Davidlohr Bueso
@ 2012-07-14 10:19 ` Borislav Petkov
2012-07-14 13:33 ` Davidlohr Bueso
2012-07-14 13:38 ` H. Peter Anvin
0 siblings, 2 replies; 6+ messages in thread
From: Borislav Petkov @ 2012-07-14 10:19 UTC (permalink / raw)
To: Davidlohr Bueso; +Cc: H. Peter Anvin, Ingo Molnar, lkml, kvm list, x86
On Fri, Jul 13, 2012 at 08:02:55PM +0200, Davidlohr Bueso wrote:
> From: Davidlohr Bueso <dave@gnu.org>
>
> AMD has renamed nested page table technology to rapid virtualization indexing,
> reflect this change in the kernel.
>
> Signed-off-by: Davidlohr Bueso <dave@gnu.org>
You know that /proc/cpuinfo is a userspace ABI, right?
And are you sure nothing is using that string -
"npt" - since it got added almost three years ago by
414bb144efa2d2fe16d104d836d0d6b6e9265788?
--
Regards/Gruss,
Boris.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86, amd: rename vmmu support capability
2012-07-14 10:19 ` Borislav Petkov
@ 2012-07-14 13:33 ` Davidlohr Bueso
2012-07-14 13:38 ` H. Peter Anvin
1 sibling, 0 replies; 6+ messages in thread
From: Davidlohr Bueso @ 2012-07-14 13:33 UTC (permalink / raw)
To: Borislav Petkov; +Cc: H. Peter Anvin, Ingo Molnar, lkml, kvm list, x86
On Sat, 2012-07-14 at 12:19 +0200, Borislav Petkov wrote:
> On Fri, Jul 13, 2012 at 08:02:55PM +0200, Davidlohr Bueso wrote:
> > From: Davidlohr Bueso <dave@gnu.org>
> >
> > AMD has renamed nested page table technology to rapid virtualization indexing,
> > reflect this change in the kernel.
> >
> > Signed-off-by: Davidlohr Bueso <dave@gnu.org>
>
> You know that /proc/cpuinfo is a userspace ABI, right?
Yes.
>
> And are you sure nothing is using that string -
> "npt" - since it got added almost three years ago by
> 414bb144efa2d2fe16d104d836d0d6b6e9265788?
AFAIK no, it's not being used - that doesn't mean, of course, that there
are no users.
Thanks,
Davidlohr
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86, amd: rename vmmu support capability
2012-07-14 10:19 ` Borislav Petkov
2012-07-14 13:33 ` Davidlohr Bueso
@ 2012-07-14 13:38 ` H. Peter Anvin
2012-07-14 14:28 ` Davidlohr Bueso
1 sibling, 1 reply; 6+ messages in thread
From: H. Peter Anvin @ 2012-07-14 13:38 UTC (permalink / raw)
To: Borislav Petkov, Davidlohr Bueso; +Cc: Ingo Molnar, lkml, kvm list, x86
Yep, NAK on this one.
Borislav Petkov <bp@alien8.de> wrote:
>On Fri, Jul 13, 2012 at 08:02:55PM +0200, Davidlohr Bueso wrote:
>> From: Davidlohr Bueso <dave@gnu.org>
>>
>> AMD has renamed nested page table technology to rapid virtualization
>indexing,
>> reflect this change in the kernel.
>>
>> Signed-off-by: Davidlohr Bueso <dave@gnu.org>
>
>You know that /proc/cpuinfo is a userspace ABI, right?
>
>And are you sure nothing is using that string -
>"npt" - since it got added almost three years ago by
>414bb144efa2d2fe16d104d836d0d6b6e9265788?
>
>--
>Regards/Gruss,
> Boris.
--
Sent from my mobile phone. Please excuse brevity and lack of formatting.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86, amd: rename vmmu support capability
2012-07-14 13:38 ` H. Peter Anvin
@ 2012-07-14 14:28 ` Davidlohr Bueso
2012-07-14 17:32 ` Borislav Petkov
0 siblings, 1 reply; 6+ messages in thread
From: Davidlohr Bueso @ 2012-07-14 14:28 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Borislav Petkov, Ingo Molnar, lkml, kvm list, x86
On Sat, 2012-07-14 at 15:38 +0200, H. Peter Anvin wrote:
> Yep, NAK on this one.
Ok, we could at least add a comment when defining X86_FEATURE_NPT.
Thanks,
Davidlohr
>
> Borislav Petkov <bp@alien8.de> wrote:
>
> >On Fri, Jul 13, 2012 at 08:02:55PM +0200, Davidlohr Bueso wrote:
> >> From: Davidlohr Bueso <dave@gnu.org>
> >>
> >> AMD has renamed nested page table technology to rapid virtualization
> >indexing,
> >> reflect this change in the kernel.
> >>
> >> Signed-off-by: Davidlohr Bueso <dave@gnu.org>
> >
> >You know that /proc/cpuinfo is a userspace ABI, right?
> >
> >And are you sure nothing is using that string -
> >"npt" - since it got added almost three years ago by
> >414bb144efa2d2fe16d104d836d0d6b6e9265788?
> >
> >--
> >Regards/Gruss,
> > Boris.
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86, amd: rename vmmu support capability
2012-07-14 14:28 ` Davidlohr Bueso
@ 2012-07-14 17:32 ` Borislav Petkov
0 siblings, 0 replies; 6+ messages in thread
From: Borislav Petkov @ 2012-07-14 17:32 UTC (permalink / raw)
To: Davidlohr Bueso; +Cc: H. Peter Anvin, Ingo Molnar, lkml, kvm list, x86
On Sat, Jul 14, 2012 at 04:28:51PM +0200, Davidlohr Bueso wrote:
> On Sat, 2012-07-14 at 15:38 +0200, H. Peter Anvin wrote:
> > Yep, NAK on this one.
>
> Ok, we could at least add a comment when defining X86_FEATURE_NPT.
And the valid, sane, technical reason for having a comment where any
internet search could do, is... ?
--
Regards/Gruss,
Boris.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-14 17:32 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-13 18:02 [PATCH] x86, amd: rename vmmu support capability Davidlohr Bueso
2012-07-14 10:19 ` Borislav Petkov
2012-07-14 13:33 ` Davidlohr Bueso
2012-07-14 13:38 ` H. Peter Anvin
2012-07-14 14:28 ` Davidlohr Bueso
2012-07-14 17:32 ` Borislav Petkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox