Skip to content

uprobes: memory leak in enable/disable loop

Oleg Nesterov lkml 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2uprobes: fix the memcg accountingkernel/events/uprobes.c+3 -20 replies
2/2uprobes: rename the "struct page *" args of __replace_page()kernel/events/uprobes.c+18 -180 replies

Reviews trailers per patch

1 of 2 patches has a Reviewed-by or Acked-by.

PatchBrenden BlancoMichal HockoJohannes Weiner
1/2Reported-and-tested-by A-b R-b
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 0/2] uprobes: memory leak in enable/disable loop
Quoted 2 lines · Brenden Blanco

This passes my tests, thanks!

Great, thanks Brenden!

Quoted 3 lines

Please note that I applied this to 4.4.15+ubuntu-patches kernel, since that was what I had most handy, therefore I had to adjust the patch to remove the unavailable 'compound' bool parameter in 4.4 kernels.

Yes, sure, this is correct.

Ingo, could you take this fix? IMO 1/2 is 4.8 and -stable matterial.

As for 2/2 it is purely cosmetic but I simply can't resist, I wanted to make this change every time I had to look at __replace_page().

Oleg.

 kernel/events/uprobes.c | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)