]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ARM: 7659/1: mm: make mm->context.id an atomic64_t variable
authorWill Deacon <will.deacon@arm.com>
Thu, 28 Feb 2013 16:47:36 +0000 (17:47 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 14 Mar 2013 18:26:23 +0000 (11:26 -0700)
commit9f400745af791e5c9bf43bd07ee316a02afa13e0
tree1116b8294497fd31231429c00ef0892ef87eb9b1
parent810f63799982d066cdc2781ba366fcee508918a8
ARM: 7659/1: mm: make mm->context.id an atomic64_t variable

commit 8a4e3a9ead7e37ce1505602b564c15da09ac039f upstream.

mm->context.id is updated under asid_lock when a new ASID is allocated
to an mm_struct. However, it is also read without the lock when a task
is being scheduled and checking whether or not the current ASID
generation is up-to-date.

If two threads of the same process are being scheduled in parallel and
the bottom bits of the generation in their mm->context.id match the
current generation (that is, the mm_struct has not been used for ~2^24
rollovers) then the non-atomic, lockless access to mm->context.id may
yield the incorrect ASID.

This patch fixes this issue by making mm->context.id and atomic64_t,
ensuring that the generation is always read consistently. For code that
only requires access to the ASID bits (e.g. TLB flushing by mm), then
the value is accessed directly, which GCC converts to an ldrb.

Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/arm/include/asm/mmu.h
arch/arm/include/asm/mmu_context.h
arch/arm/kernel/asm-offsets.c
arch/arm/mm/context.c