Simplifying siginfo users
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest22 patches 24 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/22 | signal/alpha: Use send_sig_fault where appropriatearch/ arch/ arch/ | +24 | 0 replies | |
| 2/22 | signal/alpha: Use force_sig_fault where appropriatearch/ arch/ | +3 | 0 replies | |
| 3/22 | signal/c6x: Use force_sig_fault where appropriatearch/ | +2 | 0 replies | |
| 4/22 | signal/hexagon: Use force_sig_fault as appropriatearch/ arch/ | +9 | 0 replies | |
| 5/22 | signal/m68k: Use force_sig_fault where appropriatearch/ arch/ | +36 | 0 replies | |
| 6/22 | signal/microblaze: Remove the commented out force_sig_info in do_page_faultarch/ | +0 | 0 replies | |
| 7/22 | signal/microblaze: Use force_sig_fault where appropriatearch/ arch/ | +2 | 0 replies | |
| 8/22 | signal/mips: Use force_sig_fault where appropriatearch/ arch/ | +23 | 4 replies | |
| 9/22 | signal/nds32: Use force_sig_fault where appropriatearch/ arch/ | +9 | 3 replies | |
| 10/22 | signal/nios2: Use force_sig_fault where appropriatearch/ | +1 | 0 replies | |
| 11/22 | signal/openrisc: Use force_sig_fault where appropriatearch/ arch/ | +9 | 0 replies | |
| 12/22 | signal/parisc: Use force_sig_mceerr where appropriatearch/ | +15 | 0 replies | |
| 13/22 | signal/parisc: Use force_sig_fault where appropriatearch/ arch/ arch/ +2 more | +39 | 1 reply | |
| 14/22 | signal/riscv: Use force_sig_fault where appropriatearch/ | +1 | 4 replies | |
| 15/22 | signal/s390: Use force_sig_fault where appropriatearch/ arch/ | +12 | 1 reply | |
| 16/22 | signal/sh: Use force_sig_fault where appropriatearch/ arch/ arch/ | +8 | 4 replies | |
| 17/22 | signal/sparc: Use send_sig_fault where appropriatearch/ arch/ | +20 | 0 replies | |
| 18/22 | signal/sparc: Use force_sig_fault where appropriatearch/ arch/ arch/ +4 more | +37 | 0 replies | |
| 19/22 | signal/um: Use force_sig_fault in relay_signal.arch/ | +13 | 13 replies | |
| 20/22 | signal/um: Use force_sig_fault where appropriatearch/ arch/ | +11 | 0 replies | |
| 21/22 | signal/xtensa: Consistenly use SIGBUS in do_unaligned_userarch/ | +1 | 1 reply | |
| 22/22 | signal/xtensa: Use force_sig_fault where appropriatearch/ arch/ | +6 | 1 reply |
Reviews trailers per patch
6 of 22 patches have a Reviewed-by or Acked-by.
| Patch | Rich Felker | Max Filippov | Helge Deller | Martin Schwidefsky | Palmer Dabbelt |
|---|---|---|---|---|---|
| 1/22 | |||||
| 2/22 | |||||
| 3/22 | |||||
| 4/22 | |||||
| 5/22 | |||||
| 6/22 | |||||
| 7/22 | |||||
| 8/22 | |||||
| 9/22 | |||||
| 10/22 | |||||
| 11/22 | |||||
| 12/22 | |||||
| 13/22 | A-b | ||||
| 14/22 | A-b | ||||
| 15/22 | A-b | ||||
| 16/22 | A-b | ||||
| 17/22 | |||||
| 18/22 | |||||
| 19/22 | |||||
| 20/22 | |||||
| 21/22 | A-b | ||||
| 22/22 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 5 reviewers.
Cover letter [REVIEW][PATCH 00/22] Simplifying siginfo users
This patchset is almost completely updating the various pieces of architecture code from filling out a struct siginfo and then calling force_sig_info. To using the helper force_sig_fault instead. Which takes the values that would have gone in siginfo as arguments.
As the diffstat below shows this noticeable reduces the amount of code.
There are a couple of minor and straight forward bug fixes as well that I came across while working on the aforementioned changes.
This applies on my last round of siginfo changes simply to remove the possibility of conflicts. The changes should be semantically separate.
I intend to merge these changes through my siginfo tree. If you have any objections or would prefer to carry these changes in your architecture trees please let me know.
The changes are also available at:
git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm/user-namespace.git siginfo-review2
Eric W. Biederman (22):
signal/alpha: Use send_sig_fault where appropriate signal/alpha: Use force_sig_fault where appropriate signal/c6x: Use force_sig_fault where appropriate signal/hexagon: Use force_sig_fault as appropriate signal/m68k: Use force_sig_fault where appropriate signal/microblaze: Remove the commented out force_sig_info in do_page_fault signal/microblaze: Use force_sig_fault where appropriate signal/mips: Use force_sig_fault where appropriate signal/nds32: Use force_sig_fault where appropriate signal/nios2: Use force_sig_fault where appropriate signal/openrisc: Use force_sig_fault where appropriate signal/parisc: Use force_sig_mceerr where appropriate signal/parisc: Use force_sig_fault where appropriate signal/riscv: Use force_sig_fault where appropriate signal/s390: Use force_sig_fault where appropriate
signal/sh: Use force_sig_fault where appropriate
signal/sparc: Use send_sig_fault where appropriate signal/sparc: Use force_sig_fault where appropriate
signal/um: Use force_sig_fault in relay_signal.
signal/um: Use force_sig_fault where appropriate signal/xtensa: Consistenly use SIGBUS in do_unaligned_user signal/xtensa: Use force_sig_fault where appropriate
arch/alpha/kernel/osf_sys.c | 10 +-- arch/alpha/kernel/signal.c | 22 +----- arch/alpha/kernel/traps.c | 80 +++++--------------- arch/alpha/mm/fault.c | 15 +--- arch/c6x/kernel/traps.c | 10 +-- arch/hexagon/kernel/traps.c | 10 +-- arch/hexagon/mm/vm_fault.c | 21 ++---- arch/m68k/kernel/traps.c | 60 +++++++-------- arch/m68k/mm/fault.c | 25 +++---- arch/microblaze/kernel/exceptions.c | 9 +-- arch/microblaze/mm/fault.c | 14 +--- arch/mips/kernel/traps.c | 65 +++++----------- arch/mips/mm/fault.c | 19 ++--- arch/nds32/kernel/traps.c | 20 +---- arch/nds32/mm/fault.c | 19 ++--- arch/nios2/kernel/traps.c | 9 +-- arch/openrisc/kernel/traps.c | 33 +------- arch/openrisc/mm/fault.c | 19 ++--- arch/parisc/kernel/ptrace.c | 11 +-- arch/parisc/kernel/traps.c | 63 +++++----------- arch/parisc/kernel/unaligned.c | 16 +--- arch/parisc/math-emu/driver.c | 9 +-- arch/parisc/mm/fault.c | 55 +++++++------- arch/riscv/kernel/traps.c | 9 +-- arch/s390/kernel/traps.c | 32 ++------ arch/s390/mm/fault.c | 23 ++---- arch/sh/kernel/traps_32.c | 19 ++--- arch/sh/math-emu/math.c | 9 +-- arch/sh/mm/fault.c | 10 +-- arch/sparc/kernel/process_64.c | 10 +-- arch/sparc/kernel/sys_sparc_32.c | 9 +-- arch/sparc/kernel/sys_sparc_64.c | 8 +- arch/sparc/kernel/traps_32.c | 114 +++++----------------------- arch/sparc/kernel/traps_64.c | 145 +++++++----------------------------- arch/sparc/kernel/unaligned_32.c | 12 +-- arch/sparc/mm/fault_32.c | 13 +--- arch/sparc/mm/fault_64.c | 9 +-- arch/um/kernel/ptrace.c | 13 +--- arch/um/kernel/trap.c | 54 ++++++-------- arch/xtensa/kernel/traps.c | 10 +-- arch/xtensa/mm/fault.c | 19 ++--- 41 files changed, 280 insertions(+), 852 deletions(-)
Eric