Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1310916 > unrolled thread

[PATCH] x86/cpufeature: Add AMD AVIC bit

Started byBorislav Petkov <bp@alien8.de>
First post2016-01-16 11:00 +0100
Last post2016-01-19 14:40 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] x86/cpufeature: Add AMD AVIC bit Borislav Petkov <bp@alien8.de> - 2016-01-16 11:00 +0100
    Re: [PATCH] x86/cpufeature: Add AMD AVIC bit Joerg Roedel <joro@8bytes.org> - 2016-01-19 13:40 +0100
      Re: [PATCH] x86/cpufeature: Add AMD AVIC bit Borislav Petkov <bp@alien8.de> - 2016-01-19 14:50 +0100
        Re: [PATCH] x86/cpufeature: Add AMD AVIC bit Joerg Roedel <joro@8bytes.org> - 2016-01-19 15:00 +0100
    [tip:x86/cpu] x86/cpufeature: Add AMD AVIC bit tip-bot for Borislav Petkov <tipbot@zytor.com> - 2016-01-19 14:40 +0100

#1310916 — [PATCH] x86/cpufeature: Add AMD AVIC bit

FromBorislav Petkov <bp@alien8.de>
Date2016-01-16 11:00 +0100
Subject[PATCH] x86/cpufeature: Add AMD AVIC bit
Message-ID<qRvOh-2DZ-3@gated-at.bofh.it>
From: Borislav Petkov <bp@suse.de>

CPUID Fn8000_000A_EDX[13] denotes support for AMD's Virtual Interrupt
controller, i.e., APIC virtualization.

Cc: David Kaplan <david.kaplan@amd.com>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
---
 arch/x86/include/asm/cpufeature.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h
index 7ad8c9464297..bbf166e805be 100644
--- a/arch/x86/include/asm/cpufeature.h
+++ b/arch/x86/include/asm/cpufeature.h
@@ -268,6 +268,7 @@
 #define X86_FEATURE_DECODEASSISTS (15*32+ 7) /* Decode Assists support */
 #define X86_FEATURE_PAUSEFILTER (15*32+10) /* filtered pause intercept */
 #define X86_FEATURE_PFTHRESHOLD (15*32+12) /* pause filter threshold */
+#define X86_FEATURE_AVIC	(15*32+13) /* Virtual Interrupt Controller */
 
 /*
  * BUG word(s)
-- 
2.3.5

[toc] | [next] | [standalone]


#1312182

FromJoerg Roedel <joro@8bytes.org>
Date2016-01-19 13:40 +0100
Message-ID<qSDJM-7SY-15@gated-at.bofh.it>
In reply to#1310916
On Sat, Jan 16, 2016 at 10:58:12AM +0100, Borislav Petkov wrote:
> From: Borislav Petkov <bp@suse.de>
> 
> CPUID Fn8000_000A_EDX[13] denotes support for AMD's Virtual Interrupt
> controller, i.e., APIC virtualization.
> 
> Cc: David Kaplan <david.kaplan@amd.com>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Signed-off-by: Borislav Petkov <bp@suse.de>
> ---
>  arch/x86/include/asm/cpufeature.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h
> index 7ad8c9464297..bbf166e805be 100644
> --- a/arch/x86/include/asm/cpufeature.h
> +++ b/arch/x86/include/asm/cpufeature.h
> @@ -268,6 +268,7 @@
>  #define X86_FEATURE_DECODEASSISTS (15*32+ 7) /* Decode Assists support */
>  #define X86_FEATURE_PAUSEFILTER (15*32+10) /* filtered pause intercept */
>  #define X86_FEATURE_PFTHRESHOLD (15*32+12) /* pause filter threshold */
> +#define X86_FEATURE_AVIC	(15*32+13) /* Virtual Interrupt Controller */

That is not enough, you also need to add the feature to the scattered
cpuid_bits in arch/x86/kernel/cpu/scattered.c.
The cpuid Fn8000_000A_EDX is not mapped 1-1 into the x86_capability
array.


	Joerg

[toc] | [prev] | [next] | [standalone]


#1312253

FromBorislav Petkov <bp@alien8.de>
Date2016-01-19 14:50 +0100
Message-ID<qSEPw-8d-31@gated-at.bofh.it>
In reply to#1312182
On Tue, Jan 19, 2016 at 01:39:00PM +0100, Joerg Roedel wrote:
> That is not enough, you also need to add the feature to the
> scattered cpuid_bits in arch/x86/kernel/cpu/scattered.c. The cpuid
> Fn8000_000A_EDX is not mapped 1-1 into the x86_capability array.

2ccd71f1b278 ("x86/cpufeature: Move some of the scattered feature bits to x86_capability")

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

[toc] | [prev] | [next] | [standalone]


#1312265

FromJoerg Roedel <joro@8bytes.org>
Date2016-01-19 15:00 +0100
Message-ID<qSEZc-bz-9@gated-at.bofh.it>
In reply to#1312253
On Tue, Jan 19, 2016 at 02:44:33PM +0100, Borislav Petkov wrote:
> On Tue, Jan 19, 2016 at 01:39:00PM +0100, Joerg Roedel wrote:
> > That is not enough, you also need to add the feature to the
> > scattered cpuid_bits in arch/x86/kernel/cpu/scattered.c. The cpuid
> > Fn8000_000A_EDX is not mapped 1-1 into the x86_capability array.
> 
> 2ccd71f1b278 ("x86/cpufeature: Move some of the scattered feature bits to x86_capability")

Ah I see, that commit wasn't present in the tree I looked at. In that
case:

Reviewed-by: Joerg Roedel <jroedel@suse.de>

[toc] | [prev] | [next] | [standalone]


#1312231 — [tip:x86/cpu] x86/cpufeature: Add AMD AVIC bit

Fromtip-bot for Borislav Petkov <tipbot@zytor.com>
Date2016-01-19 14:40 +0100
Subject[tip:x86/cpu] x86/cpufeature: Add AMD AVIC bit
Message-ID<qSEFQ-4S-17@gated-at.bofh.it>
In reply to#1310916
Commit-ID:  a1ff5726081858a9ad98934eff7af6616c576875
Gitweb:     http://git.kernel.org/tip/a1ff5726081858a9ad98934eff7af6616c576875
Author:     Borislav Petkov <bp@suse.de>
AuthorDate: Sat, 16 Jan 2016 10:58:12 +0100
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Tue, 19 Jan 2016 08:29:28 +0100

x86/cpufeature: Add AMD AVIC bit

CPUID Fn8000_000A_EDX[13] denotes support for AMD's Virtual
Interrupt controller, i.e., APIC virtualization.

Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brian Gerst <brgerst@gmail.com>
Cc: David Kaplan <david.kaplan@amd.com>
Cc: Denys Vlasenko <dvlasenk@redhat.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Joerg Roedel <joro@8bytes.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Link: http://lkml.kernel.org/r/1452938292-12327-1-git-send-email-bp@alien8.de
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 arch/x86/include/asm/cpufeature.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h
index 7ad8c94..bbf166e 100644
--- a/arch/x86/include/asm/cpufeature.h
+++ b/arch/x86/include/asm/cpufeature.h
@@ -268,6 +268,7 @@
 #define X86_FEATURE_DECODEASSISTS (15*32+ 7) /* Decode Assists support */
 #define X86_FEATURE_PAUSEFILTER (15*32+10) /* filtered pause intercept */
 #define X86_FEATURE_PFTHRESHOLD (15*32+12) /* pause filter threshold */
+#define X86_FEATURE_AVIC	(15*32+13) /* Virtual Interrupt Controller */
 
 /*
  * BUG word(s)

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web