Date: Wed, 28 Nov 2007 17:35:32 +0100
From: Cornelia Huck <cornelia.huck@de.ibm.com>
To: Greg KH <greg@kroah.com>
Cc: linux-kernel@vger.kernel.org, Kay Sievers <kay.sievers@vrfy.org>,
       Alan Stern <stern@rowland.harvard.edu>,
       Jonathan Corbet <corbet@lwn.net>,
       Randy Dunlap <randy.dunlap@oracle.com>
Subject: Re: [RFC] Sample kset/ktype/kobject implementation
Message-ID: <20071128173532.00bc72af@gondolin.boeblingen.de.ibm.com>
In-Reply-To: <20071127230406.GD10038@kroah.com>
References: <20071127230252.GB10038@kroah.com>
	<20071127230406.GD10038@kroah.com>
Organization: IBM Deutschland Entwicklung GmbH Vorsitzender des
 Aufsichtsrats: Martin Jetter =?ISO-8859-15?Q?Gesch=E4ftsf=FChrung:?=
 Herbert Kircher Sitz der Gesellschaft: =?ISO-8859-15?Q?B=F6blingen?=
 Registergericht: Amtsgericht Stuttgart, HRB 243294
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1249
Lines: 44

On Tue, 27 Nov 2007 15:04:06 -0800,
Greg KH <greg@kroah.com> wrote:

> static struct foo_obj *create_foo_obj(const char *name)
> {
> 	struct foo_obj *foo;
> 	int retval;
> 
> 	/* allocate the memory for the whole object */
> 	foo = kzalloc(sizeof(*foo), GFP_KERNEL);
> 	if (!foo)
> 		return NULL;
> 
> 	/* initialize the kobject portion of the object properly */
> 	kobject_set_name(&foo->kobj, "%s", name);

Returncode not checked :)

> 	foo->kobj.kset = example_kset;
> 	foo->kobj.ktype = &foo_ktype;
> 
> 	/*
> 	 * Register the kobject with the kernel, all the default files will
> 	 * be created here and the uevent will be sent out.  If we were to
> 	 * call kobject_init() and then kobject_add() we would be
> 	 * responsible for sending out the initial KOBJ_ADD uevent.
> 	 */
> 	retval = kobject_register(&foo->kobj);
> 	if (retval) {
> 		kfree(foo);

kobject_put(foo) is needed since it gets you through kobject_cleanup()
where the name can be freed.

> 		return NULL;
> 	}
> 
> 	return foo;
> }
-
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/