diff mbox

[v2] hwsim tests: check kernel messages for lockdep complaints

Message ID 1384789086-25760-1-git-send-email-johannes@sipsolutions.net
State Accepted
Headers show

Commit Message

Johannes Berg Nov. 18, 2013, 3:38 p.m. UTC 
From: Johannes Berg <johannes.berg@intel.com>

Lockdep complaints are never good, so check for them in the
kernel messages, not just for warnings and bugs.

Signed-hostap: Johannes Berg <johannes.berg@intel.com>
---
 tests/hwsim/check_kernel.py | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

Comments

Jouni Malinen Nov. 20, 2013, 3:06 p.m. UTC | #1 
On Mon, Nov 18, 2013 at 04:38:06PM +0100, Johannes Berg wrote:
> Lockdep complaints are never good, so check for them in the
> kernel messages, not just for warnings and bugs.

Thanks, applied.
diff mbox

Patch

diff --git a/tests/hwsim/check_kernel.py b/tests/hwsim/check_kernel.py
index 30f6718..9c83833 100644
--- a/tests/hwsim/check_kernel.py
+++ b/tests/hwsim/check_kernel.py
@@ -13,7 +13,16 @@  Tests for kernel messages to find if there were any issues in them.
 
 import re
 
-issue = re.compile('(\[[0-9 .]*\] )?(WARNING:|BUG:).*')
+lockdep_messages = [
+  'possible circular locking dependency',
+  '.*-safe -> .*unsafe lock order detected',
+  'possible recursive locking detected',
+  'inconsistent lock state',
+  'possible irq lock inversion dependency',
+  'suspicious RCU usage',
+]
+lockdep = r'(\[\s*)?INFO: (%s)' % ('|'.join(lockdep_messages), )
+issue = re.compile('(\[[0-9 .]*\] )?(WARNING:|BUG:|%s).*' % lockdep)
 
 def check_kernel(logfile):
     for line in open(logfile, 'r'):