lkml.org 
[lkml]   [2012]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] PM / Sleep: Print name of wakeup source that aborts suspend
Date
On Friday, July 20, 2012, Todd Poynor wrote:
> On Wed, Jul 18, 2012 at 09:47:34PM +0200, Rafael J. Wysocki wrote:
> > On Wednesday, July 18, 2012, Todd Poynor wrote:
> ...
> > > +{
> > > + struct wakeup_source *ws;
> > > + int active = 0;
> > > + struct wakeup_source *last_activity_ws = NULL;
> > > +
> > > + rcu_read_lock();
> > > + list_for_each_entry_rcu(ws, &wakeup_sources, entry) {
> > > + if (ws->active) {
> > > + pr_info("active wakeup source: %s\n", ws->name);
> > > + active = 1;
> >
> > Can we do a break here? Or do we want to get all of them?
>
> I haven't seen more than 3 activated during suspend entry at a time, but
> could just list one. I haven't made that change yet but will if it's
> preferred.

That depend on what your needs are. For debugging purposes it probably is
better to print them all.

Thanks,
Rafael


\
 
 \ /
  Last update: 2012-07-20 12:22    [W:0.042 / U:0.524 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site