suspend: use WARN not WARN_ON to print the message
authorArjan van de Ven <arjan@linux.intel.com>
Tue, 18 Nov 2008 14:56:51 +0000 (06:56 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 18 Nov 2008 16:07:36 +0000 (08:07 -0800)
By using WARN(), kerneloops.org can collect which component is causing
the delay and make statistics about that. suspend_test_finish() is
currently the number 2 item but unless we can collect who's causing
it we're not going to be able to fix the hot topic ones..

Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
kernel/power/main.c

index 19122cf..b8f7ce9 100644 (file)
@@ -174,7 +174,7 @@ static void suspend_test_finish(const char *label)
         * has some performance issues.  The stack dump of a WARN_ON
         * is more likely to get the right attention than a printk...
         */
-       WARN_ON(msec > (TEST_SUSPEND_SECONDS * 1000));
+       WARN(msec > (TEST_SUSPEND_SECONDS * 1000), "Component: %s\n", label);
 }
 
 #else