]> git.baikalelectronics.ru Git - kernel.git/commit
arm64: entry: unmask IRQ in el0_sp()
authorMark Rutland <mark.rutland@arm.com>
Fri, 28 Feb 2020 14:59:42 +0000 (14:59 +0000)
committerCatalin Marinas <catalin.marinas@arm.com>
Wed, 11 Mar 2020 14:34:28 +0000 (14:34 +0000)
commita0a51c76f77dd6f0093293ee5140d53be685949f
tree77a43ee032b5aa7e8f6e02a12c3c28f24d7585b8
parentc358e56ca586887c89540b11dae82f43b8171db9
arm64: entry: unmask IRQ in el0_sp()

Currently, the EL0 SP alignment handler masks IRQs unnecessarily. It
does so due to historic code sharing of the EL0 SP and PC alignment
handlers, and branch predictor hardening applicable to the EL0 SP
handler.

We began masking IRQs in the EL0 SP alignment handler in commit:

  6a77cd868db983ad ("arm64: entry: Apply BP hardening for high-priority synchronous exception")

... as this shared code with the EL0 PC alignment handler, and branch
predictor hardening made it necessary to disable IRQs for early parts of
the EL0 PC alignment handler. It was not necessary to mask IRQs during
EL0 SP alignment exceptions, but it was not considered harmful to do so.

This masking was carried forward into C code in commit:

  7677ffc529dee730 ("arm64: entry: convert el0_sync to C")

... where the SP/PC cases were split into separate handlers, and the
masking duplicated.

Subsequently the EL0 PC alignment handler was refactored to perform
branch predictor hardening before unmasking IRQs, in commit:

  d68d5ad66e0799f9 ("arm64: entry-common: don't touch daif before bp-hardening")

... but the redundant masking of IRQs was not removed from the EL0 SP
alignment handler.

Let's do so now, and make it interruptible as with most other
synchronous exception handlers.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Cc: Will Deacon <will@kernel.org>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Reviewed-by: James Morse <james.morse@arm.com>
arch/arm64/kernel/entry-common.c