Messages in this thread | | | Date | Wed, 14 Mar 2012 16:32:25 -0700 | From | Greg KH <> | Subject | Re: [PATCH 0/6] Add a per-dimm structure |
| |
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
| |