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

On Mon, Feb 15, 2010 at 06:46:08AM -0800, Bernie Thompson wrote:

Add sysfs controls for edid and performance metrics

There are 8 new files exposed in /sys/class/graphics/fb*

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

edid - returns 128 byte edid blog, suitable for parsing with parse-edid
metrics_bytes_identical
metrics_bytes_rendered
metrics_bytes_sent
metrics_cpu_kcycles_used

These are all single value files, which is fine.

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.

and metrics_reset, which resets all perf metrics to zero

Actually, why not use debugfs for all of these files?  They aren't
really necessary for any user or tool, and are only a debugging thing.
That's what debugfs was created for :)

thanks,

greg k-h