From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH] PM: add statistics sysfs file for suspend to ram Date: Thu, 4 Aug 2011 21:50:42 +0200 Message-ID: <201108042150.42926.rjw@sisk.pl> References: <6E3BC7F7C9A4BF4286DD4C043110F30B5B790E5741@shsmsx502.ccr.corp.intel.com> <20110804121728.9280c9af.rdunlap@xenotime.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20110804121728.9280c9af.rdunlap@xenotime.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: linux-pm@lists.linux-foundation.org Cc: "Liu, ShuoX" , "gregkh@suse.de" , "Brown, Len" List-Id: linux-pm@vger.kernel.org On Thursday, August 04, 2011, Randy Dunlap wrote: > On Thu, 04 Aug 2011 15:05:10 -0400 (EDT) Len Brown wrote: > > > > Record S3 failure time about each reason and the latest two failed > > > devices' name in S3 progress. > > > We can check it through /sys/power/suspend_stats. > > > > these sound more like debugfs than sysfs things. > > debugfs also allows more than one value per file, like I think > I saw in this patch. However, I still would like to know what's insufficient in the current debug stuff in the PM core that makes this new code necessary. Thanks, Rafael