diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-18 19:32:11 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-18 19:32:11 -0500 |
commit | 28ee5809ff7365d935d217c387ba959b8aa7182f (patch) | |
tree | bf65008ea345162c0873114c53a61809e934bc5c | |
parent | 22943a6a6ba3bbbf5cc11cec409e779374e88107 (diff) | |
parent | 66d0f7ec9f1038452178b1993fc07fd96d30fd38 (diff) |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc
Pull sparc fix from David Miller:
"Sparc32 locking bug fix from Andreas Larsson"
* git://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc:
sparc32: destroy_context() and switch_mm() needs to disable interrupts.
-rw-r--r-- | arch/sparc/mm/srmmu.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/arch/sparc/mm/srmmu.c b/arch/sparc/mm/srmmu.c index be65f035d18a..5cbc96d801ff 100644 --- a/arch/sparc/mm/srmmu.c +++ b/arch/sparc/mm/srmmu.c | |||
@@ -460,10 +460,12 @@ static void __init sparc_context_init(int numctx) | |||
460 | void switch_mm(struct mm_struct *old_mm, struct mm_struct *mm, | 460 | void switch_mm(struct mm_struct *old_mm, struct mm_struct *mm, |
461 | struct task_struct *tsk) | 461 | struct task_struct *tsk) |
462 | { | 462 | { |
463 | unsigned long flags; | ||
464 | |||
463 | if (mm->context == NO_CONTEXT) { | 465 | if (mm->context == NO_CONTEXT) { |
464 | spin_lock(&srmmu_context_spinlock); | 466 | spin_lock_irqsave(&srmmu_context_spinlock, flags); |
465 | alloc_context(old_mm, mm); | 467 | alloc_context(old_mm, mm); |
466 | spin_unlock(&srmmu_context_spinlock); | 468 | spin_unlock_irqrestore(&srmmu_context_spinlock, flags); |
467 | srmmu_ctxd_set(&srmmu_context_table[mm->context], mm->pgd); | 469 | srmmu_ctxd_set(&srmmu_context_table[mm->context], mm->pgd); |
468 | } | 470 | } |
469 | 471 | ||
@@ -986,14 +988,15 @@ int init_new_context(struct task_struct *tsk, struct mm_struct *mm) | |||
986 | 988 | ||
987 | void destroy_context(struct mm_struct *mm) | 989 | void destroy_context(struct mm_struct *mm) |
988 | { | 990 | { |
991 | unsigned long flags; | ||
989 | 992 | ||
990 | if (mm->context != NO_CONTEXT) { | 993 | if (mm->context != NO_CONTEXT) { |
991 | flush_cache_mm(mm); | 994 | flush_cache_mm(mm); |
992 | srmmu_ctxd_set(&srmmu_context_table[mm->context], srmmu_swapper_pg_dir); | 995 | srmmu_ctxd_set(&srmmu_context_table[mm->context], srmmu_swapper_pg_dir); |
993 | flush_tlb_mm(mm); | 996 | flush_tlb_mm(mm); |
994 | spin_lock(&srmmu_context_spinlock); | 997 | spin_lock_irqsave(&srmmu_context_spinlock, flags); |
995 | free_context(mm->context); | 998 | free_context(mm->context); |
996 | spin_unlock(&srmmu_context_spinlock); | 999 | spin_unlock_irqrestore(&srmmu_context_spinlock, flags); |
997 | mm->context = NO_CONTEXT; | 1000 | mm->context = NO_CONTEXT; |
998 | } | 1001 | } |
999 | } | 1002 | } |