Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752451AbdGGNXD (ORCPT ); Fri, 7 Jul 2017 09:23:03 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33978 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752351AbdGGNXA (ORCPT ); Fri, 7 Jul 2017 09:23:00 -0400 Date: Fri, 7 Jul 2017 15:22:53 +0200 From: "gregkh@linuxfoundation.org" To: Ingo Molnar Cc: "Reshetova, Elena" , Thomas Gleixner , Peter Zijlstra , "linux-kernel@vger.kernel.org" , "cgroups@vger.kernel.org" , "linux-audit@redhat.com" , "linux-fsdevel@vger.kernel.org" , "viro@zeniv.linux.org.uk" , "tj@kernel.org" , "mingo@redhat.com" , "hannes@cmpxchg.org" , "lizefan@huawei.com" , "acme@kernel.org" , "alexander.shishkin@linux.intel.com" , "eparis@redhat.com" , "akpm@linux-foundation.org" , "arnd@arndb.de" , "luto@kernel.org" , "keescook@chromium.org" , Hans Liljestrand , David Windsor Subject: Re: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t Message-ID: <20170707132253.GB15245@kroah.com> References: <1499418269-3686-1-git-send-email-elena.reshetova@intel.com> <1499418269-3686-15-git-send-email-elena.reshetova@intel.com> <20170707092603.rmt4ne4c75k5qg56@hirez.programming.kicks-ass.net> <2236FBA76BA1254E88B949DDB74E612B6FF21DCD@IRSMSX102.ger.corp.intel.com> <20170707103516.d2xvqvijaomj342j@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170707103516.d2xvqvijaomj342j@gmail.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2226 Lines: 57 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 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 > > > > > Signed-off-by: Hans Liljestrand > > > > > Signed-off-by: Kees Cook > > > > > Signed-off-by: David Windsor > > > > > > > > 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? > > > > I will fix SOB on all patches and resend. > > Please don't resend any of these until the merge window is over! This is probably > the worst possible moment to seek review feedback and merging... > > Thanks, > > Ingo Ah, you need a "I'm ignoring your patches for two weeks" email bot, feel free to steal the text of mine below :) ---------------------- Hi, This is the friendly semi-automated patch-bot of Greg Kroah-Hartman. You have sent him a patch that has triggered this response. Right now, the development tree you have sent a patch for is "closed" due to the timing of the merge window. Don't worry, the patch(es) you have sent are not lost, and will be looked at after the merge window is over (after the -rc1 kernel is released by Linus). So thank you for your patience and your patches will be reviewed at this later time, you do not have to do anything further, this is just a short note to let you know the patch status and so you don't worry they didn't make it through. thanks, greg k-h's patch email bot