Re: [PATCH 0/6] Add a per-dimm structure

[Greg KH]

On Wed, Mar 14, 2012 at 07:20:01PM -0300, Mauro Carvalho Chehab wrote:
> > Easy rule to remember, never mix "raw" kobjects and 'struct device',
> > which is what you would be doing here, right?  We can handle many
> > hundreds of thousands of files and devices in a single directory, no
> > problem.
> 
> No. They're all generated with raw kobjects, using kobject_init_and_add() or
> sysfs_create_file() calls.

Ick, no.

Please, never use "raw" kobjects.  Why doesn't 'struct device' work for
you properly here?  You just messed userspace all up in that it can not
properly account for the parts of sysfs here in the device tree.

Anything below /sys/devices/ should be using 'struct device' unless
there is a _very_ good reason not to.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/