[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 10 Oct 2007 10:30:22 -0700
From: Greg KH <greg@...ah.com>
To: Martin Bligh <mbligh@...igh.org>
Cc: Tejun Heo <htejun@...il.com>, ebiederm@...ssion.com,
cornelia.huck@...ibm.com, stern@...land.harvard.edu,
kay.sievers@...y.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and driver
model
On Wed, Oct 10, 2007 at 10:24:43AM -0700, Martin Bligh wrote:
>> The rules for sysfs files are the following:
>> - one value, in text format, per file.
>> - no action apon open/close
>> - binary files are only allowed for "pass-through" type files
>> that the kernel does not touch (like for firmware and pci
>> config space)
>> - directories should be associated with a kobject where it makes
>> sense (no nesting deep subdirectories without a kobject
>> present)
>> - when a directory is created/removed, a uevent should happen
>> declaring what type of device was created/removed.
>
> So you'll be removing:
>
> /sys/devices/system/node/node?/meminfo
>
> then?
>
> along with:
>
> /sys/devices/system/node/node?/distance
> /sys/devices/system/node/node?/numastat
>
> and all the other things that violate the rules?
I would love to do that :)
And that goes to show how trying to enforce these kinds of rules is damm
hard. Things slip by that I never notice because they are only for odd
types of hardware :)
> (which I do agree with ... I just don't think sysfs works for
> performance stats as we've discussed several times before ;-))
I agree that this doesn't work too, but also that if it's really needed,
it can be done, just let us know about it (like
/sys/block/BLOCKDEV/stat)
thanks,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists