diff options
| author | Dave Kleikamp <dave.kleikamp@oracle.com> | 2013-06-18 16:05:36 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-06-19 11:10:30 +0200 |
| commit | 23a01138efe216f8084cfaa74b0b90dd4b097441 (patch) | |
| tree | 3a13b9b2383b9077e54d998abb2e41365abe7aa6 /arch/sparc/mm | |
| parent | sparc,leon: Convert to use devm_ioremap_resource (diff) | |
| download | linux-23a01138efe216f8084cfaa74b0b90dd4b097441.tar.xz linux-23a01138efe216f8084cfaa74b0b90dd4b097441.zip | |
sparc: tsb must be flushed before tlb
This fixes a race where a cpu may re-load a tlb from a stale tsb right
after it has been flushed by a remote function call.
I still see some instability when stressing the system with parallel
kernel builds while creating memory pressure by writing to
/proc/sys/vm/nr_hugepages, but this patch improves the stability
significantly.
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
Acked-by: Bob Picco <bob.picco@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch/sparc/mm')
| -rw-r--r-- | arch/sparc/mm/tlb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/sparc/mm/tlb.c b/arch/sparc/mm/tlb.c index 83d89bcb44af..37e7bc4c95b3 100644 --- a/arch/sparc/mm/tlb.c +++ b/arch/sparc/mm/tlb.c @@ -85,8 +85,8 @@ static void tlb_batch_add_one(struct mm_struct *mm, unsigned long vaddr, } if (!tb->active) { - global_flush_tlb_page(mm, vaddr); flush_tsb_user_page(mm, vaddr); + global_flush_tlb_page(mm, vaddr); goto out; } |