From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755338AbYCCMmv (ORCPT ); Mon, 3 Mar 2008 07:42:51 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751389AbYCCMmm (ORCPT ); Mon, 3 Mar 2008 07:42:42 -0500 Received: from ns1.suse.de ([195.135.220.2]:46845 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751377AbYCCMml (ORCPT ); Mon, 3 Mar 2008 07:42:41 -0500 To: Thomas Gleixner Cc: LKML , Andrew Morton , Ingo Molnar , Greg KH , chris.mason@oracle.com Subject: Re: [patch 0/2] object debugging infrastructure From: Andi Kleen References: <20080301100019.640027768@linutronix.de> Date: Mon, 03 Mar 2008 13:42:40 +0100 In-Reply-To: <20080301100019.640027768@linutronix.de> (Thomas Gleixner's message of "Sat\, 01 Mar 2008 10\:24\:52 -0000") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1 (gnu/linux) 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 Thomas Gleixner writes: > > 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 providing additional checks > whenever kernel memory is freed. Nice idea to make this generic. If you push this it would be also good to consider some variant of the "crasher" code that is in SUSE Kernels to include with it. It was originally written by Chris Mason and it also makes it easier to stabilize such bugs. The basic idea is that a background thread allocates lots of slabs and pages always poisons them and checks them in the background. If someone corrupts memory that is noticed earlier then Basically it extends what your patchkit is doing to free memory too. Chris' old patch: http://firstfloor.org/~andi/crasher-26.diff (various variants of that have existed over time) -Andi