Re: [PATCH 5/10] udlfb: Add functions to expose sysfs metrics and

On Thu, Feb 18, 2010 at 03:03:37PM -0800, Bernie Thompson wrote:

Hi Greg,

On Thu, Feb 18, 2010 at 7:57 AM, Greg KH [off-list ref] wrote:

quoted

All new sysfs files will need a Documentation/ABI/ patch as well to
explain the files.

I'll construct an additional patch to document the edid file (this
aught to be a standardized thing for all framebuffers, anyway - and
others have done similar/same).

If it's a standard framebuffer file, it should be documented already.
If not, that's a simple patch to get upstream :)

And I'll do a separate one for the individual metrics if they're kept
in sysfs in the future.

thanks, but I don't think they should be, and it looks like you don't
either.

quoted

quoted

metrics_misc

This isn't.  sysfs files are "one value per file".  If you want to do
something else, like this file, I suggest using debugfs instead.

The metrics stuff is a good candidate for moving to debugfs, I guess,
in a future patch. For now, anything in that set which violates sysfs
guidelines (metrics_misc) aught to just be removed, I guess.

These metrics have already been very helpful for comparing different
rendering algorithms, for debugging, and it's very helpful for
gathering info from end-users that may not have anything other than
sysfs set up. There are so many different hardware/software scenarios
that there's often no way to have any sense of what's happening
without more detailed instrumentation like this.

Then use debugfs for this, that is what it is explicitly for.  All
distros mount the thing now, so you can easily get information from it
from people who do not build their own kernels.  It's quite powerful.

thanks,

greg k-h