Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760549AbZF3AH3 (ORCPT ); Mon, 29 Jun 2009 20:07:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756537AbZF3AHT (ORCPT ); Mon, 29 Jun 2009 20:07:19 -0400 Received: from waste.org ([66.93.16.53]:38964 "EHLO waste.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758663AbZF3AHS (ORCPT ); Mon, 29 Jun 2009 20:07:18 -0400 Subject: Re: [PATCH RFC] fix RCU-callback-after-kmem_cache_destroy problem in sl[aou]b From: Matt Mackall To: Christoph Lameter Cc: "Paul E. McKenney" , linux-kernel@vger.kernel.org, linux-mm@kvack.org, penberg@cs.helsinki.fi, jdb@comx.dk In-Reply-To: References: <20090625193137.GA16861@linux.vnet.ibm.com> <1246315553.21295.100.camel@calx> Content-Type: text/plain Date: Mon, 29 Jun 2009 19:06:34 -0500 Message-Id: <1246320394.21295.105.camel@calx> Mime-Version: 1.0 X-Mailer: Evolution 2.26.1.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1353 Lines: 32 On Mon, 2009-06-29 at 19:19 -0400, Christoph Lameter wrote: > On Mon, 29 Jun 2009, Matt Mackall wrote: > > > This is a reasonable point, and in keeping with the design principle > > 'callers should handle their own special cases'. However, I think it > > would be more than a little surprising for kmem_cache_free() to do the > > right thing, but not kmem_cache_destroy(). > > kmem_cache_free() must be used carefully when using SLAB_DESTROY_BY_RCU. > The freed object can be accessed after free until the rcu interval > expires (well sortof, it may even be reallocated within the interval). > > There are special RCU considerations coming already with the use of > kmem_cache_free(). > > Adding RCU operations to the kmem_cache_destroy() logic may result in > unnecessary RCU actions for slabs where the coder is ensuring that the > RCU interval has passed by other means. Do we care? Cache destruction shouldn't be in anyone's fast path. Correctness is more important and users are more liable to be correct with this patch. -- http://selenic.com : development and support for Mercurial and Linux -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/