From mboxrd@z Thu Jan 1 00:00:00 1970 From: Emmanuel Benisty Subject: Re: [PATCH -next] ipc: make refcounter atomic (was Re: linux-next: Tree for Apr 23 [ Call-Traces: lib/debugobjects.c | kernel/rcupdate.c | kernel/rcutree.c ]) Date: Thu, 25 Apr 2013 20:14:42 +0700 Message-ID: References: <1366741069.1802.12.camel@buesod1.americas.hpqcorp.net> <1366741394.1802.13.camel@buesod1.americas.hpqcorp.net> <1366841773.1790.7.camel@buesod1.americas.hpqcorp.net> <517927E2.1090907@surriel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-qc0-f171.google.com ([209.85.216.171]:33921 "EHLO mail-qc0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932219Ab3DYNOn (ORCPT ); Thu, 25 Apr 2013 09:14:43 -0400 In-Reply-To: <517927E2.1090907@surriel.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: Rik van Riel Cc: sedat.dilek@gmail.com, Davidlohr Bueso , Linus Torvalds , Andrew Morton , Paul McKenney , Paul McKenney , linux-next , Linux Kernel Mailing List On Thu, Apr 25, 2013 at 7:56 PM, Rik van Riel wrote: > On 04/24/2013 07:05 PM, Sedat Dilek wrote: >> >> On Thu, Apr 25, 2013 at 12:16 AM, Davidlohr Bueso >> wrote: > > >>> This patch converts the IPC RCU header's reference counter to atomic_t. >>> The return of >>> ipc_rcu_getref() is modified to inform the callers if it actually >>> succeeded. >>> >>> Now all callers return -EIDRM upon failure and abort the current >>> operation. Two exceptions are >>> in semaphore code where sem_getref_and_unlock() and sem_getref() trigger >>> a warning but proceed >>> to freeing up any held locks. >>> >>> Signed-off-by: Davidlohr Bueso >>> Suggested-by: Linus Torvalds >>> CC: Rik van Riel >>> CC: Paul McKenney >>> CC: Sedat Dilek >>> CC: Emmanuel Benisty >>> CC: Andrew Morton >> >> >> Missing my Reported-by ...! > > >> ...so please add also my Tested-by! > > > And my ax! > > I mean ... Reviewed-by: Rik van Riel FWIW, my machine survived building chromium with this patch instead of Linus'. Thanks. -- Emmanuel