RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-18 Thread Reshetova, Elena
> On Mon, 17 Jul 2017, Reshetova, Elena wrote: > > > On Mon, 17 Jul 2017, Elena Reshetova wrote: > > > > refcount_t type and corresponding API should be > > > > used instead of atomic_t when the variable is used as > > > > a reference counter. This allows to avoid accidental > > > > refcounter over

RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, Reshetova, Elena wrote: > > On Mon, 17 Jul 2017, Elena Reshetova wrote: > > > refcount_t type and corresponding API should be > > > used instead of atomic_t when the variable is used as > > > a reference counter. This allows to avoid accidental > > > refcounter overflows that m

RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-17 Thread Reshetova, Elena
> On Mon, 17 Jul 2017, Elena Reshetova wrote: > > > Subject: kernel: convert futex_pi_state.refcount from atomic_t to refcount_t > > Several people including myself told you already, that subjects consist of > > SUBSYSTEMPREFIX: Concise description > > It's easy enough to figure the prefix out

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, Elena Reshetova wrote: > Subject: kernel: convert futex_pi_state.refcount from atomic_t to refcount_t Several people including myself told you already, that subjects consist of SUBSYSTEMPREFIX: Concise description It's easy enough to figure the prefix out by looking at the

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread gre...@linuxfoundation.org
On Fri, Jul 07, 2017 at 12:35:16PM +0200, Ingo Molnar wrote: > > * Reshetova, Elena wrote: > > > > On Fri, 7 Jul 2017, Peter Zijlstra wrote: > > > > > > > On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > > > > > refcount_t type and corresponding API should be > > > > > used in

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Peter Zijlstra
On Fri, Jul 07, 2017 at 10:24:20AM +, Reshetova, Elena wrote: > It is not so trivial as you might think. Unless right person shows up > as maintainer/supporter when I run get_maintainer script, In this case though it should: FUTEX SUBSYSTEM M: Thomas Gleixner M: Ingo Molnar R:

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Ingo Molnar
* Reshetova, Elena wrote: > > On Fri, 7 Jul 2017, Peter Zijlstra wrote: > > > > > On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > > > > refcount_t type and corresponding API should be > > > > used instead of atomic_t when the variable is used as > > > > a reference counter. T

RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Reshetova, Elena
> On Fri, 7 Jul 2017, Peter Zijlstra wrote: > > > On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > > > refcount_t type and corresponding API should be > > > used instead of atomic_t when the variable is used as > > > a reference counter. This allows to avoid accidental > > > refc

RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Reshetova, Elena
> On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > > si

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Thomas Gleixner
On Fri, 7 Jul 2017, Peter Zijlstra wrote: > On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflo

Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t

2017-07-07 Thread Peter Zijlstra
On Fri, Jul 07, 2017 at 12:04:28PM +0300, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. >