tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.
Apache License 2.0
473 stars 75 forks source link

Update watcher asserts to report assert filenames #6098

Open tt-dma opened 8 months ago

tt-dma commented 8 months ago

Current implementation can only report line numbers, since we don't want to store the file name strings in the limited device mailbox.

We can do this similarly to how it's done in budabackend, see: https://github.com/tenstorrent/tt-budabackend/blob/main/src/firmware/riscv/common/fw_debug.h https://github.com/tenstorrent/tt-budabackend/blob/main/src/firmware/riscv/toolchain/fwlog.py

Need to generate a .d file, see: d_file.patch

Process would be:

tt-dma commented 8 months ago

Mo has a way to do this for profiler, we should leverage that if we can. Also, update watcher pause feature to report filename + line as well.