Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752675AbZLPRtf (ORCPT ); Wed, 16 Dec 2009 12:49:35 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753771AbZLPRt3 (ORCPT ); Wed, 16 Dec 2009 12:49:29 -0500 Received: from kroah.org ([198.145.64.141]:44309 "EHLO coco.kroah.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752498AbZLPRt1 (ORCPT ); Wed, 16 Dec 2009 12:49:27 -0500 Date: Wed, 16 Dec 2009 09:46:32 -0800 From: Greg KH To: Zdenek Kabelac Cc: device-mapper development , Milan Broz , linux-kernel@vger.kernel.org, Alasdair G Kergon Subject: Re: [dm-devel] Re: dm: sysfs add empty release function to avoid debug warning Message-ID: <20091216174632.GA12473@kroah.com> References: <200912151735.nBFHZ8XP018683@hera.kernel.org> <20091216004739.GB4898@kroah.com> <4B28AC09.2000302@redhat.com> <20091216134538.GA20857@kroah.com> <4B290B95.10306@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4B290B95.10306@redhat.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2700 Lines: 73 On Wed, Dec 16, 2009 at 05:32:21PM +0100, Zdenek Kabelac wrote: > Dne 16.12.2009 14:45, Greg KH napsal(a): > > On Wed, Dec 16, 2009 at 10:44:41AM +0100, Milan Broz wrote: > >> On 12/16/2009 01:47 AM, Greg KH wrote: > >>> On Tue, Dec 15, 2009 at 05:35:08PM +0000, James Bottomley wrote: > >>>> commit: d2bb7df8cac647b92f51fb84ae735771e7adbfa7 > >>>> From: Milan Broz > >>>> Date: Thu, 10 Dec 2009 23:51:53 +0000 > >>>> Subject: [PATCH] dm: sysfs add empty release function to avoid debug warning > >>>> > >>>> This patch just removes an unnecessary warning: > >>>> kobject: 'dm': does not have a release() function, > >>>> it is broken and must be fixed. > >>>> > >>>> The kobject is embedded in mapped device struct, so > >>>> code does not need to release memory explicitly here. > >>> > >> > >>> Please, this is totally and completly wrong. And if you feel that it is > >>> needed, then your design is wrong and it needs to be fixed. > >> > >> There are several places in kernel, where kobject have not defined release > >> method. Yes, something is wrong here. > > > > Where are those instances, becides the use of static kobjects, which is > > being worked on? > > > > That still does not make this change acceptable, it is incorrect. > > > >> So quietly ignoring warning is ok? Why is not there BUG_ON(!release) then? > > > > I was trying to be nice and not crash your machine and give you the > > opportunity to fix it easily. > > > >> The sysfs attributes here just represents attributes of block device object, > >> this device is always removed before release here is called. > > > > Then why use a kobject at all? Your reference counting will be all > > wrong if you use it incorrectly like this. > > > >> So if there is preferred to do another alloc/free, no problem. > > > > Please do. > > > > Also, with the addition of this patch, your comments for the kobject are > > now incorrect as well. > > > > Here is list of just some errors I get on my 2.6.32 linux kernel. > (error message itself is removed) > Messages are printed during module removal/reboot. > > 'aead' (ffffffffa017c050): does not have a release() function, it is broken > and must be fixed. > 'bridge' > 'cdrom' You will find that all of these are modules, right? That's being worked on still. Again, please fix your code not to have an empty release function. Will you revert this change and fix it properly? thanks, greg k-h -- 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/