Skip to content

Commit

Permalink
Revert "ARM: 6944/1: mm: allow ASID 0 to be allocated to tasks"
Browse files Browse the repository at this point in the history
This reverts commit 45b9523.

Will Deacon reports that:

 In 52af9c6 ("ARM: 6943/1: mm: use TTBR1 instead of reserved context ID")
 I updated the ASID rollover code to use only the kernel page tables
 whilst updating the ASID.

 Unfortunately, the code to restore the user page tables was part of a
 later patch which isn't yet in mainline, so this leaves the code
 quite broken.

We're also in the process of eliminating __ARCH_WANT_INTERRUPTS_ON_CTXSW
from ARM, so lets revert these until we can properly sort out what we're
doing with the context switching.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
  • Loading branch information
Russell King committed Jun 9, 2011
1 parent 07989b7 commit a0a54d3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions arch/arm/mm/context.c
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ static void reset_context(void *info)
return;

smp_rmb();
asid = cpu_last_asid + cpu;
asid = cpu_last_asid + cpu + 1;

flush_context();
set_mm_context(mm, asid);
Expand Down Expand Up @@ -143,13 +143,13 @@ void __new_context(struct mm_struct *mm)
* to start a new version and flush the TLB.
*/
if (unlikely((asid & ~ASID_MASK) == 0)) {
asid = cpu_last_asid + smp_processor_id();
asid = cpu_last_asid + smp_processor_id() + 1;
flush_context();
#ifdef CONFIG_SMP
smp_wmb();
smp_call_function(reset_context, NULL, 1);
#endif
cpu_last_asid += NR_CPUS - 1;
cpu_last_asid += NR_CPUS;
}

set_mm_context(mm, asid);
Expand Down

0 comments on commit a0a54d3

Please sign in to comment.