0181-x86-mm-Put-MMU-to-hardware-ASID-translation-in-one-p.patch 4.0 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110
  1. From 427fbe54a8df451acd3cd31a4d6dbb1753036dee Mon Sep 17 00:00:00 2001
  2. From: Dave Hansen <[email protected]>
  3. Date: Mon, 4 Dec 2017 15:07:56 +0100
  4. Subject: [PATCH 181/241] x86/mm: Put MMU to hardware ASID translation in one
  5. place
  6. MIME-Version: 1.0
  7. Content-Type: text/plain; charset=UTF-8
  8. Content-Transfer-Encoding: 8bit
  9. CVE-2017-5754
  10. There are effectively two ASID types:
  11. 1. The one stored in the mmu_context that goes from 0..5
  12. 2. The one programmed into the hardware that goes from 1..6
  13. This consolidates the locations where converting between the two (by doing
  14. a +1) to a single place which gives us a nice place to comment.
  15. PAGE_TABLE_ISOLATION will also need to, given an ASID, know which hardware
  16. ASID to flush for the userspace mapping.
  17. Signed-off-by: Dave Hansen <[email protected]>
  18. Signed-off-by: Thomas Gleixner <[email protected]>
  19. Cc: Andy Lutomirski <[email protected]>
  20. Cc: Boris Ostrovsky <[email protected]>
  21. Cc: Borislav Petkov <[email protected]>
  22. Cc: Brian Gerst <[email protected]>
  23. Cc: Dave Hansen <[email protected]>
  24. Cc: David Laight <[email protected]>
  25. Cc: Denys Vlasenko <[email protected]>
  26. Cc: Eduardo Valentin <[email protected]>
  27. Cc: Greg KH <[email protected]>
  28. Cc: H. Peter Anvin <[email protected]>
  29. Cc: Josh Poimboeuf <[email protected]>
  30. Cc: Juergen Gross <[email protected]>
  31. Cc: Linus Torvalds <[email protected]>
  32. Cc: Peter Zijlstra <[email protected]>
  33. Cc: Will Deacon <[email protected]>
  34. Cc: [email protected]
  35. Cc: [email protected]
  36. Cc: [email protected]
  37. Cc: [email protected]
  38. Cc: [email protected]
  39. Signed-off-by: Ingo Molnar <[email protected]>
  40. (cherry picked from commit dd95f1a4b5ca904c78e6a097091eb21436478abb)
  41. Signed-off-by: Andy Whitcroft <[email protected]>
  42. Signed-off-by: Kleber Sacilotto de Souza <[email protected]>
  43. (cherry picked from commit 6f3e88a8f41123ac339d28cfdda5da0e85bec550)
  44. Signed-off-by: Fabian Grünbichler <[email protected]>
  45. ---
  46. arch/x86/include/asm/tlbflush.h | 31 +++++++++++++++++++------------
  47. 1 file changed, 19 insertions(+), 12 deletions(-)
  48. diff --git a/arch/x86/include/asm/tlbflush.h b/arch/x86/include/asm/tlbflush.h
  49. index c1c10db4156c..ecd634f87e4e 100644
  50. --- a/arch/x86/include/asm/tlbflush.h
  51. +++ b/arch/x86/include/asm/tlbflush.h
  52. @@ -84,30 +84,37 @@ static inline u64 inc_mm_tlb_gen(struct mm_struct *mm)
  53. */
  54. #define MAX_ASID_AVAILABLE ((1 << CR3_AVAIL_ASID_BITS) - 2)
  55. -/*
  56. - * If PCID is on, ASID-aware code paths put the ASID+1 into the PCID bits.
  57. - * This serves two purposes. It prevents a nasty situation in which
  58. - * PCID-unaware code saves CR3, loads some other value (with PCID == 0),
  59. - * and then restores CR3, thus corrupting the TLB for ASID 0 if the saved
  60. - * ASID was nonzero. It also means that any bugs involving loading a
  61. - * PCID-enabled CR3 with CR4.PCIDE off will trigger deterministically.
  62. - */
  63. +static inline u16 kern_pcid(u16 asid)
  64. +{
  65. + VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
  66. + /*
  67. + * If PCID is on, ASID-aware code paths put the ASID+1 into the
  68. + * PCID bits. This serves two purposes. It prevents a nasty
  69. + * situation in which PCID-unaware code saves CR3, loads some other
  70. + * value (with PCID == 0), and then restores CR3, thus corrupting
  71. + * the TLB for ASID 0 if the saved ASID was nonzero. It also means
  72. + * that any bugs involving loading a PCID-enabled CR3 with
  73. + * CR4.PCIDE off will trigger deterministically.
  74. + */
  75. + return asid + 1;
  76. +}
  77. +
  78. struct pgd_t;
  79. static inline unsigned long build_cr3(pgd_t *pgd, u16 asid)
  80. {
  81. if (static_cpu_has(X86_FEATURE_PCID)) {
  82. - VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
  83. - return __sme_pa(pgd) | (asid + 1);
  84. + return __pa(pgd) | kern_pcid(asid);
  85. } else {
  86. VM_WARN_ON_ONCE(asid != 0);
  87. - return __sme_pa(pgd);
  88. + return __pa(pgd);
  89. }
  90. }
  91. static inline unsigned long build_cr3_noflush(pgd_t *pgd, u16 asid)
  92. {
  93. VM_WARN_ON_ONCE(asid > MAX_ASID_AVAILABLE);
  94. - return __sme_pa(pgd) | (asid + 1) | CR3_NOFLUSH;
  95. + VM_WARN_ON_ONCE(!this_cpu_has(X86_FEATURE_PCID));
  96. + return __pa(pgd) | kern_pcid(asid) | CR3_NOFLUSH;
  97. }
  98. #ifdef CONFIG_PARAVIRT
  99. --
  100. 2.14.2