virtually mapped stacks and thread_info cleanup
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v5 Latest32 patches 26 repliesIn review
- v429 patches 55 repliesSuperseded
Changes in v5 from the cover letter
- Fix kthread (Oleg)
- Tidy up some changelongs and fold some patches (Borislav, Josh)
- Add "x86/mm/64: In vmalloc_fault(), use CR3 instead of current->active_mm"
- Make VMAP_STACK depend on !KASAN (not worth waiting for the fix, I think)
Patches in v5 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/32 | bluetooth: Switch SMP to crypto_cipher_encrypt_one()net/ | +28 | 3 replies | |
| 2/32 | x86/mm/hotplug: Don't remove PGD entries in remove_pagetable()arch/ | +0 | 0 replies | |
| 3/32 | x86/cpa: In populate_pgd, don't set the pgd entry until it's populatedarch/ | +6 | 10 replies | |
| 4/32 | x86/mm: Remove kernel_unmap_pages_in_pgd() and efi_cleanup_page_tables()arch/ arch/ arch/ +3 more | +0 | 0 replies | |
| 5/32 | mm: Track NR_KERNEL_STACK in KiB instead of number of stacksdrivers/ fs/ include/ +2 more | +6 | 0 replies | |
| 6/32 | mm: Fix memcg stack accounting for sub-page stacksinclude/ kernel/ mm/ | +10 | 0 replies | |
| 7/32 | fork: Add generic vmalloced stack supportarch/ arch/ include/ +1 more | +125 | 0 replies | |
| 8/32 | dma-api: Teach the "DMA-from-stack" check about vmapped stackslib/ | +33 | 0 replies | |
| 9/32 | x86/dumpstack: When OOPSing, rewind the stack before do_exit()arch/ arch/ arch/ | +31 | 0 replies | |
| 10/32 | x86/dumpstack: Honor supplied @regs argarch/ arch/ | +6 | 0 replies | |
| 11/32 | x86/dumpstack: Try harder to get a call trace on stack overflowarch/ | +9 | 0 replies | |
| 12/32 | x86/dumpstack/64: Handle faults when printing the "Stack:" part of an OOPSarch/ | +10 | 0 replies | |
| 13/32 | x86/mm/64: In vmalloc_fault(), use CR3 instead of current->active_mmarch/ | +1 | 2 replies | |
| 14/32 | x86/mm/64: Enable vmapped stacksarch/ arch/ arch/ +1 more | +105 | 5 replies | |
| 15/32 | x86/mm: Improve stack-overflow #PF handlingarch/ arch/ arch/ | +43 | 0 replies | |
| 16/32 | x86: Move uaccess_err and sig_on_uaccess_err to thread_structarch/ arch/ arch/ +3 more | +10 | 0 replies | |
| 17/32 | x86: Move addr_limit to thread_structarch/ arch/ arch/ +9 more | +40 | 0 replies | |
| 18/32 | signal: Consolidate {TS,TLF}_RESTORE_SIGMASK codearch/ arch/ arch/ +7 more | +63 | 2 replies | |
| 19/32 | x86/smp: Remove stack_smp_processor_id()arch/ arch/ arch/ | +1 | 0 replies | |
| 20/32 | x86/smp: Remove unnecessary initialization of thread_info::cpuarch/ | +0 | 0 replies | |
| 21/32 | x86/asm: Move 'status' from struct thread_info to struct thread_structarch/ arch/ arch/ +5 more | +26 | 0 replies | |
| 22/32 | kdb: Use task_cpu() instead of task_thread_info()->cpuinclude/ | +1 | 0 replies | |
| 23/32 | printk: When dumping regs, show the stack, not thread_infokernel/ | +2 | 0 replies | |
| 24/32 | x86/entry: Get rid of pt_regs_to_thread_info()arch/ | +7 | 0 replies | |
| 25/32 | um: Stop conflating task_struct::stack with thread_infoarch/ | +4 | 0 replies | |
| 26/32 | sched: Allow putting thread_info into task_structinclude/ include/ include/ +3 more | +74 | 0 replies | |
| 27/32 | x86: Move thread_info into task_structarch/ arch/ arch/ +4 more | +13 | 0 replies | |
| 28/32 | sched: Add try_get_task_stack() and put_task_stack()include/ init/ | +19 | 0 replies | |
| 29/32 | kthread: to_live_kthread() needs try_get_task_stack()kernel/ | +6 | 0 replies | |
| 30/32 | x86/dumpstack: Pin the target stack in save_stack_trace_tsk()arch/ | +5 | 0 replies | |
| 31/32 | sched: Free the stack early if CONFIG_THREAD_INFO_IN_TASKinclude/ include/ kernel/ +1 more | +55 | 0 replies | |
| 32/32 | fork: Cache two thread stacks per cpu if CONFIG_VMAP_STACK is setkernel/ | +53 | 0 replies |
Reviews trailers per patch
7 of 32 patches have a Reviewed-by or Acked-by.
| Patch | Herbert Xu | Johan Hedberg | Matt Fleming | Borislav Petkov | Josh Poimboeuf | Vladimir Davydov | Michal Hocko |
|---|---|---|---|---|---|---|---|
| 1/32 | A-b | Acked-and-tested-by | |||||
| 2/32 | |||||||
| 3/32 | |||||||
| 4/32 | R-b | A-b | |||||
| 5/32 | R-b | R-b | A-b | ||||
| 6/32 | R-b | R-b | A-b | ||||
| 7/32 | |||||||
| 8/32 | |||||||
| 9/32 | R-b | ||||||
| 10/32 | |||||||
| 11/32 | R-b | ||||||
| 12/32 | R-b | ||||||
| 13/32 | |||||||
| 14/32 | |||||||
| 15/32 | |||||||
| 16/32 | |||||||
| 17/32 | |||||||
| 18/32 | |||||||
| 19/32 | |||||||
| 20/32 | |||||||
| 21/32 | |||||||
| 22/32 | |||||||
| 23/32 | |||||||
| 24/32 | |||||||
| 25/32 | |||||||
| 26/32 | |||||||
| 27/32 | |||||||
| 28/32 | |||||||
| 29/32 | |||||||
| 30/32 | |||||||
| 31/32 | |||||||
| 32/32 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 7 reviewers.
Cover letter [PATCH v5 00/32] virtually mapped stacks and thread_info cleanup
Hi all-
Since the dawn of time, a kernel stack overflow has been a real PITA to debug, has caused nondeterministic crashes some time after the actual overflow, and has generally been easy to exploit for root.
With this series, arches can enable HAVE_ARCH_VMAP_STACK. Arches that enable it (just x86 for now) get virtually mapped stacks with guard pages. This causes reliable faults when the stack overflows.
If the arch implements it well, we get a nice OOPS on stack overflow (as opposed to panicing directly or otherwise exploding badly). On x86, the OOPS is nice, has a usable call trace, and the overflowing task is killed cleanly.
This series (starting with v4) also extensively cleans up thread_info. thread_info has been partially redundant with thread_struct for a long time -- both are places for arch code to add additional per-task variables. thread_struct is much cleaner:
it's always in task_struct, and there's nothing particularly magical about it. So this series contains a bunch of cleanups on x86 to move almost everything from thread_info to thread_struct (which, even by itself, deletes more code than it adds) and to remove x86's dependence on thread_info's position on the stack. Then it opts x86 into a new config option THREAD_INFO_IN_TASK to get rid of arch-specific thread_info entirely and simply embed a defanged thread_info (containing only flags) and 'int cpu' into task_struct.
Once thread_info stops being magical, there's another benefit: we can free the thread stack as soon as the task is dead (without waiting for RCU) and then, if vmapped stacks are in use, cache the entire stack for reuse on the same cpu.
This seems to be an overall speedup of about 0.5-1 µs per pthread_create/join in a simple test -- a percpu cache of vmalloced stacks appears to be a bit faster than a high-order stack allocation, at least when the cache hits. (I expect that workloads with a low cache hit rate are likely to be dominated by other effects anyway.)
This does not address interrupt stacks.
It's worth noting that s390 has an arch-specific gcc feature that detects stack overflows by adjusting function prologues. Arches with features like that may wish to avoid using vmapped stacks to minimize the performance hit.
Known issues: - tcp md5, rxkad, virtio_net, and virtio_console will have issues. Eric Dumazet has a patch for tcp md5, and Michael Tsirkin says he'll fix virtio_net and virtio_console. rxkad will be fixed via net-next.
Changes from v4:
- Fix kthread (Oleg)
- Tidy up some changelongs and fold some patches (Borislav, Josh)
- Add "x86/mm/64: In vmalloc_fault(), use CR3 instead of current->active_mm"
- Make VMAP_STACK depend on !KASAN (not worth waiting for the fix, I think)
Changes from v3:
- Minor cleanups
- Rebased onto Linus' tree
- All the thread_info stuff is new
Changes from v2:
- Delete kerne_unmap_pages_in_pgd rather than hardening it (Borislav)
- Fix sub-page stack accounting better (Josh)
Changes from v1:
- Fix rewind_stack_and_do_exit (Josh)
- Fix deadlock under load
- Clean up generic stack vmalloc code
- Many other minor fixes
Andy Lutomirski (28):
bluetooth: Switch SMP to crypto_cipher_encrypt_one() x86/cpa: In populate_pgd, don't set the pgd entry until it's populated x86/mm: Remove kernel_unmap_pages_in_pgd() and
efi_cleanup_page_tables()
mm: Track NR_KERNEL_STACK in KiB instead of number of stacks
mm: Fix memcg stack accounting for sub-page stacks
fork: Add generic vmalloced stack support
dma-api: Teach the "DMA-from-stack" check about vmapped stacks x86/dumpstack: When OOPSing, rewind the stack before do_exit() x86/dumpstack: Honor supplied @regs arg
x86/dumpstack: Try harder to get a call trace on stack overflow x86/dumpstack/64: Handle faults when printing the "Stack:" part of an
OOPS
x86/mm/64: In vmalloc_fault(), use CR3 instead of current->active_mm x86/mm/64: Enable vmapped stacks
x86/mm: Improve stack-overflow #PF handling
x86: Move uaccess_err and sig_on_uaccess_err to thread_struct
x86: Move addr_limit to thread_struct
signal: Consolidate {TS,TLF}_RESTORE_SIGMASK code
x86/smp: Remove stack_smp_processor_id()
x86/smp: Remove unnecessary initialization of thread_info::cpu x86/asm: Move 'status' from struct thread_info to struct thread_struct
kdb: Use task_cpu() instead of task_thread_info()->cpu
printk: When dumping regs, show the stack, not thread_info
sched: Allow putting thread_info into task_struct
x86: Move thread_info into task_struct
sched: Add try_get_task_stack() and put_task_stack() x86/dumpstack: Pin the target stack in save_stack_trace_tsk()
sched: Free the stack early if CONFIG_THREAD_INFO_IN_TASK
fork: Cache two thread stacks per cpu if CONFIG_VMAP_STACK is set
Ingo Molnar (1):
x86/mm/hotplug: Don't remove PGD entries in remove_pagetable()
Linus Torvalds (2):
x86/entry: Get rid of pt_regs_to_thread_info()
um: Stop conflating task_struct::stack with thread_info
Oleg Nesterov (1):
kthread: to_live_kthread() needs try_get_task_stack()
arch/Kconfig | 33 ++++++ arch/alpha/include/asm/thread_info.h | 27 ----- arch/ia64/include/asm/thread_info.h | 30 +---- arch/microblaze/include/asm/thread_info.h | 27 ----- arch/powerpc/include/asm/thread_info.h | 25 ----- arch/sh/include/asm/thread_info.h | 26 ----- arch/sparc/include/asm/thread_info_64.h | 24 ---- arch/tile/include/asm/thread_info.h | 27 ----- arch/x86/Kconfig | 2 + arch/x86/entry/common.c | 25 ++--- arch/x86/entry/entry_32.S | 11 ++ arch/x86/entry/entry_64.S | 20 +++- arch/x86/entry/vsyscall/vsyscall_64.c | 6 +- arch/x86/include/asm/checksum_32.h | 3 +- arch/x86/include/asm/cpu.h | 1 - arch/x86/include/asm/efi.h | 1 - arch/x86/include/asm/pgtable_types.h | 2 - arch/x86/include/asm/processor.h | 32 ++++-- arch/x86/include/asm/smp.h | 6 - arch/x86/include/asm/switch_to.h | 34 +++++- arch/x86/include/asm/syscall.h | 23 +--- arch/x86/include/asm/thread_info.h | 102 +---------------- arch/x86/include/asm/traps.h | 6 + arch/x86/include/asm/uaccess.h | 10 +- arch/x86/kernel/asm-offsets.c | 5 +- arch/x86/kernel/cpu/common.c | 2 +- arch/x86/kernel/dumpstack.c | 20 +++- arch/x86/kernel/dumpstack_32.c | 4 +- arch/x86/kernel/dumpstack_64.c | 16 ++- arch/x86/kernel/fpu/init.c | 1 - arch/x86/kernel/irq_64.c | 3 +- arch/x86/kernel/process.c | 6 +- arch/x86/kernel/process_64.c | 4 +- arch/x86/kernel/ptrace.c | 2 +- arch/x86/kernel/smpboot.c | 1 - arch/x86/kernel/stacktrace.c | 5 + arch/x86/kernel/traps.c | 62 +++++++++++ arch/x86/lib/copy_user_64.S | 8 +- arch/x86/lib/csum-wrappers_64.c | 1 + arch/x86/lib/getuser.S | 20 ++-- arch/x86/lib/putuser.S | 10 +- arch/x86/lib/usercopy_64.c | 2 +- arch/x86/mm/extable.c | 2 +- arch/x86/mm/fault.c | 38 ++++++- arch/x86/mm/init_64.c | 27 ----- arch/x86/mm/pageattr.c | 37 +------ arch/x86/mm/tlb.c | 15 +++ arch/x86/platform/efi/efi.c | 2 - arch/x86/platform/efi/efi_32.c | 3 - arch/x86/platform/efi/efi_64.c | 5 - arch/x86/um/ptrace_32.c | 8 +- drivers/base/node.c | 3 +- drivers/pnp/isapnp/proc.c | 2 +- fs/proc/meminfo.c | 2 +- include/linux/init_task.h | 11 ++ include/linux/kdb.h | 2 +- include/linux/memcontrol.h | 2 +- include/linux/mmzone.h | 2 +- include/linux/sched.h | 144 +++++++++++++++++++++++- include/linux/thread_info.h | 56 +++------- init/Kconfig | 10 ++ init/init_task.c | 7 +- kernel/fork.c | 177 ++++++++++++++++++++++++++---- kernel/kthread.c | 8 +- kernel/printk/printk.c | 5 +- kernel/sched/core.c | 4 + kernel/sched/sched.h | 4 + lib/bitmap.c | 2 +- lib/dma-debug.c | 39 ++++++- mm/memcontrol.c | 2 +- mm/page_alloc.c | 3 +- net/bluetooth/smp.c | 67 +++++------ 72 files changed, 767 insertions(+), 597 deletions(-)
Signature · 1 line
2.7.4