Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755038AbZINDNx (ORCPT ); Sun, 13 Sep 2009 23:13:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752798AbZINDNw (ORCPT ); Sun, 13 Sep 2009 23:13:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43189 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752445AbZINDNv (ORCPT ); Sun, 13 Sep 2009 23:13:51 -0400 Subject: Re: [GIT BISECT] BUG kmalloc-8192: Object already free from kmem_cache_destroy From: Eric Paris To: Danny Feng Cc: cl@linux-foundation.org, penberg@cs.helsinki.fi, mingo@elte.hu, linux-kernel@vger.kernel.org, linux-mm@kvack.org In-Reply-To: <4AADA1F9.9080305@redhat.com> References: <1252866835.13780.37.camel@dhcp231-106.rdu.redhat.com> <1252883493.16335.8.camel@dhcp231-106.rdu.redhat.com> <4AADA1F9.9080305@redhat.com> Content-Type: text/plain; charset="UTF-8" Date: Sun, 13 Sep 2009 23:13:05 -0400 Message-Id: <1252897985.5793.2.camel@dhcp231-106.rdu.redhat.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1825 Lines: 46 On Mon, 2009-09-14 at 09:52 +0800, Danny Feng wrote: > On 09/14/2009 07:11 AM, Eric Paris wrote: > > On Sun, 2009-09-13 at 14:33 -0400, Eric Paris wrote: > >> 2a38a002fbee06556489091c30b04746222167e4 is first bad commit > >> commit 2a38a002fbee06556489091c30b04746222167e4 > >> Author: Xiaotian Feng > >> Date: Wed Jul 22 17:03:57 2009 +0800 > >> > >> slub: sysfs_slab_remove should free kmem_cache when debug is enabled > >> > >> kmem_cache_destroy use sysfs_slab_remove to release the kmem_cache, > >> but when CONFIG_SLUB_DEBUG is enabled, sysfs_slab_remove just release > >> related kobject, the whole kmem_cache is missed to release and cause > >> a memory leak. > >> > >> Acked-by: Christoph Lameer > >> Signed-off-by: Xiaotian Feng > >> Signed-off-by: Pekka Enberg > >> > >> CONFIG_SLUB_DEBUG=y > >> CONFIG_SLUB=y > >> CONFIG_SLUB_DEBUG_ON=y > >> # CONFIG_SLUB_STATS is not set > > > > I also had problems destroying a kmem_cache in a security_initcall() > > function which had a different backtrace (it's what made me create the > > module and bisect.) So be sure to let me know what you find so I can > > be sure that we fix that place as well (I believe that was a kref > > problem rather than a double free) > > > > -Eric > > > > > Could you please tell me the tree you're using? I'll debug on it first... I was looking at the linux-next tree from Sept 11 http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=summary -Eric -- 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/