| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110 |
- From 427fbe54a8df451acd3cd31a4d6dbb1753036dee Mon Sep 17 00:00:00 2001
- From: Dave Hansen <[email protected]>
- Date: Mon, 4 Dec 2017 15:07:56 +0100
- Subject: [PATCH 181/241] x86/mm: Put MMU to hardware ASID translation in one
- place
- MIME-Version: 1.0
- Content-Type: text/plain; charset=UTF-8
- Content-Transfer-Encoding: 8bit
- CVE-2017-5754
- There are effectively two ASID types:
- 1. The one stored in the mmu_context that goes from 0..5
- 2. The one programmed into the hardware that goes from 1..6
- This consolidates the locations where converting between the two (by doing
- a +1) to a single place which gives us a nice place to comment.
- PAGE_TABLE_ISOLATION will also need to, given an ASID, know which hardware
- ASID to flush for the userspace mapping.
- Signed-off-by: Dave Hansen <[email protected]>
- Signed-off-by: Thomas Gleixner <[email protected]>
- Cc: Andy Lutomirski <[email protected]>
- Cc: Boris Ostrovsky <[email protected]>
- Cc: Borislav Petkov <[email protected]>
- Cc: Brian Gerst <[email protected]>
- Cc: Dave Hansen <[email protected]>
- Cc: David Laight <[email protected]>
- Cc: Denys Vlasenko <[email protected]>
- Cc: Eduardo Valentin <[email protected]>
- Cc: Greg KH <[email protected]>
- Cc: H. Peter Anvin <[email protected]>
- Cc: Josh Poimboeuf <[email protected]>
- Cc: Juergen Gross <[email protected]>
- Cc: Linus Torvalds <[email protected]>
- Cc: Peter Zijlstra <[email protected]>
- Cc: Will Deacon <[email protected]>
- Cc: [email protected]
- Cc: [email protected]
- Cc: [email protected]
- Cc: [email protected]
- Cc: [email protected]
- Signed-off-by: Ingo Molnar <[email protected]>
- (cherry picked from commit dd95f1a4b5ca904c78e6a097091eb21436478abb)
- Signed-off-by: Andy Whitcroft <[email protected]>
- Signed-off-by: Kleber Sacilotto de Souza <[email protected]>
- (cherry picked from commit 6f3e88a8f41123ac339d28cfdda5da0e85bec550)
- Signed-off-by: Fabian Grünbichler <[email protected]>
- ---
- arch/x86/include/asm/tlbflush.h | 31 +++++++++++++++++++------------
- 1 file changed, 19 insertions(+), 12 deletions(-)
- diff --git a/arch/x86/include/asm/tlbflush.h b/arch/x86/include/asm/tlbflush.h
- index c1c10db4156c..ecd634f87e4e 100644
- --- a/arch/x86/include/asm/tlbflush.h
- +++ b/arch/x86/include/asm/tlbflush.h
- @@ -84,30 +84,37 @@ static inline u64 inc_mm_tlb_gen(struct mm_struct *mm)
- */
- #define MAX_ASID_AVAILABLE ((1 << CR3_AVAIL_ASID_BITS) - 2)
-
- -/*
- - * If PCID is on, ASID-aware code paths put the ASID+1 into the PCID bits.
- - * This serves two purposes. It prevents a nasty situation in which
- - * PCID-unaware code saves CR3, loads some other value (with PCID == 0),
- - * and then restores CR3, thus corrupting the TLB for ASID 0 if the saved
- - * ASID was nonzero. It also means that any bugs involving loading a
- - * PCID-enabled CR3 with CR4.PCIDE off will trigger deterministically.
- - */
- +static inline u16 kern_pcid(u16 asid)
- +{
- + VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
- + /*
- + * If PCID is on, ASID-aware code paths put the ASID+1 into the
- + * PCID bits. This serves two purposes. It prevents a nasty
- + * situation in which PCID-unaware code saves CR3, loads some other
- + * value (with PCID == 0), and then restores CR3, thus corrupting
- + * the TLB for ASID 0 if the saved ASID was nonzero. It also means
- + * that any bugs involving loading a PCID-enabled CR3 with
- + * CR4.PCIDE off will trigger deterministically.
- + */
- + return asid + 1;
- +}
- +
- struct pgd_t;
- static inline unsigned long build_cr3(pgd_t *pgd, u16 asid)
- {
- if (static_cpu_has(X86_FEATURE_PCID)) {
- - VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
- - return __sme_pa(pgd) | (asid + 1);
- + return __pa(pgd) | kern_pcid(asid);
- } else {
- VM_WARN_ON_ONCE(asid != 0);
- - return __sme_pa(pgd);
- + return __pa(pgd);
- }
- }
-
- static inline unsigned long build_cr3_noflush(pgd_t *pgd, u16 asid)
- {
- VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
- - return __sme_pa(pgd) | (asid + 1) | CR3_NOFLUSH;
- + VM_WARN_ON_ONCE(!this_cpu_has(X86_FEATURE_PCID));
- + return __pa(pgd) | kern_pcid(asid) | CR3_NOFLUSH;
- }
-
- #ifdef CONFIG_PARAVIRT
- --
- 2.14.2
|