Messages in this thread | | | Date | Mon, 15 Aug 2022 15:33:27 +0200 | From | Peter Zijlstra <> | Subject | Re: [PATCHv6 04/11] x86/mm: Handle LAM on context switch |
| |
On Mon, Aug 15, 2022 at 07:17:56AM +0300, Kirill A. Shutemov wrote: > @@ -1071,8 +1082,10 @@ void flush_tlb_kernel_range(unsigned long start, unsigned long end) > */ > unsigned long __get_current_cr3_fast(void) > { > - unsigned long cr3 = build_cr3(this_cpu_read(cpu_tlbstate.loaded_mm)->pgd, > - this_cpu_read(cpu_tlbstate.loaded_mm_asid)); > + unsigned long cr3 = > + build_cr3(this_cpu_read(cpu_tlbstate.loaded_mm)->pgd, > + this_cpu_read(cpu_tlbstate.loaded_mm_asid), > + tlbstate_lam_cr3_mask());
My editor auto indents this as:
unsigned long cr3 = build_cr3(this_cpu_read(cpu_tlbstate.loaded_mm)->pgd, this_cpu_read(cpu_tlbstate.loaded_mm_asid), tlbstate_lam_cr3_mask());
which (to me) is *much* clearer.
| |