Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752218AbdGGK1g convert rfc822-to-8bit (ORCPT ); Fri, 7 Jul 2017 06:27:36 -0400 Received: from mga03.intel.com ([134.134.136.65]:9043 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750878AbdGGK1e (ORCPT ); Fri, 7 Jul 2017 06:27:34 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,322,1496127600"; d="scan'208";a="1169785287" From: "Reshetova, Elena" To: Thomas Gleixner , Peter Zijlstra CC: "linux-kernel@vger.kernel.org" , "cgroups@vger.kernel.org" , "linux-audit@redhat.com" , "linux-fsdevel@vger.kernel.org" , "gregkh@linuxfoundation.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 Thread-Topic: [PATCH 14/15] kernel: convert futex_pi_state.refcount from atomic_t to refcount_t Thread-Index: AQHS9wCo5QY4hvTBzE2JOYLHihWHc6JIB1aAgAAHhICAABoX8A== Date: Fri, 7 Jul 2017 10:27:27 +0000 Message-ID: <2236FBA76BA1254E88B949DDB74E612B6FF21DCD@IRSMSX102.ger.corp.intel.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> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1045 Lines: 25 > 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. Best Regards, Elena