uprobes: kill uprobes_state->count, add MMF_HAS_UPROBES
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest7 patches 9 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/7 | uprobes: kill uprobes_state->countinclude/ kernel/ | +3 | 1 reply | |
| 2/7 | uprobes: kill dup_mmap()->uprobe_mmap(), simplify uprobe_mmap/munmapkernel/ kernel/ | +3 | 1 reply | |
| 3/7 | uprobes: change uprobe_mmap() to ignore the errors but check fatal_signal_pending()kernel/ | +6 | 1 reply | |
| 4/7 | uprobes: do not use -EEXIST in install_breakpoint() pathskernel/ | +5 | 1 reply | |
| 5/7 | uprobes: introduce MMF_HAS_UPROBESinclude/ include/ kernel/ +1 more | +29 | 3 replies | |
| 6/7 | uprobes: fold uprobe_reset_state() into uprobe_dup_mmap()include/ kernel/ kernel/ | +2 | 1 reply | |
| 7/7 | uprobes: remove "verify" argument from set_orig_insn()include/ kernel/ | +10 | 1 reply |
Reviews trailers per patch
All 7 patches have a Reviewed-by or Acked-by.
| Patch | Srikar Dronamraju |
|---|---|
| 1/7 | A-b |
| 2/7 | A-b |
| 3/7 | A-b |
| 4/7 | A-b |
| 5/7 | A-b |
| 6/7 | A-b |
| 7/7 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/7] uprobes: kill uprobes_state->count, add MMF_HAS_UPROBES
Hello.
Srikar, please review.
This is the first part. The next series will teach uprobes to clear MMF_HAS_UPROBES, but perhaps we should simply remove uprobe_munmap() instead. A wrong MMF_HAS_UPROBES doesn't hurt unless the task hits the non-uprobe "int3", not sure it really makes sense to try to speedup this case. We will see.
7/7 is offtopic.
Oleg.
include/linux/sched.h | 2 + include/linux/uprobes.h | 13 ++-- kernel/events/uprobes.c | 141 ++++++++++++---------------------------------- kernel/fork.c | 6 +-- 4 files changed, 47 insertions(+), 115 deletions(-)