2.4-ac: free_pgtable P4 TLB race fix
Subsystems (3)
PATCH 1/1 2.4-ac: free_pgtable P4 TLB race fix
Alan,
Attached is a patch to resolve the free_pgtable P4 TLB reported and fixed by Intel and Linus.
The problem lies in freeing page tables before the TLB is flushed on Pentium 4s. The solution is to make certain that the page table pages are not freed prior to flushing the TLB
More information can be had at:
http://marc.theaimsgroup.com/?l=linux-kernel&m=102208353523931&w=2
Patch is against 2.4.19-pre9-ac3, please apply.
Robert Love
arch/i386/kernel/process.c +1 -1
@@ -308,7 +308,7 @@/* * Use `swapper_pg_dir' as our page directory. */ - asmvolatile ("movl %0,%%cr3" : :"r" (__pa (swapper_pg_dir))); +load_cr3 (swapper_pg_dir);/* Write 0x1234 to absolute memory location 0x472. The BIOS reads this on booting to tell it to "Bypass memory test (also warm
arch/i386/kernel/smp.c +4 -0
@@ -298,11 +298,15 @@/* * We cannot call mmdrop() because we are in interrupt context, * instead update mm->cpu_vm_mask. +* +* We need to reload %cr3 since the page tables may be going +* away from under us.. */ static void inline leave_mm (unsigned long cpu) {BUG_ON (cpu_tlbstate[cpu].state ==TLBSTATE_OK );clear_bit (cpu, &cpu_tlbstate[cpu].active_mm->cpu_vm_mask); +load_cr3 (swapper_pg_dir); }/*
arch/i386/mm/init.c +1 -1
@@ -331,7 +331,7 @@ {pagetable_init (); - __asm__("movl %%ecx,%%cr3\n" ::"c" (__pa (swapper_pg_dir))); +load_cr3 (swapper_pg_dir);#if CONFIG_X86_PAE /*
include/asm-i386/mmu_context.h +3 -3
@@ -42,7 +42,7 @@set_bit (cpu, &next->cpu_vm_mask);set_bit (cpu, &next->context.cpuvalid);/* Re-load page tables */ - asmvolatile ("movl %0,%%cr3" : :"r" (__pa (next->pgd))); +load_cr3 (next->pgd); }#ifdef CONFIG_SMP else {
@@ -51,9 +51,9 @@out_of_line_bug ();if (!test_and_set_bit (cpu, &next->cpu_vm_mask)) {/* We were in lazy tlb mode and leave_mm disabled -* tlb flush IPI delivery. We must flush our tlb. +* tlb flush IPI delivery. We must reload %cr3. */ -local_flush_tlb (); +load_cr3 (next->pgd); }if (!test_and_set_bit (cpu, &next->context.cpuvalid))load_LDT (next);
include/asm-i386/pgalloc.h +3 -4
@@ -139,7 +139,7 @@free_page ((unsigned long )pte); } -#define pte_free (pte) pte_free_slow(pte) +#define pte_free (pte) pte_free_fast(pte)#define pgd_free (pgd) free_pgd_slow(pgd)#define pgd_alloc (mm) get_pgd_fast()
@@ -228,13 +228,12 @@ };extern struct tlb_state cpu_tlbstate[NR_CPUS ]; - -#endif +#endif /* CONFIG_SMP */ static inline void flush_tlb_pgtables (struct mm_struct *mm,unsigned long start,unsigned long end) { -/* i386 does not keep any page table caches in TLB */ +flush_tlb_mm (mm); }#endif /* _I386_PGALLOC_H */
include/asm-i386/processor.h +3 -0
@@ -191,6 +191,9 @@#define X86_CR4_OSFXSR 0x0200/* enable fast FPU save and restore */ #define X86_CR4_OSXMMEXCPT 0x0400/* enable unmasked SSE exceptions */ +#define load_cr3 (pgdir) \ + asm volatile("movl %0,%%cr3": :"r" (__pa(pgdir))); +/* * Save the cr4 feature set we're using (ie * Pentium 4MB enable and PPro Global page
mm/memory.c +3 -0
@@ -164,6 +164,7 @@void clear_page_tables (struct mm_struct *mm,unsigned long first,int nr) {pgd_t * page_dir = mm->pgd; +unsigned long last = first + nr;spin_lock (&mm->page_table_lock); page_dir += first;
@@ -173,6 +174,8 @@ }while (--nr);spin_unlock (&mm->page_table_lock); +flush_tlb_pgtables (mm, first *PGDIR_SIZE , last *PGDIR_SIZE ); +/* keep the page table cache within bounds */ check_pgt_cache (); }
mm/mmap.c +0 -1
@@ -1073,7 +1073,6 @@ end_index =pgd_index (last);if (end_index > start_index) {clear_page_tables (mm, start_index, end_index - start_index); -flush_tlb_pgtables (mm, first &PGDIR_MASK , last &PGDIR_MASK ); } }