OSDN Git Service

arm64/sve: Better handle failure to allocate SVE register storage
authorMark Brown <broonie@kernel.org>
Tue, 24 Aug 2021 15:34:17 +0000 (16:34 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Tue, 24 Aug 2021 15:44:30 +0000 (16:44 +0100)
commit7559b7d7d651d397debbcd838bd49ec4b9e0a4a4
treebe2dc4ad68f306fdb35c22c47cb3ed6663a462ee
parente3849765037b85e61b2432ded488ee9fb3ff126d
arm64/sve: Better handle failure to allocate SVE register storage

Currently we "handle" failure to allocate the SVE register storage by
doing a BUG_ON() and hoping for the best. This is obviously not great and
the memory allocation failure will already be loud enough without the
BUG_ON(). As the comment says it is a corner case but let's try to do a bit
better, remove the BUG_ON() and add code to handle the failure in the
callers.

For the ptrace and signal code we can return -ENOMEM gracefully however
we have no real error reporting path available to us for the SVE access
trap so instead generate a SIGKILL if the allocation fails there. This
at least means that we won't try to soldier on and end up trying to
access the nonexistant state and while it's obviously not ideal for
userspace SIGKILL doesn't allow any handling so minimises the ABI
impact, making it easier to improve the interface later if we come up
with a better idea.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20210824153417.18371-1-broonie@kernel.org
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/kernel/fpsimd.c
arch/arm64/kernel/ptrace.c
arch/arm64/kernel/signal.c