Messages in this thread | | | Date | Sun, 25 Apr 2010 12:53:01 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH 5/9] PM: suspend_block: Add debugfs file |
| |
On 04/25/10 11:15, Greg KH wrote: > On Fri, Apr 23, 2010 at 01:58:53PM -0700, Randy Dunlap wrote: >> On Thu, 22 Apr 2010 18:08:54 -0700 Arve Hj?nnev?g wrote: >> >>> Report active and inactive suspend blockers in >>> /sys/kernel/debug/suspend_blockers. >> >> >> This looks more like user interface information than it does debug information. >> If that's correct, it shouldn't be in debugfs. > > It's debug-like information, and has more than one value per file, so > debugfs seems like the proper place for it. I have no objection to it > going there.
I have no objection if it really is debug info, but I'm not convinced of that yet.
-- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***
| |