Add arch support for tombstone.py

build/android/tombstones.py

 #!/usr/bin/env python
 #
 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 #
 # Find the most recent tombstone file(s) on all connected devices
 # and prints their stacks.
 #
 # Assumes tombstone file was created with current symbols.
 
 import datetime
 import multiprocessing
 import os
+import re
 import subprocess
 import sys
 import optparse
 
 from pylib import android_commands
 from pylib.device import device_utils
 
 
 def _ListTombstones(device):
   """List the tombstone files on the device.
@@ skipping 20 matching lines @@
     device: An instance of DeviceUtils.
 
   Returns:
     A datetime instance.
   """
   device_now_string = device.RunShellCommand('TZ=UTC date')
   return datetime.datetime.strptime(
       device_now_string[0], '%a %b %d %H:%M:%S %Z %Y')
 
 
+def _GetDeviceArch(device):

[jbudorick, 2014/07/25 14:19:02]

This function isn't needed.

+  """Get CPU architecture of the device.
+
+  Args:
+    device: An instance of DeviceUtils.
+
+  Returns:
+    CPU architecture string.
+  """
+  lines = device.RunShellCommand('getprop')
+  for line in lines:
+    if 'ro.product.cpu.abi' in line:
+      return re.sub('[\[\]]', '', line.split(':')[1].strip())
+
+
 def _GetTombstoneData(device, tombstone_file):
   """Retrieve the tombstone data from the device
 
   Args:
     device: An instance of DeviceUtils.
     tombstone_file: the tombstone to retrieve
 
   Returns:
     A list of lines
   """
   return device.ReadFile('/data/tombstones/' + tombstone_file, as_root=True)
 
 
 def _EraseTombstone(device, tombstone_file):
   """Deletes a tombstone from the device.
 
   Args:
     device: An instance of DeviceUtils.
     tombstone_file: the tombstone to delete.
   """
   return device.RunShellCommand(
       'rm /data/tombstones/' + tombstone_file, as_root=True)
 
 
-def _ResolveSymbols(tombstone_data, include_stack):
+def _ResolveSymbols(tombstone_data, include_stack, arch):
   """Run the stack tool for given tombstone input.
 
   Args:
     tombstone_data: a list of strings of tombstone data.
     include_stack: boolean whether to include stack data in output.

[jbudorick, 2014/07/25 14:19:02]

Please add a description for arch here.

 
   Yields:
     A string for each line of resolved stack output.
   """
   stack_tool = os.path.join(os.path.dirname(__file__), '..', '..',
                             'third_party', 'android_platform', 'development',
                             'scripts', 'stack')
-  proc = subprocess.Popen(stack_tool, stdin=subprocess.PIPE,
+  proc = subprocess.Popen([stack_tool, '--arch', arch], stdin=subprocess.PIPE,
                           stdout=subprocess.PIPE)
   output = proc.communicate(input='\n'.join(tombstone_data))[0]
   for line in output.split('\n'):
     if not include_stack and 'Stack Data:' in line:
       break
     yield line
 
 
 def _ResolveTombstone(tombstone):
   lines = []
   lines += [tombstone['file'] + ' created on ' + str(tombstone['time']) +
             ', about this long ago: ' +
             (str(tombstone['device_now'] - tombstone['time']) +
             ' Device: ' + tombstone['serial'])]
   print '\n'.join(lines)
   print 'Resolving...'
-  lines += _ResolveSymbols(tombstone['data'], tombstone['stack'])
+  lines += _ResolveSymbols(tombstone['data'], tombstone['stack'],
+                           tombstone['arch'])
   return lines
 
 
 def _ResolveTombstones(jobs, tombstones):
   """Resolve a list of tombstones.
 
   Args:
     jobs: the number of jobs to use with multiprocess.
     tombstones: a list of tombstones.
   """
@@ skipping 24 matching lines @@
 
   # Sort the tombstones in date order, descending
   all_tombstones.sort(cmp=lambda a, b: cmp(b[1], a[1]))
 
   # Only resolve the most recent unless --all-tombstones given.
   tombstones = all_tombstones if options.all_tombstones else [all_tombstones[0]]
 
   device_now = _GetDeviceDateTime(device)
   for tombstone_file, tombstone_time in tombstones:
     ret += [{'serial': str(device),
+             'arch': _GetDeviceArch(device),

[jbudorick, 2014/07/25 14:19:02]

  _GetDeviceArch(device)

should be replaced by

  device.GetProp('ro.product.cpu.api')

              'device_now': device_now,
              'time': tombstone_time,
              'file': tombstone_file,
              'stack': options.stack,
              'data': _GetTombstoneData(device, tombstone_file)}]
 
   # Erase all the tombstones if desired.
   if options.wipe_tombstones:
     for tombstone_file, _ in all_tombstones:
       _EraseTombstone(device, tombstone_file)
 
   return ret
 
+
 def main():
   parser = optparse.OptionParser()
   parser.add_option('--device',
                     help='The serial number of the device. If not specified '
                          'will use all devices.')
   parser.add_option('-a', '--all-tombstones', action='store_true',
                     help="""Resolve symbols for all tombstones, rather than just
                          the most recent""")
   parser.add_option('-s', '--stack', action='store_true',
                     help='Also include symbols for stack data')
@@ skipping 12 matching lines @@
 
   tombstones = []
   for device_serial in devices:
     device = device_utils.DeviceUtils(device_serial)
     tombstones += _GetTombstonesForDevice(device, options)
 
   _ResolveTombstones(options.jobs, tombstones)
 
 if __name__ == '__main__':
   sys.exit(main())