]> Git Repo - linux.git/commit
ARM: 9294/2: vfp: Fix broken softirq handling with instrumentation enabled
authorArd Biesheuvel <[email protected]>
Wed, 12 Apr 2023 08:50:20 +0000 (09:50 +0100)
committerRussell King (Oracle) <[email protected]>
Wed, 12 Apr 2023 09:04:56 +0000 (10:04 +0100)
commitc76c6c4ecbec0deb56a4f9e932b26866024a508f
treee6da7ee937a9e9ee7b29dcab602a24d2ee974714
parent3a2bdad0b46649cc73fb3b3f9e2b91ef97a7fa63
ARM: 9294/2: vfp: Fix broken softirq handling with instrumentation enabled

Commit 62b95a7b44d1 ("ARM: 9282/1: vfp: Manipulate task VFP state with
softirqs disabled") replaced the en/disable preemption calls inside the
VFP state handling code with en/disabling of soft IRQs, which is
necessary to allow kernel use of the VFP/SIMD unit when handling a soft
IRQ.

Unfortunately, when lockdep is enabled (or other instrumentation that
enables TRACE_IRQFLAGS), the disable path implemented in asm fails to
perform the lockdep and RCU related bookkeeping, resulting in spurious
warnings and other badness.

Set let's rework the VFP entry code a little bit so we can make the
local_bh_disable() call from C, with all the instrumentations that
happen to have been configured. Calling local_bh_enable() can be done
from asm, as it is a simple wrapper around __local_bh_enable_ip(), which
is always a callable function.

Link: https://lore.kernel.org/all/[email protected]/
Fixes: 62b95a7b44d1 ("ARM: 9282/1: vfp: Manipulate task VFP state with softirqs disabled")
Signed-off-by: Ard Biesheuvel <[email protected]>
Reviewed-by: Linus Walleij <[email protected]>
Tested-by: Guenter Roeck <[email protected]>
Signed-off-by: Russell King (Oracle) <[email protected]>
arch/arm/include/asm/assembler.h
arch/arm/vfp/entry.S
arch/arm/vfp/vfphw.S
arch/arm/vfp/vfpmodule.c
This page took 0.054652 seconds and 4 git commands to generate.