Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932869AbZD1QPu (ORCPT ); Tue, 28 Apr 2009 12:15:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754736AbZD1QPi (ORCPT ); Tue, 28 Apr 2009 12:15:38 -0400 Received: from ey-out-2122.google.com ([74.125.78.26]:55484 "EHLO ey-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753024AbZD1QPh convert rfc822-to-8bit (ORCPT ); Tue, 28 Apr 2009 12:15:37 -0400 MIME-Version: 1.0 In-Reply-To: <49F729F1.2020107@kernel.org> References: <20090418192314.GA22107@suse.de> <49F6B1E6.7050008@kernel.org> <49F6B350.9080703@kernel.org> <49F71EF6.3050102@kernel.org> <49F72205.7030306@kernel.org> <20090428153906.GA8896@suse.de> <49F7260F.8030405@kernel.org> <49F729F1.2020107@kernel.org> From: Kay Sievers Date: Tue, 28 Apr 2009 18:15:15 +0200 Message-ID: Subject: Re: [RFC PATCH] use dev_set_name(,NULL) to prevent leaking To: Yinghai Lu Cc: Greg KH , Ingo Molnar , Linus Torvalds , Jesse Barnes , Andrew Morton , "H. Peter Anvin" , Thomas Gleixner , "linux-kernel@vger.kernel.org" , linux-pci@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1106 Lines: 30 On Tue, Apr 28, 2009 at 18:08, Yinghai Lu wrote: > Kay Sievers wrote: >> On Tue, Apr 28, 2009 at 17:51, Yinghai Lu wrote: >> >>> before device_register==>device_initialize is called, kobj->ref is still 0. >>> >>> will get warn from >>>                if (!kobj->state_initialized) >> >> Initialize the device before you do anything with it. And call _put() >> any time to get rid of ressources, which might have been allocated >> before registering. > so you mean don't call dev_set_name before device_register and let device_register or device_add take name param? No. I meant: device_initialize() call put_device() any time you want to get rid of ressources of the device device_set_name() call put_device() any time you want to get rid of ressources of the device device_register() ... Kay -- 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/