[v3] arm64/signal: Signal handling cleanups

[PATCH v3 0/7] arm64/signal: Signal handling cleanups

Posted by Mark Brown 2 years, 7 months ago

This series collects a number of small cleanups to the signal handling
code which removes redundant validation of size information and avoids
reading the same data from userspace twice.

There are some overlaps with both the TPIDR2 signal handling and SME2
serieses which are also in flight, applying this will require
adjustments in those serieses and vice versa.

Changes in v3:
- Rebase onto arm64/for-next/core, updating for the addition of ZT and
  TPIDR2 signal frames.
- Link to v2: https://lore.kernel.org/r/20221212-arm64-signal-cleanup-v2-0-14a8f3e088b7@kernel.org
v2:
 - Rebase onto v6.2-rc1

Signed-off-by: Mark Brown <broonie@kernel.org>
---
Mark Brown (7):
      arm64/signal: Don't redundantly verify FPSIMD magic
      arm64/signal: Remove redundant size validation from parse_user_sigframe()
      arm64/signal: Make interface for restore_fpsimd_context() consistent
      arm64/signal: Avoid rereading context frame sizes
      arm64/signal: Only read new data when parsing the SVE context
      arm64/signal: Only read new data when parsing the ZA context
      arm64/signal: Only read new data when parsing the ZT context

 arch/arm64/kernel/signal.c | 121 +++++++++++++++++++++++----------------------
 1 file changed, 61 insertions(+), 60 deletions(-)
---
base-commit: 8154ffb7a51882c00730952ed21d80ed76f165d7
change-id: 20221212-arm64-signal-cleanup-bcd7272de5a9

Best regards,
-- 
Mark Brown <broonie@kernel.org>

Re: [PATCH v3 0/7] arm64/signal: Signal handling cleanups

Posted by Catalin Marinas 2 years, 7 months ago

On Tue, 31 Jan 2023 22:20:38 +0000, Mark Brown wrote:
> This series collects a number of small cleanups to the signal handling
> code which removes redundant validation of size information and avoids
> reading the same data from userspace twice.
> 
> There are some overlaps with both the TPIDR2 signal handling and SME2
> serieses which are also in flight, applying this will require
> adjustments in those serieses and vice versa.
> 
> [...]

Applied to arm64 (for-next/signal), thanks!

[1/7] arm64/signal: Don't redundantly verify FPSIMD magic
      https://git.kernel.org/arm64/c/92f14518cc43
[2/7] arm64/signal: Remove redundant size validation from parse_user_sigframe()
      https://git.kernel.org/arm64/c/0eb23720f29e
[3/7] arm64/signal: Make interface for restore_fpsimd_context() consistent
      https://git.kernel.org/arm64/c/4e4e93045fe1
[4/7] arm64/signal: Avoid rereading context frame sizes
      https://git.kernel.org/arm64/c/b57682b31558
[5/7] arm64/signal: Only read new data when parsing the SVE context
      https://git.kernel.org/arm64/c/f3ac48aa3a58
[6/7] arm64/signal: Only read new data when parsing the ZA context
      https://git.kernel.org/arm64/c/24d68345a02a
[7/7] arm64/signal: Only read new data when parsing the ZT context
      https://git.kernel.org/arm64/c/ad678be42387

-- 
Catalin