x86/entry: Consolidation part II (syscalls)
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest23 patches 19 repliesIn review
- v213 patches 23 repliesApplied
- v18 patches 47 repliesSuperseded
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/23 | rcu: Dont acquire lock in NMI handler in rcu_nmi_enter_common()kernel/ | +1 | 1 reply | |
| 2/23 | rcu: Add comments marking transitions between RCU watching and notkernel/ | +25 | 1 reply | |
| 3/23 | vmlinux.lds.h: Create section for protection against instrumentationinclude/ include/ include/ +2 more | +36 | 2 replies | |
| 4/23 | kprobes: Prevent probes in .noinstr.text sectionkernel/ | +11 | 5 replies | |
| 5/23 | tracing: Provide lockdep less trace_hardirqs_on/off() variantsinclude/ kernel/ | +27 | 1 reply | |
| 6/23 | bug: Annotate WARN/BUG/stackfail as noinstr safearch/ include/ kernel/ | +13 | 4 replies | |
| 7/23 | lockdep: Prepare for noinstr sectionsinclude/ include/ kernel/ +2 more | +55 | 0 replies | |
| 8/23 | x86/entry: Mark enter_from_user_mode() noinstrarch/ | +1 | 0 replies | |
| 9/23 | x86/entry/common: Protect against instrumentationarch/ | +89 | 0 replies | |
| 10/23 | x86/entry: Move irq tracing on syscall entry to C-codearch/ arch/ arch/ +1 more | +19 | 0 replies | |
| 11/23 | x86/entry: Move irq flags tracing to prepare_exit_to_usermode()arch/ arch/ arch/ +1 more | +26 | 0 replies | |
| 12/23 | context_tracking: Ensure that the critical path cannot be instrumentedinclude/ include/ kernel/ | +14 | 0 replies | |
| 13/23 | lib/smp_processor_id: Move it into noinstr sectionlib/ | +5 | 0 replies | |
| 14/23 | x86/speculation/mds: Mark mds_user_clear_cpu_buffers() __always_inlinearch/ | +2 | 0 replies | |
| 15/23 | x86/entry/64: Check IF in __preempt_enable_notrace() thunkarch/ arch/ arch/ | +25 | 0 replies | |
| 16/23 | x86/entry/64: Mark ___preempt_schedule_notrace() thunk noinstrarch/ | +18 | 0 replies | |
| 17/23 | rcu/tree: Mark the idle relevant functions noinstrkernel/ kernel/ kernel/ | +42 | 3 replies | |
| 18/23 | x86/kvm: Move context tracking where it belongsarch/ arch/ arch/ | +28 | 0 replies | |
| 19/23 | x86/kvm/vmx: Add hardirq tracing to guest enter/exitarch/ | +23 | 2 replies | |
| 20/23 | x86/kvm/svm: Handle hardirqs proper on guest enter/exitarch/ | +28 | 0 replies | |
| 21/23 | context_tracking: Make guest_enter/exit_irqoff() .noinstr readyinclude/ | +16 | 0 replies | |
| 22/23 | x86/kvm/vmx: Move guest enter/exit into .noinstr.textarch/ arch/ arch/ +2 more | +68 | 0 replies | |
| 23/23 | x86/kvm/svm: Move guest enter/exit into .noinstr.textarch/ | +62 | 0 replies |
Reviews trailers per patch
2 of 23 patches have a Reviewed-by or Acked-by.
| Patch | Joel Fernandes (Google) | Frederic Weisbecker |
|---|---|---|
| 1/23 | R-b | R-b |
| 2/23 | R-b | |
| 3/23 | ||
| 4/23 | ||
| 5/23 | ||
| 6/23 | ||
| 7/23 | ||
| 8/23 | ||
| 9/23 | ||
| 10/23 | ||
| 11/23 | ||
| 12/23 | ||
| 13/23 | ||
| 14/23 | ||
| 15/23 | ||
| 16/23 | ||
| 17/23 | ||
| 18/23 | ||
| 19/23 | ||
| 20/23 | ||
| 21/23 | ||
| 22/23 | ||
| 23/23 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [patch V3 00/23] x86/entry: Consolidation part II (syscalls)
Hi!
This is the third version of the syscall entry code consolidation series. V2 can be found here:
https://lore.kernel.org/r/[email protected]
It applies on top of
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/entry
and is also available from git:
git://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel entry-v3-part2
The changes vs. V2:
- A massive rework utilizing Peter Zijlstras objtool patches to analyze the new .noinstr.text section:
https://lore.kernel.org/r/[email protected]
Working with this was really helpful as it clearly pin pointed code which calls out of the protected section which is much more efficient and focussed than chasing everything manually.
- Picked up the two RCU patches from Paul for completeness. The bugfix is required anyway and the comments have been really helpful to see where the defense line has to be.
- As the tool flagged KVM as red zone, I looked at the context tracking usage there and it has similar if not worse issues. New set of patches dealing with that.
Please have a close look at the approach and the resulting protected areas.
Known issues:
- The kprobes '.noinstr.text' exclusion currently works only for built in code. Haven't figured out how to to fix that, but I'm sure that Masami knows :)
- The various SANitizers if enabled ruin the picture. Peter and I still have no brilliant idea what to do about that.
Thanks,
tglx
--- arch/x86/entry/common.c | 173 ++++++++++++++++++++++++--------- arch/x86/entry/entry_32.S | 24 ---- arch/x86/entry/entry_64.S | 6 - arch/x86/entry/entry_64_compat.S | 32 ------ arch/x86/entry/thunk_64.S | 45 +++++++- arch/x86/include/asm/bug.h | 3 arch/x86/include/asm/hardirq.h | 4 arch/x86/include/asm/irqflags.h | 3 arch/x86/include/asm/nospec-branch.h | 4 arch/x86/include/asm/paravirt.h | 3 arch/x86/kvm/svm.c | 152 ++++++++++++++++++---------- arch/x86/kvm/vmx/ops.h | 4 arch/x86/kvm/vmx/vmenter.S | 2 arch/x86/kvm/vmx/vmx.c | 78 +++++++++++--- arch/x86/kvm/x86.c | 4 b/include/asm-generic/bug.h | 9 + include/asm-generic/sections.h | 3 include/asm-generic/vmlinux.lds.h | 4 include/linux/compiler.h | 24 ++++ include/linux/compiler_types.h | 4 include/linux/context_tracking.h | 27 +++-- include/linux/context_tracking_state.h | 6 - include/linux/irqflags.h | 6 + include/linux/sched.h | 1 kernel/context_tracking.c | 14 +- kernel/kprobes.c | 11 ++ kernel/locking/lockdep.c | 66 +++++++++--- kernel/panic.c | 4 kernel/rcu/tree.c | 91 +++++++++++------ kernel/rcu/tree_plugin.h | 4 kernel/rcu/update.c | 7 - kernel/trace/trace_preemptirq.c | 25 ++++ lib/debug_locks.c | 2 lib/smp_processor_id.c | 10 - scripts/mod/modpost.c | 2 35 files changed, 590 insertions(+), 267 deletions(-)