diff options
author | Med Ismail Bennani <ismail@bennani.ma> | 2024-06-20 18:23:50 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-06-20 18:23:50 -0700 |
commit | 1373f7c714824f5957aa5fabf8370286f86e6b14 (patch) | |
tree | 23317d8c79f3aab5a444973cbadd46b13d06e4a0 /lldb | |
parent | 03921b979d67657bfc9cf8240add2484cc4df6a7 (diff) | |
download | llvm-1373f7c714824f5957aa5fabf8370286f86e6b14.zip llvm-1373f7c714824f5957aa5fabf8370286f86e6b14.tar.gz llvm-1373f7c714824f5957aa5fabf8370286f86e6b14.tar.bz2 |
Revert "[lldb/crashlog] Make interactive mode the new default" (#96263)
Reverts llvm/llvm-project#94575 since introduces test failure:
https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/as-lldb-cmake/6166/
Diffstat (limited to 'lldb')
6 files changed, 55 insertions, 83 deletions
diff --git a/lldb/examples/python/crashlog.py b/lldb/examples/python/crashlog.py index d3952e3..1c0d717 100755 --- a/lldb/examples/python/crashlog.py +++ b/lldb/examples/python/crashlog.py @@ -31,7 +31,6 @@ import argparse import concurrent.futures import contextlib import datetime -import enum import json import os import platform @@ -46,6 +45,7 @@ import threading import time import uuid + print_lock = threading.RLock() try: @@ -1582,12 +1582,9 @@ def load_crashlog_in_scripted_process(debugger, crashlog_path, options, result): debugger.RunCommandInterpreter(True, False, run_options, 0, False, True) -class CrashLogLoadingMode(str, enum.Enum): - batch = "batch" - interactive = "interactive" - - -def CreateSymbolicateCrashLogOptions(command_name, description): +def CreateSymbolicateCrashLogOptions( + command_name, description, add_interactive_options +): usage = "crashlog [options] <FILE> [FILE ...]" arg_parser = argparse.ArgumentParser( description=description, @@ -1604,12 +1601,6 @@ def CreateSymbolicateCrashLogOptions(command_name, description): ) arg_parser.add_argument( - "-m", - "--mode", - choices=[mode.value for mode in CrashLogLoadingMode], - help="change how the symbolicated process and threads are displayed to the user (default: interactive)", - ) - arg_parser.add_argument( "--version", "-V", dest="version", @@ -1745,35 +1736,36 @@ def CreateSymbolicateCrashLogOptions(command_name, description): help=argparse.SUPPRESS, default=False, ) - arg_parser.add_argument( - "--target", - "-t", - dest="target_path", - help="the target binary path that should be used for interactive crashlog (optional)", - default=None, - ) - arg_parser.add_argument( - "--skip-status", - "-s", - dest="skip_status", - action="store_true", - help="prevent the interactive crashlog to dump the process status and thread backtrace at launch", - default=False, - ) - legacy_group = arg_parser.add_mutually_exclusive_group() - legacy_group.add_argument( - "-i", - "--interactive", - action="store_true", - help=argparse.SUPPRESS, - ) - legacy_group.add_argument( - "-b", - "--batch", - action="store_true", - help=argparse.SUPPRESS, - ) - + if add_interactive_options: + arg_parser.add_argument( + "-i", + "--interactive", + action="store_true", + help="parse a crash log and load it in a ScriptedProcess", + default=False, + ) + arg_parser.add_argument( + "-b", + "--batch", + action="store_true", + help="dump symbolicated stackframes without creating a debug session", + default=True, + ) + arg_parser.add_argument( + "--target", + "-t", + dest="target_path", + help="the target binary path that should be used for interactive crashlog (optional)", + default=None, + ) + arg_parser.add_argument( + "--skip-status", + "-s", + dest="skip_status", + action="store_true", + help="prevent the interactive crashlog to dump the process status and thread backtrace at launch", + default=False, + ) return arg_parser @@ -1786,7 +1778,7 @@ for use at the LLDB command line. After a crash log has been parsed and symbolic created that has all of the shared libraries loaded at the load addresses found in the crash log file. This allows you to explore the program as if it were stopped at the locations described in the crash log and functions can be disassembled and lookups can be performed using the addresses found in the crash log.""" - return CreateSymbolicateCrashLogOptions("crashlog", description) + return CreateSymbolicateCrashLogOptions("crashlog", description, True) def SymbolicateCrashLogs(debugger, command_args, result, is_command): @@ -1802,35 +1794,8 @@ def SymbolicateCrashLogs(debugger, command_args, result, is_command): result.SetError(str(e)) return - # To avoid breaking existing users, we should keep supporting legacy flags - # even if we don't use them / advertise them anymore. - if not options.mode: - if options.batch: - options.mode = CrashLogLoadingMode.batch - else: - options.mode = CrashLogLoadingMode.interactive - - if options.mode != CrashLogLoadingMode.interactive and ( - options.target_path or options.skip_status - ): - print( - "Target path (-t) and skipping process status (-s) options can only used in interactive mode (-m=interactive)." - ) - print("Aborting symbolication.") - arg_parser.print_help() - return - - if options.version: - print(debugger.GetVersionString()) - return - - if options.debug: - print("command_args = %s" % command_args) - print("options", options) - print("args", options.reports) - # Interactive mode requires running the crashlog command from inside lldb. - if options.mode == CrashLogLoadingMode.interactive and not is_command: + if options.interactive and not is_command: lldb_exec = ( subprocess.check_output(["/usr/bin/xcrun", "-f", "lldb"]) .decode("utf-8") @@ -1856,24 +1821,31 @@ def SymbolicateCrashLogs(debugger, command_args, result, is_command): print(debugger.GetVersionString()) return + if options.debug: + print("command_args = %s" % command_args) + print("options", options) + print("args", options.reports) + if options.debug_delay > 0: print("Waiting %u seconds for debugger to attach..." % options.debug_delay) time.sleep(options.debug_delay) error = lldb.SBError() def should_run_in_interactive_mode(options, ci): - if options.mode == CrashLogLoadingMode.batch: - return False - elif ci and ci.IsInteractive(): + if options.interactive: return True + elif options.batch: + return False + # elif ci and ci.IsInteractive(): + # return True else: - return sys.stdout.isatty() + return False ci = debugger.GetCommandInterpreter() if options.reports: for crashlog_file in options.reports: - crashlog_path = os.path.normpath(os.path.expanduser(crashlog_file)) + crashlog_path = os.path.expanduser(crashlog_file) if not os.path.exists(crashlog_path): raise FileNotFoundError( "crashlog file %s does not exist" % crashlog_path diff --git a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/altered_threadState.test b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/altered_threadState.test index d925324..5a946a3 100644 --- a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/altered_threadState.test +++ b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/altered_threadState.test @@ -1,7 +1,7 @@ # RUN: %clang_host -g %S/Inputs/test.c -o %t.out # RUN: cp %S/Inputs/altered_threadState.crash %t.crash # RUN: %python %S/patch-crashlog.py --binary %t.out --crashlog %t.crash --offsets '{"main":20, "bar":9, "foo":16}' -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog -b %t.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog %t.crash' 2>&1 | FileCheck %s # CHECK: "crashlog" {{.*}} commands have been installed, use the "--help" options on these commands diff --git a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/json.test b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/json.test index d5c6d91..c2e23e8 100644 --- a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/json.test +++ b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/json.test @@ -2,12 +2,12 @@ # RUN: cp %S/Inputs/a.out.ips %t.crash # RUN: %python %S/patch-crashlog.py --binary %t.out --crashlog %t.crash --offsets '{"main":20, "bar":9, "foo":16}' --json -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog --mode batch %t.crash' 2>&1 | FileCheck %s -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog --mode batch -c %t.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog %t.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog -c %t.crash' 2>&1 | FileCheck %s # RUN: cp %S/Inputs/a.out.ips %t.nometadata.crash # RUN: %python %S/patch-crashlog.py --binary %t.out --crashlog %t.nometadata.crash --offsets '{"main":20, "bar":9, "foo":16}' --json --no-metadata -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog --mode batch %t.nometadata.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog %t.nometadata.crash' 2>&1 | FileCheck %s # CHECK: "crashlog" {{.*}} commands have been installed, use the "--help" options on these commands diff --git a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/no_threadState.test b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/no_threadState.test index 2e4b46c..5b5cef4 100644 --- a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/no_threadState.test +++ b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/no_threadState.test @@ -2,7 +2,7 @@ # RUN: cp %S/Inputs/no_threadState.ips %t.crash # RUN: %python %S/patch-crashlog.py --binary %t.out --crashlog %t.crash --offsets '{"main":20, "bar":9, "foo":16}' --json -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog --mode batch %t.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog %t.crash' 2>&1 | FileCheck %s # CHECK: "crashlog" {{.*}} commands have been installed, use the "--help" options on these commands diff --git a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/skipped_status_interactive_crashlog.test b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/skipped_status_interactive_crashlog.test index 52a185b..64cd090 100644 --- a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/skipped_status_interactive_crashlog.test +++ b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/skipped_status_interactive_crashlog.test @@ -3,7 +3,7 @@ # RUN: mkdir -p %t.dir # RUN: yaml2obj %S/Inputs/interactive_crashlog/multithread-test.yaml > %t.dir/multithread-test # RUN: %lldb -b -o 'command script import lldb.macosx.crashlog' \ -# RUN: -o 'crashlog -a -s -t %t.dir/multithread-test %S/Inputs/interactive_crashlog/multithread-test.ips' \ +# RUN: -o 'crashlog -a -i -s -t %t.dir/multithread-test %S/Inputs/interactive_crashlog/multithread-test.ips' \ # RUN: -o 'command source -s 0 %s' 2>&1 | FileCheck %s # CHECK: "crashlog" {{.*}} commands have been installed, use the "--help" options on these commands diff --git a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/text.test b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/text.test index eec30a1..e9d1c5e 100644 --- a/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/text.test +++ b/lldb/test/Shell/ScriptInterpreter/Python/Crashlog/text.test @@ -1,7 +1,7 @@ # RUN: %clang_host -g %S/Inputs/test.c -o %t.out # RUN: cp %S/Inputs/a.out.crash %t.crash # RUN: %python %S/patch-crashlog.py --binary %t.out --crashlog %t.crash --offsets '{"main":20, "bar":9, "foo":16}' -# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog -b %t.crash' 2>&1 | FileCheck %s +# RUN: %lldb %t.out -o 'command script import lldb.macosx.crashlog' -o 'crashlog %t.crash' 2>&1 | FileCheck %s # CHECK: "crashlog" {{.*}} commands have been installed, use the "--help" options on these commands |