Skip to content

futex: Plug a pi_state leak and clarify the refcounting

Thomas Gleixner lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5futex: Drop refcount if requeue_pi() acquired the rtmutexkernel/futex.c+5 -00 replies
2/5futex: Rename free_pi_state() to put_pi_state()kernel/futex.c+10 -70 replies
3/5futex: Document pi_state refcounting in requeue codekernel/futex.c+33 -31 reply
4/5futex: Remove pointless put_pi_state calls in requeue()kernel/futex.c+2 -40 replies
5/5futex: Cleanup the goto confusion in requeue_pi()kernel/futex.c+3 -24 replies

Reviews trailers per patch

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

PatchDarren Hart
1/5
2/5
3/5
4/5
5/5R-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/5] futex: Plug a pi_state leak and clarify the refcounting

A recent patch claimed that there is a double free in the requeue_pi code, which is not the case.

While analysing the issue I found the contrary, i.e. a leak. This series fixes the leak and clarifies the code so it's more clear how that refcounting on the pi state works.

Thanks,

tglx