uprobes: Kill arch_uprobe_enable/disable_step() hooks
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest4 patches 9 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/4 | uprobes/powerpc: Don't clear TIF_UPROBE in do_notify_resume()arch/ | +1 | 5 replies | |
| 2/4 | uprobes/powerpc: Do not use arch_uprobe_*_step() helpersarch/ kernel/ | +6 | 2 replies | |
| 3/4 | uprobes/x86: Cleanup the single-stepping codearch/ | +19 | 1 reply | |
| 4/4 | uprobes: Kill arch_uprobe_enable/disable_step() hooksinclude/ kernel/ | +0 | 1 reply |
Reviews trailers per patch
All 4 patches have a Reviewed-by or Acked-by.
| Patch | Ananth N Mavinakayanahalli | Srikar Dronamraju |
|---|---|---|
| 1/4 | A-b | A-b |
| 2/4 | A-b | A-b |
| 3/4 | A-b | |
| 4/4 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/4] uprobes: Kill arch_uprobe_enable/disable_step() hooks
Hello.
arch_uprobe_enable/disable_step() were only needed to not break the pending powerpc port. They buy nothing and they are simply wrong. Uprobes should not use ptrace helpers for the stepping.
Now that powepc port was merged we should kill them asap, before arm port.
1/4 is minor/offtopic cleanup. powerpc changes were not tested at all and thus need the review from Ananth.
Oleg.