stack: Adjust Pre-M Android incorrect debuggerd addresses.

third_party/android_platform/development/scripts/stack

 #!/usr/bin/env python
 #
 # Copyright (C) 2013 The Android Open Source Project
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #      http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
 """stack symbolizes native crash dumps."""
 
 import getopt
 import glob
 import logging
 import os
 import sys
 
 import stack_core
+import stack_libs
 import subprocess
 import symbol
 import sys
 
 DEFAULT_SYMROOT='/tmp/symbols'
+DEFAULT_APK='chrome_apk'
 
 def PrintUsage():
   """Print usage and exit with error."""
   # pylint: disable-msg=C6310
   print
   print "  usage: " + sys.argv[0] + " [options] [FILE]"
   print
   print "  --symbols-dir=path"
   print "       the path to a symbols dir, such as =/tmp/out/target/product/dream/symbols"
   print
   print "  --chrome-symbols-dir=path"
   print "       the path to a Chrome symbols dir (can be absolute or relative"
   print "       to src), such as =out/Debug/lib"
   print "       If not specified, will look for the newest lib in out/Debug or"
   print "       out/Release"
   print
+  print "  --packed-relocation-adjustments"
+  print "  --no-packed-relocation-adjustments"
+  print "       turn packed relocation adjustment on and off (default is off)"
+  print "       If running on pre-M Android and the stack trace appears to"
+  print "       make no sense, try turning this feature on."
+  print
+  print "  --chrome-apk-dir=path"
+  print "       the path to the APK staging dir (can be absolute or relative"
+  print "       to src), such as =out/Debug/chrome_apk"
+  print "       If not specified, uses =|chrome-symbols-dir|/../chrome_apk"
+  print "       Parses libraries here to find data for adjusting debuggerd"
+  print "       tombstones where relocations are packed."
+  print "       Enable/disable with --[no-]packed-relocation-adjustments."
+  print
   print "  --symbols-zip=path"
   print "       the path to a symbols zip file, such as =dream-symbols-12345.zip"
   print
   print "  --more-info"
   print "  --less-info"
   print "       Change the level of detail in the output."
   print "       --more-info is slower and more verbose, but more functions will"
   print "       be fully qualified with namespace/classname and have full"
   print "       argument information. Also, the 'stack data' section will be"
   print "       printed."
@@ skipping 51 matching lines @@
   else:
     # This is a zip of Chrome symbols, so symbol.CHROME_SYMBOLS_DIR needs to be
     # updated to point here.
     symbol.CHROME_SYMBOLS_DIR = symdir
     return (symdir, symdir)
 
 
 def main(argv):
   try:
     options, arguments = getopt.getopt(argv, "",
-                                       ["more-info",
+                                       ["packed-relocation-adjustments",
+                                        "no-packed-relocation-adjustments",

[rmcilroy, 2015/10/23 16:25:10]

Do we really need no-packed-relocation-adjustments? The default is False  so we
should only need the packed-relocation-adjustments flag to sets it to true.

[simonb (inactive), 2015/10/26 12:38:31]

We don't *need* it. It's here for symmetry with the existing
--more-info/--less-info, which behaves the same.

[rmcilroy, 2015/10/26 16:42:17]

I'm not convinced the symmetry adds anything, even when more/less-info has that
symmetry. Unless we need it I would just remove it.

[simonb (inactive), 2015/10/27 11:12:27]

Now useful as a way to override any state that may be incorrectly auto-detected
from parsing the build fingerprint (introduced in patch set 4).

+                                        "more-info",
                                         "less-info",
                                         "chrome-symbols-dir=",
+                                        "chrome-apk-dir=",
                                         "symbols-dir=",
                                         "symbols-zip=",
                                         "arch=",
                                         "verbose",
                                         "help"])
   except getopt.GetoptError, unused_error:
     PrintUsage()
 
   zip_arg = None
   more_info = False
+  chrome_apk_dir = symbol.CHROME_SYMBOLS_DIR
+  chrome_apk_dir_supplied = False
+  packed_relocation_adjustments = False
   for option, value in options:
     if option == "--help":
       PrintUsage()
     elif option == "--symbols-dir":
       symbol.SYMBOLS_DIR = os.path.expanduser(value)
     elif option == "--symbols-zip":
       zip_arg = os.path.expanduser(value)
     elif option == "--arch":
       symbol.ARCH = value
     elif option == "--chrome-symbols-dir":
       symbol.CHROME_SYMBOLS_DIR = os.path.join(symbol.CHROME_SYMBOLS_DIR, value)
+    elif option == "--chrome-apk-dir":
+      chrome_apk_dir = os.path.join(chrome_apk_dir, value)
+      chrome_apk_dir_supplied = True
+    elif option == "--packed-relocation-adjustments":
+      packed_relocation_adjustments = True
+    elif option == "--no-packed-relocation-adjustments":
+      packed_relocation_adjustments = False
     elif option == "--more-info":
       more_info = True
     elif option == "--less-info":
       more_info = False
     elif option == "--verbose":
       logging.basicConfig(level=logging.DEBUG)
 
+  if not chrome_apk_dir_supplied:
+    chrome_apk_dir = os.path.join(symbol.CHROME_SYMBOLS_DIR, '..', DEFAULT_APK)

[rmcilroy, 2015/10/23 16:25:10]

nit - just make should chrome_apk_dir be chrome_apk_path? Also, could you just
set:
  chrome_apk_path = os.path.join(symbol.CHROME_SYMBOLS_DIR, '..', DEFAULT_APK)
on line 150, and update it in the else block for "--chrome-apk-dir" (which
should possibly also be --chrome-apk-path) ?

[simonb (inactive), 2015/10/26 12:38:31]

Again, symmetry. In this case with the existing --chrome-symbols-dir. Both are
directory paths.
Not really. This isn't the correct value onto which to join any value supplied
with --chrome-apk-dir (line 165).

The confusion occurs because this loop may (counter-intuitively?) alter
symbol.CHROME_SYMBOLS_DIR if --chrome-symbols-dir. I didn't want to disturb that
existing code.

[rmcilroy, 2015/10/26 16:42:17]

dir is fine - my confusion stemed from the fact that it looked like you were
appending an APK filename, so it was no longer a directory - could you rename
DEFAULT_APK to DEFAULT_APK_DIR to make this more obvious.
Ahh, modifying CHROME_SYMBOLS_DIR which looks like a constant, nice... ;). OK,
in that case could you have a seperate variable for chrome-apk-dir (i.e., have
it set chrome-apk-dir-suffix) and just append it to chrome_apk_dir once after
the loop.

[simonb (inactive), 2015/10/27 11:12:27]

Done

+
   if len(arguments) > 1:
     PrintUsage()
 
   if not arguments or arguments[0] == "-":
     print "Reading native crash info from stdin"
     f = sys.stdin
   else:
-    print "Searching for native crashes in %s" % arguments[0]
+    print "Searching for native crashes in: " + os.path.realpath(arguments[0])
     f = open(arguments[0], "r")
 
   lines = f.readlines()
   f.close()
 
   rootdir = None
   if zip_arg:
     rootdir, symbol.SYMBOLS_DIR = UnzipSymbols(zip_arg)
 
-  print "Reading Android symbols from", symbol.SYMBOLS_DIR
-  print "Reading Chrome symbols from", symbol.CHROME_SYMBOLS_DIR
-  stack_core.ConvertTrace(lines, more_info)
+  if packed_relocation_adjustments:
+    print ("Reading Chrome APK library data from: "
+           + os.path.normpath(chrome_apk_dir))
+    load_vaddrs = stack_libs.GetLoadVaddrs(chrome_apk_dir)
+  else:
+    load_vaddrs = {}
+
+  print ("Reading Android symbols from: "
+         + os.path.normpath(symbol.SYMBOLS_DIR))
+  print ("Reading Chrome symbols from: "
+         + os.path.normpath(symbol.CHROME_SYMBOLS_DIR))
+  stack_core.ConvertTrace(lines, load_vaddrs, more_info)
 
   if rootdir:
     # be a good citizen and clean up...os.rmdir and os.removedirs() don't work
     cmd = "rm -rf \"%s\"" % rootdir
     print "\ncleaning up (%s)" % cmd
     os.system(cmd)
 
 if __name__ == "__main__":
   sys.exit(main(sys.argv[1:]))
 
 # vi: ts=2 sw=2