✨ Add interactive stack trace

pros/cli/terminal.py

@@ -30,6 +30,7 @@ def terminal_cli():
               help='Specify 2 ports for the "share" backend. The default option deterministically selects ports '
                    'based on the serial port name')
 @click.option('--banner/--no-banner', 'request_banner', default=True)
+@click.option('--auto_stack_trace', default=True)
 @click.option('--output', nargs = 1, type=str, is_eager = True, help='Redirect terminal output to a file', default=None)
 
 def terminal(port: str, backend: str, **kwargs):
@@ -83,7 +84,7 @@ def terminal(port: str, backend: str, **kwargs):
         device = devices.RawStreamDevice(ser)
     else:
         device = devices.vex.V5UserDevice(ser)
-    term = Terminal(device, request_banner=kwargs.pop('request_banner', True))
+    term = Terminal(device, request_banner=kwargs.pop('request_banner', True), request_banner=kwargs.pop('auto_stack_trace', True))
 
     class TerminalOutput(object):
         def __init__(self, file):

pros/serial/terminal/stack_trace.txt

@@ -0,0 +1,10 @@
+    0x7800184 : 5apple5faultEv at main.cpp:27 : ?? : ??
+    0x381a484 : ?? : ouch_bits_update_released at llemu.c:49 : vmap at lv_draw_vbasic.c:431
+    0x382ae60 : ?? : btn_signal at lv_btn.c:616 : ??
+    0x381bbe4 : ?? : ev_proc_release at lv_indev.c:722 : _init at lv_theme_alien.c:165
+    0x381c240 : ?? : ev_button_proc at lv_indev.c:571 : endar_init at lv_theme_alien.c:438
+    0x3828718 : ?? : task_exec at lv_task.c:332 : ??
+    0x38288d8 : ?? : task_handler at lv_task.c:135 : ??
+    0x381a098 : ?? : p_daemon at display.c:25 (discriminator 1) : vletter at lv_draw_vbasic.c:312
+    0x384b8cc : ?? : k_fn_wrapper at cpp_support.cpp:26 : ??
+    0x3845d5c : ?? : k_clean_up at port.c:219 : ??

pros/serial/terminal/terminal.py

@@ -1,6 +1,7 @@
 import codecs
 import os
 import signal
+import subprocess
 import sys
 import threading
 
@@ -165,12 +166,11 @@ def cleanup(self):
         'Sorry no implementation for your platform ({})'
         ' available.'.format(sys.platform))
 
-
 class Terminal(object):
     """This class is loosely based off of the pyserial miniterm"""
 
     def __init__(self, port_instance: StreamDevice, transformations=(),
-                 output_raw: bool = False, request_banner: bool = True):
+                 output_raw: bool = False, request_banner: bool = True, auto_stack_trace: bool = True):
         self.device = port_instance
         self.device.subscribe(b'sout')
         self.device.subscribe(b'serr')
@@ -208,7 +208,7 @@ def _start_tx(self):
     def _stop_tx(self):
         self.console.cancel()
         self._transmitter_alive = False
-        self.transmitter_thread.join()
+        self.transmitter_thread.join()   
 
     def reader(self):
         if self.request_banner:
@@ -225,10 +225,32 @@ def reader(self):
                     text = decode_bytes_to_str(data[1])
                 elif data[0] == b'serr':
                     text = '{}{}{}'.format(colorama.Fore.RED, decode_bytes_to_str(data[1]), colorama.Style.RESET_ALL)
+
                 elif data[0] == b'kdbg':
                     text = '{}\n\nKERNEL DEBUG:\t{}{}\n'.format(colorama.Back.GREEN + colorama.Style.BRIGHT,
                                                                 decode_bytes_to_str(data[1]),
                                                                 colorama.Style.RESET_ALL)
+                    if (self.auto_stack_trace):
+                        start = text.find("BEGIN STACK TRACE") + 18
+                        end = text.find("END OF TRACE")
+                        addrArray = text[start: end].split()
+                        out = ''
+                        for i, s in enumerate(addrArray):
+                            def getTrace(s, path):
+                                temp = subprocess.Popen(['addr2line', '-faps', '-e', path, s],
+                                    stdout=subprocess.PIPE).communicate()[0].decode('utf-8')
+                                if (temp.find('?') != -1):
+                                    return ' : ??'
+                                else:
+                                    return ' : ' + temp[15: len(temp)-2]
+                            out += "    " + s + getTrace(s, "..\..\..\\test-project2\\bin\hot.package.elf")
+                            out += getTrace(s, "..\..\..\\test-project2\\bin\cold.package.elf")
+                            out += getTrace(s, "..\..\..\\test-project2\\bin\monolith.elf") + '\n'    
+                        text = text[:start] + out + text[end:]
+                        print(text)
+                        file = open("stack_trace.txt", "w")
+                        file.write(out)
+                        file.close()
                 elif data[0] != b'':
                     text = '{}{}'.format(decode_bytes_to_str(data[0]), decode_bytes_to_str(data[1]))
                 else:

test-project2/.gitignore

@@ -0,0 +1,16 @@
+# Compiled Object files
+*.o
+*.obj
+
+# Executables
+*.bin
+*.elf
+
+# PROS
+bin/
+.vscode/
+compile_commands.json
+temp.log
+temp.errors
+*.ini
+.d/

test-project2/Makefile

@@ -0,0 +1,45 @@
+################################################################################
+######################### User configurable parameters #########################
+# filename extensions
+CEXTS:=c
+ASMEXTS:=s S
+CXXEXTS:=cpp c++ cc
+
+# probably shouldn't modify these, but you may need them below
+ROOT=.
+FWDIR:=$(ROOT)/firmware
+BINDIR=$(ROOT)/bin
+SRCDIR=$(ROOT)/src
+INCDIR=$(ROOT)/include
+
+WARNFLAGS+=
+EXTRA_CFLAGS=
+EXTRA_CXXFLAGS=
+
+# Set to 1 to enable hot/cold linking
+USE_PACKAGE:=1
+
+# Add libraries you do not wish to include in the cold image here
+# EXCLUDE_COLD_LIBRARIES:= $(FWDIR)/your_library.a
+EXCLUDE_COLD_LIBRARIES:= 
+
+# Set this to 1 to add additional rules to compile your project as a PROS library template
+IS_LIBRARY:=0
+# TODO: CHANGE THIS!
+LIBNAME:=libbest
+VERSION:=1.0.0
+# EXCLUDE_SRC_FROM_LIB= $(SRCDIR)/unpublishedfile.c
+# this line excludes opcontrol.c and similar files
+EXCLUDE_SRC_FROM_LIB+=$(foreach file, $(SRCDIR)/main,$(foreach cext,$(CEXTS),$(file).$(cext)) $(foreach cxxext,$(CXXEXTS),$(file).$(cxxext)))
+
+# files that get distributed to every user (beyond your source archive) - add
+# whatever files you want here. This line is configured to add all header files
+# that are in the the include directory get exported
+TEMPLATE_FILES=$(INCDIR)/**/*.h $(INCDIR)/**/*.hpp
+
+.DEFAULT_GOAL=quick
+
+################################################################################
+################################################################################
+########## Nothing below this line should be edited by typical users ###########
+-include ./common.mk