Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1683092

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

From Thomas Gleixner <tglx@linutronix.de>
Newsgroups linux.kernel
Subject Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t
Date 2017-07-07 12:00 +0200
Message-ID <u0ydk-2cj-9@gated-at.bofh.it> (permalink)
References <u0xqW-1Re-3@gated-at.bofh.it> <u0xqX-1Re-13@gated-at.bofh.it> <u0xKi-20Z-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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 overflows that might lead to use-after-free
> > situations.
> > 
> > Signed-off-by: Elena Reshetova <elena.reshetova@intel.com>
> > Signed-off-by: Hans Liljestrand <ishkamiel@gmail.com>
> > Signed-off-by: Kees Cook <keescook@chromium.org>
> > Signed-off-by: David Windsor <dwindsor@gmail.com>
> 
> I'll let tglx comment on the SoB chain, I know he likes those :-) You
> did Cc him right, seeing how he's the maintainer of this stuff..

Right, that SOB chain is crap. It suggests that the patch was written by
Elena and then carried on by Hans, handed over to Kees and from there to
David. And now it's resent by Elena. Circular dependencies or what?

Thanks,

	tglx

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t Elena Reshetova <elena.reshetova@intel.com> - 2017-07-07 11:10 +0200
  Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t Peter Zijlstra <peterz@infradead.org> - 2017-07-07 11:30 +0200
    Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t Thomas Gleixner <tglx@linutronix.de> - 2017-07-07 12:00 +0200
      RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t "Reshetova, Elena" <elena.reshetova@intel.com> - 2017-07-07 12:30 +0200
        Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t Ingo Molnar <mingo@kernel.org> - 2017-07-07 12:40 +0200
          Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org> - 2017-07-07 15:30 +0200
    RE: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t "Reshetova, Elena" <elena.reshetova@intel.com> - 2017-07-07 12:30 +0200
      Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from  atomic_t to refcount_t Peter Zijlstra <peterz@infradead.org> - 2017-07-07 13:20 +0200

csiph-web