siginfo bugfixes and cleanups
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest17 patches 2 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/17 | signal/alpha: Document a conflict with SI_USER for SIGFPEarch/ | +1 | 0 replies | |
| 2/17 | sparc: fix compat siginfo ABI regressionarch/ arch/ arch/ | +2 | 0 replies | |
| 3/17 | signal/sh: Use force_sig_fault in hw_breakpoint_handlerarch/ | +2 | 0 replies | |
| 4/17 | signal/nds32: Use force_sig in unhandled_interruption and unhandled_exceptionsarch/ | +2 | 1 reply | |
| 5/17 | signal/nds32: Use force_sig(SIGILL) in do_revisnarch/ | +1 | 3 replies | |
| 6/17 | signal: Ensure every siginfo we send has all bits initializedarch/ arch/ arch/ +65 more | +158 | 0 replies | |
| 7/17 | signal: Reduce copy_siginfo_to_user to just copy_to_userkernel/ | +2 | 0 replies | |
| 8/17 | signal: Stop special casing TRAP_FIXME and FPE_FIXME in siginfo_layoutkernel/ | +0 | 0 replies | |
| 9/17 | signal: Remove SEGV_BNDERR ifdefskernel/ | +0 | 0 replies | |
| 10/17 | signal: Remove ifdefs for BUS_MCEERR_AR and BUS_MCEERR_AOfs/ kernel/ | +10 | 0 replies | |
| 11/17 | signal/alpha: Replace FPE_FIXME with FPE_FLTUNKarch/ arch/ arch/ | +2 | 0 replies | |
| 12/17 | signal/ia64: Replace FPE_FIXME with FPE_FLTUNKarch/ arch/ | +2 | 0 replies | |
| 13/17 | signal/powerpc: Replace FPE_FIXME with FPE_FLTUNKarch/ arch/ | +3 | 0 replies | |
| 14/17 | signal/unicore32: Use FPE_FLTUNK instead of 0 in ucf64_raise_sigfpearch/ | +3 | 0 replies | |
| 15/17 | signal: Add TRAP_UNK si_code for undiagnosted trap exceptionsarch/ include/ | +3 | 0 replies | |
| 16/17 | signal/alpha: Replace TRAP_FIXME with TRAP_UNKarch/ arch/ | +2 | 0 replies | |
| 17/17 | signal/powerpc: Replace TRAP_FIXME with TRAP_UNKarch/ arch/ | +2 | 0 replies |
Reviews trailers per patch
None of the 17 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | David Miller | Rich Felker | Max Filippov | Helge Deller | Martin Schwidefsky | Palmer Dabbelt |
|---|---|---|---|---|---|---|
| Series | A-b | A-b A-b | A-b | A-b | A-b | |
| 1/17 | ||||||
| 2/17 | Conceptually-Acked-By | |||||
| 3/17 | ||||||
| 4/17 | ||||||
| 5/17 | ||||||
| 6/17 | ||||||
| 7/17 | ||||||
| 8/17 | ||||||
| 9/17 | ||||||
| 10/17 | ||||||
| 11/17 | ||||||
| 12/17 | ||||||
| 13/17 | ||||||
| 14/17 | ||||||
| 15/17 | ||||||
| 16/17 | ||||||
| 17/17 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.
Cover letter [REVIEW][PATCH 00/17] siginfo bugfixes and cleanups
Mostly this dealing with aliases of SI_USER, that make it impossible for userspace to tell what kind of siginfo it has received.
Also in this series is the change to ensure we have siginfo initialized so I can rip out the switch in copy_siginfo_to_user for better performance, and to ensure better backwards compatibility as siginfo as generated in the callers is passed to userspace.
There are also fixes for a few other siginfo bugs I have found.
There are cleanups of ifdefs on SEGV_BNDERR and BUS_MCERR_AX that are no longer necessary.
I plan to merge these bug fixes through my siginfo tree, but if you object to them or would prefer to have them go through your arch tree please let me know.
The changes are avaiable as a totality at:
git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm/user-namespace.git siginfo-review
Dmitry V. Levin (1):
sparc: fix compat siginfo ABI regression
Eric W. Biederman (16):
signal/alpha: Document a conflict with SI_USER for SIGFPE signal/sh: Use force_sig_fault in hw_breakpoint_handler signal/nds32: Use force_sig in unhandled_interruption and unhandled_exceptions signal/nds32: Use force_sig(SIGILL) in do_revisn
signal: Ensure every siginfo we send has all bits initialized
signal: Reduce copy_siginfo_to_user to just copy_to_user
signal: Stop special casing TRAP_FIXME and FPE_FIXME in siginfo_layout
signal: Remove SEGV_BNDERR ifdefs
signal: Remove ifdefs for BUS_MCEERR_AR and BUS_MCEERR_AO signal/alpha: Replace FPE_FIXME with FPE_FLTUNK
signal/ia64: Replace FPE_FIXME with FPE_FLTUNK
signal/powerpc: Replace FPE_FIXME with FPE_FLTUNK signal/unicore32: Use FPE_FLTUNK instead of 0 in ucf64_raise_sigfpe
signal: Add TRAP_UNK si_code for undiagnosted trap exceptions signal/alpha: Replace TRAP_FIXME with TRAP_UNK
signal/powerpc: Replace TRAP_FIXME with TRAP_UNK
arch/alpha/include/uapi/asm/siginfo.h | 14 ---- arch/alpha/kernel/osf_sys.c | 3 +- arch/alpha/kernel/signal.c | 2 + arch/alpha/kernel/traps.c | 11 ++- arch/alpha/mm/fault.c | 2 + arch/arc/mm/fault.c | 2 + arch/arm/kernel/ptrace.c | 1 + arch/arm/kernel/swp_emulate.c | 1 + arch/arm/kernel/traps.c | 5 ++ arch/arm/mm/alignment.c | 1 + arch/arm/mm/fault.c | 4 + arch/arm/vfp/vfpmodule.c | 3 +- arch/arm64/kernel/fpsimd.c | 2 +- arch/arm64/kernel/sys_compat.c | 1 + arch/arm64/kernel/traps.c | 1 + arch/arm64/mm/fault.c | 18 +++-- arch/c6x/kernel/traps.c | 1 + arch/hexagon/kernel/traps.c | 1 + arch/hexagon/mm/vm_fault.c | 1 + arch/ia64/include/uapi/asm/siginfo.h | 7 -- arch/ia64/kernel/brl_emu.c | 1 + arch/ia64/kernel/signal.c | 2 + arch/ia64/kernel/traps.c | 31 ++++++-- arch/ia64/kernel/unaligned.c | 1 + arch/ia64/mm/fault.c | 4 +- arch/m68k/kernel/traps.c | 2 + arch/microblaze/kernel/exceptions.c | 1 + arch/microblaze/mm/fault.c | 4 +- arch/mips/mm/fault.c | 1 + arch/nds32/kernel/traps.c | 19 ++--- arch/nds32/mm/fault.c | 1 + arch/nios2/kernel/traps.c | 1 + arch/openrisc/kernel/traps.c | 5 +- arch/openrisc/mm/fault.c | 1 + arch/parisc/kernel/ptrace.c | 1 + arch/parisc/kernel/traps.c | 2 + arch/parisc/kernel/unaligned.c | 1 + arch/parisc/math-emu/driver.c | 1 + arch/parisc/mm/fault.c | 1 + arch/powerpc/include/uapi/asm/siginfo.h | 15 ---- arch/powerpc/kernel/process.c | 1 + arch/powerpc/kernel/traps.c | 13 ++-- arch/powerpc/mm/fault.c | 1 + arch/powerpc/platforms/cell/spufs/fault.c | 2 +- arch/riscv/kernel/traps.c | 1 + arch/s390/kernel/traps.c | 5 +- arch/s390/mm/fault.c | 2 + arch/sh/kernel/hw_breakpoint.c | 9 +-- arch/sh/kernel/traps_32.c | 2 + arch/sh/math-emu/math.c | 1 + arch/sh/mm/fault.c | 1 + arch/sparc/include/uapi/asm/siginfo.h | 7 -- arch/sparc/kernel/process_64.c | 1 + arch/sparc/kernel/sys_sparc_32.c | 1 + arch/sparc/kernel/traps_32.c | 12 ++- arch/sparc/kernel/traps_64.c | 16 +++- arch/sparc/kernel/unaligned_32.c | 1 + arch/sparc/mm/fault_32.c | 1 + arch/sparc/mm/fault_64.c | 1 + arch/um/kernel/trap.c | 2 + arch/unicore32/kernel/fpu-ucf64.c | 8 +- arch/unicore32/mm/fault.c | 3 + arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- arch/x86/kernel/ptrace.c | 2 +- arch/x86/kernel/signal_compat.c | 2 +- arch/x86/kernel/traps.c | 3 + arch/x86/kernel/umip.c | 1 + arch/x86/kvm/mmu.c | 1 + arch/x86/mm/fault.c | 1 + arch/xtensa/kernel/traps.c | 1 + arch/xtensa/mm/fault.c | 1 + fs/signalfd.c | 15 +--- include/linux/ptrace.h | 1 - include/linux/tracehook.h | 1 + include/uapi/asm-generic/siginfo.h | 3 +- kernel/signal.c | 123 +++--------------------------- virt/kvm/arm/mmu.c | 1 + 77 files changed, 194 insertions(+), 231 deletions(-)