From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754069AbYCYIWo (ORCPT ); Tue, 25 Mar 2008 04:22:44 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751869AbYCYIWh (ORCPT ); Tue, 25 Mar 2008 04:22:37 -0400 Received: from www.tglx.de ([62.245.132.106]:50188 "EHLO www.tglx.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751507AbYCYIWg (ORCPT ); Tue, 25 Mar 2008 04:22:36 -0400 Date: Tue, 25 Mar 2008 09:19:11 +0100 (CET) From: Thomas Gleixner To: Andrew Morton cc: LKML , Ingo Molnar , Greg KH , Randy Dunlap , Peter Zijlstra Subject: Re: [patch 2/5] infrastructure to debug (dynamic) objects In-Reply-To: <20080324130454.ced60108.akpm@linux-foundation.org> Message-ID: References: <20080321200300.310393657@linutronix.de> <20080321200321.650649403@linutronix.de> <20080324130454.ced60108.akpm@linux-foundation.org> User-Agent: Alpine 1.00 (LFD 882 2007-12-20) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 24 Mar 2008, Andrew Morton wrote: > On Fri, 21 Mar 2008 20:26:18 -0000 > Thomas Gleixner wrote: > > > The debugobjects core code keeps track of operations on static and > > dynamic objects by inserting them into a hashed list and sanity > > checking them on object operations and provides additional checks > > whenever kernel memory is freed. > > Prime candidates for conversion to this interface are locks: spinlocks, > rwlocks, mutexes, etc. > > a) it'd be interesting to get that done, as a proof-of-usefulness thing. /me looks for volunteers :) > b) but this code internally uses spinlocks. Will it explode? It should not, but we can use untracked spinlocks for the internals. > also list_heads and hlists. But > > a) that might be a bit redundant against the custome debugging which lists > already have and Yup. > b) this code uses lists and hlists internally? Yes, it uses hlists. Thanks, tglx