Fix kernel testing and make it target specific.

client/bin/site_utils.py

 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-import logging, os, platform, time
+import logging, os, platform, re, tempfile, time
 from autotest_lib.client.common_lib import error
+from autotest_lib.client.common_lib import utils
 
 
 class TimeoutError(error.TestError):
     """Error raised when we time out when waiting on a condition."""
 
 
+class Crossystem(object):
+    """A wrapper for the crossystem utility."""
+
+    def __init__(self, client):
+        self.cros_system_data = {}
+        self._client = client
+
+    def init(self):
+        self.cros_system_data = {}
+        (fd, fname) = tempfile.mkstemp()
+        f = open(fname, 'w')

[Mandeep Singh Baines, 2011/04/07 21:15:42]

Why not use the fd from above?

[vb, 2011/04/07 21:34:59]

It gets messy with files opened on client/host, etc.

+        self._client.run('crossystem', stdout_tee=f)
+        f.close()
+        text = open(fname).read()

[Mandeep Singh Baines, 2011/04/07 21:15:42]

Use utils.read_file() instead.

[vb, 2011/04/07 21:34:59]

Done.

+        for line in text.splitlines():
+            assignment_string = line.strip().split('#')[0]

[Mandeep Singh Baines, 2011/04/07 21:15:42]

The strip here seems unnecessary.

[vb, 2011/04/07 21:34:59]

true, done.

+            if not assignment_string.count('='):
+                continue
+            (name, value) = assignment_string.split('=', 1)
+            self.cros_system_data[name.strip()] = value.strip()
+        os.remove(fname)
+
+    def __getattr__(self, name):
+        """
+        Retrieve a crosssystem attribute.
+
+        The call crossystemobject.name() will return the crossystem reported
+        string.
+        """
+        return lambda : self.cros_system_data[name]
+
+
 def poll_for_condition(
     condition, exception=None, timeout=10, sleep_interval=0.1, desc=None):
     """Poll until a condition becomes true.
 
     condition: function taking no args and returning bool
     exception: exception to throw if condition doesn't become true
     timeout: maximum number of seconds to wait
     sleep_interval: time to sleep between polls
     desc: description of default TimeoutError used if 'exception' is None
 
@@ skipping 47 matching lines @@
     which do not appear in the whitelist.
 
     Arguments:
       dmesg - string containing raw dmesg buffer
       message_level - minimum message priority to check
       whitelist - messages to ignore
 
     Returns:
       List of unexpected warnings
     """
-
+    whitelist_re = re.compile(r'(%s)' % '|'.join(whitelist))
     unexpected = []
     for line in dmesg.splitlines():
         if int(line[1]) <= message_level:
-            if not 'used greatest stack depth' in line:
-                stripped_line = line.split('] ', 1)[1]
-                if not stripped_line in whitelist:
-                    unexpected.append(stripped_line)
+            stripped_line = line.split('] ', 1)[1]
+            if whitelist_re.search(stripped_line):
+                continue
+            unexpected.append(stripped_line)
     return unexpected