Skip to content

uprobes: alloc utask/xol_area cleanups and minor fix

Oleg Nesterov lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 patches 17 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/7uprobes: Move alloc_page() from xol_add_vma() to xol_alloc_area()kernel/events/uprobes.c+13 -195 replies
2/7uprobes: Fold xol_alloc_area() into get_xol_area()kernel/events/uprobes.c+16 -223 replies
3/7uprobes: Turn add_utask() into get_utask()kernel/events/uprobes.c+9 -181 reply
4/7uprobes: Do not play with utask in xol_get_insn_slot()kernel/events/uprobes.c+21 -161 reply
5/7uprobes: Fix utask->xol_vaddr leak in pre_ssout()kernel/events/uprobes.c+8 -13 replies
6/7uprobes: Do not allocate current->utask unnecessarykernel/events/uprobes.c+6 -102 replies
7/7uprobes: Kill the bogus IS_ERR_VALUE(xol_vaddr) checkkernel/events/uprobes.c+1 -21 reply

Reviews trailers per patch

6 of 7 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchSrikar DronamrajuAnton Arapov
SeriesA-b
1/7A-b
2/7A-b
3/7A-b
4/7A-b
5/7A-b
6/7
7/7A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/7] uprobes: alloc utask/xol_area cleanups and minor fix

Hello.

This series fixes the minor bug and cleanups the usage of add_utask() and xol_alloc_area(). Plus it cleanups the initializaion of ->utask in handle_swbp() paths.

Anton, this conflicts with your uretprobe patches, but I think we should do this to avoid the code duplication. IOW, I consider this series as a minor preparations for uretprobes as well.

Oleg.

 kernel/events/uprobes.c |  150 +++++++++++++++++++++--------------------------
 1 files changed, 68 insertions(+), 82 deletions(-)