ARM: 7417/1: vfp: ensure preemption is disabled when enabling VFP access
authorWill Deacon <will.deacon@arm.com>
Fri, 11 May 2012 16:42:37 +0000 (17:42 +0100)
committer黄涛 <huangtao@rock-chips.com>
Fri, 24 May 2013 08:13:38 +0000 (16:13 +0800)
commit2a4ff059eb52393e2d6e7daa668eb43a782579c8
tree3034685c60e7f303b02cf7d06676bbad8d184378
parent69c56ee242d07612459d42e2f2decc1c59c3f04c
ARM: 7417/1: vfp: ensure preemption is disabled when enabling VFP access

The vfp_enable function enables access to the VFP co-processor register
space (cp10 and cp11) on the current CPU and must be called with
preemption disabled. Unfortunately, the vfp_init late initcall does not
disable preemption and can lead to an oops during boot if thread
migration occurs at the wrong time and we end up attempting to access
the FPSID on a CPU with VFP access disabled.

This patch fixes the initcall to call vfp_enable from a non-preemptible
context on each CPU and adds a BUG_ON(preemptible) to ensure that any
similar problems are easily spotted in the future.

Cc: stable@vger.kernel.org
Reported-by: Hyungwoo Yang <hwoo.yang@gmail.com>
Signed-off-by: Hyungwoo Yang <hyungwooy@nvidia.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/vfp/vfpmodule.c