- Table of Contents
Thank you for your interest in this project.
If you are interested in making a contribution, then please familiarize yourself with our technical contribution standards as set forth in this guide.
Next, please request appropriate write permissions by opening an issue for GitHub permissions.
All contributions require:
- an issue
- Your issue should be filed under an appropriate project. Please file a feature support request or bug report under Issues to get help with finding an appropriate project to get a maintainer's attention.
- a pull request (PR).
- Your PR must be approved by appropriate reviewers.
Furthermore, all PRs must follow the contribution standards.
Hugepages is required to both run and develop on the Metalium project.
If you ever need to re-enable Hugepages, you can try the script we homemade for this:
sudo python3 infra/machine_setup/scripts/setup_hugepages.py enable
Then to check if Hugepages is enabled:
python3 infra/machine_setup/scripts/setup_hugepages.py check
Currently, the most convenient way to develop is to do so on our cloud machines. They have prerequisite dependencies, model files, and other settings set up for users.
Please refer to the README for source installation and environment setup instructions, then please read the the Getting Started page.
We use #
as a special character to denote issue numbers in our commit
messages. Please change your comment character in your Git to not conflict with
this:
git config core.commentchar ">"
In order to get debug level log messages, set the environment variable
TT_METAL_LOGGER_LEVEL=Debug
.
For example,
TT_METAL_LOGGER_LEVEL=Debug ./build/test/tt_metal/test_add_two_ints
-
First, ensure that you have built the project and activated the Python environment, along with any required
PYTHONPATH
variables. -
Build the HTML documentation.
cd docs
make clean
make html
You can optionally build and view the ttnn sweeps results with:
make ttnn_sweeps/check_directory
make ttnn_sweeps
then turn on the server to view.
make server
You can customize the port by using the PORT=<port>
environment variable. If
you're using a customer-facing cloud machine, please disregard this point.
- Navigate to the docs page.
Navigate your web browser to http://<ip address>:<port>
, where <ip address>
is the IP address of the machine on which you launched the web server. For
example: http://10.250.37.37:4242
, for port 4242
.
If you forwarded your port, navigate to http://localhost:8888
.
- If you make changes, you may need to check spelling errors.
We use the spell-checker, Aspell, to ensure we don't sneak in some typos in our documentation. This is enforced by static-checks on github workflows as well.
To check if your updated docs pass this check you can run,
$ cd ${TT_METAL_HOME} && ./docs/spellcheck.sh
If there are errors in this check you will see an exit code non-zero.
To update the documentation for spelling errors or any out-of-dictionary words you can run,
$ cd ${TT_METAL_HOME} && ./docs/spellcheck.sh update
Commit your changes and the personal dictionary, at docs/aspell-dictionary.pws, that is changed.
Ensure you're in a developer Python environment with necessary environment variables set as documentating in the developing section.
This includes the environment variables, Python dev environment etc.
All developers are responsible for ensuring that post-commit regressions pass upon any submission to the project. We will cover how to run these regressions both locally and on CI. Failure to ensure these tests pass will constitute a major regression and will likely mean reverting your commits.
You must run post-commit regressions before you commit something.
These regressions will also run after every pushed commit to the GitHub repo.
cmake --build build --target install
cmake --build build --target tests
./tests/scripts/run_tests.sh --tt-arch $ARCH_NAME --pipeline-type post_commit
If changes affect tensor
or tt_dnn
libraries, run this suite of pytests
which tests tensor
APIs and tt_dnn
ops. These are also tested in post
commit.
pytest tests/python_api_testing/unit_testing/ -vvv
pytest tests/python_api_testing/sweep_tests/pytests/ -vvv
If you would like to run the post-commit tests on GitHub Actions, please refer to using CI for development.
Make sure to add post-commit tests in the at the lowest two levels of the tests directory to make sure tests are executed on the workflows.
New shell scripts added above the lowest two levels may not be executed on the post-commit workflows!
After building the repo and activating the dev environment with the appropriate environment variables, you have two options for running performance regressions on model tests.
If you are using a machine with virtual machine specs, please use
./tests/scripts/run_tests.sh --tt-arch $ARCH_NAME --pipeline-type models_performance_virtual_machine
If you are using a machine with bare metal machine specs, please use
./tests/scripts/run_tests.sh --tt-arch $ARCH_NAME --pipeline-type models_performance_bare_metal
We have a legacy suite of C++ integration tests that are built like standalone executables. This section goes over how to generally run such tests if there's a specific one you'd like to run.
- Build the API integration tests using the make command,
cmake --build build --target tests
- Run the test binaries from the path ${TT_METAL_HOME}/build/test/tt_metal
The new fangled way we run our tests is with Googletest. The way we generally structure our tests with this framework is to bundle it into a single executable.
You can use --gtest_filter_test
to filter out the specific test you'd like.
For example, to build and run the CommonFixture.DRAMLoopbackSingleCore
on
fast dispatch, you can
- Build the unit tests:
cmake --build build --target tests
- Run the test:
./build/test/tt_metal/unit_tests_fast_dispatch --gtest_filter="CommonFixture.DRAMLoopbackSingleCore"
On slow dispatch, to run another specific test, the equivalent would be:
- Build the unit tests as you would above.
- Run with the slow dispatch mode:
export TT_METAL_SLOW_DISPATCH_MODE=1 ./build/test/tt_metal/unit_tests/fast_dispatch --gtest_filter_test="BasicFixture.TestL1BuffersAllocatedTopDown"
We have split our tests into the two dispatch modes for less pollution of state between the two. We would like to eventually enable switching between the two modes easily.
We use pytest to run our Python-based tests. This is the general procedure for running such tests.
- Run the specific test point with pytest tool, e.g.
$ pytest tests/tt_eager/python_api_testing/sweep_tests/pytests/tt_dnn/test_composite.py
- If you have any issues with import paths for python libraries include the following environment variable,
$ export PYTHONPATH=${PYTHONPATH}:${TT_METAL_HOME}
- GDB can be used to debug Metalium C++ host APIs and C++ Python binding files.
- Build with debug symbols:
CONFIG=Debug ./build_metal.sh
- To debug Metalium C++ host APIs, run
gdb --args <generated binary>
- To debug the C++ binding file itself:
- Ensure the python file you wish to debug is standalone and has a main function.
- Run
gdb --args python <python file>
- Breakpoints can be added for future loaded libraries. For example, to add a breakpoint to
Device
object construtor:
- Build with debug symbols:
(gdb) b device.cpp:Device::Device
No source file named device.cpp.
Make breakpoint pending on future shared library load? (y or [n]) y
Breakpoint 1 (device.cpp:Device::Device) pending.
(gdb) r
...
Breakpoint 1, tt::tt_metal::Device::Device (this=0x3c, device_id=21845, num_hw_cqs=24 '\030', l1_small_size=140737349447680, l1_bank_remap=<>, minimal=119) at tt-metal/tt_metal/impl/device/device.cpp
71 Device::Device(
- To log the compiler defines passed in with
-D
during the kernel build phase:- Run with Watcher enabled,
export TT_METAL_WATCHER=1
- Files with the kernel configurations are generated as
<tt-metal dir>/built/<device id>/kernels/kernel_args.csv
- Run with Watcher enabled,
- To examine the compile time arguments of a kernel:
- Within your kernel, assign the arguments to constexpr like this:
constexpr uint32_t in1_mcast_sender_noc_y = get_compile_time_arg_val(0);
- Run
dump-constexprs.py
script on the generated ELF file. E.g.python tt_metal/tools/dump-consts.py built/0/kernels/command_queue_producer/1129845549852061924/brisc/brisc.elf --function kernel_main
. Note: debug information (DWARF) must be present in ELF files (compiler option-g
). To enable, add TT_METAL_RISCV_DEBUG_INFO=1 environment variable.
- Within your kernel, assign the arguments to constexpr like this:
- For developing device-side code, it is recommended to always run with Watcher enabled. Set the environment variable to 10 to have the watcher server update every 10 seconds:
export TT_METAL_WATCHER=10
- Running with watcher enabled will include code that validates NoC transactions, as well as on-device assertions.
- Watcher will flag illegal NoC transactions that may seem to run ok without watcher, this is expected (e.g., 0 length transactions are not considered safe but appear safe in practice).
- If watcher detects an error, an appropriate message will be displayed, the problematic core will be stalled, and the program will exit. For more information on watcher debug features, see the Watcher documentation.
- Once the design has been "proven", disable watcher for performance testing.
- To print within a kernel, use the Debug Print API:
- Define the environment variable to specify which cores to print from,
export TT_METAL_DPRINT_CORES=(0,0)-(4,4)
to print from a 5x5 grid of cores. - In the kernel,
#include "debug/dprint.h"
, and to print a variablex
,DPRINT << x << ENDL();
- For more information on kernel printing, see the Kernel Debug Print documentation.
- Define the environment variable to specify which cores to print from,
- Try to always develop with Watcher enabled. It can catch certain errors and asserts and report them, as well as providing useful debug information in the case of a hang.
- If watcher is enabled when your program hangs, make sure that
Watcher checking device <n>
is being printed, then kill your program.- Make sure that the watcher didn't explicitly catch any errors and print them on
stdout
. For example, the following is printed if the watcher catches a NoC transaction with bad alignment:
- Make sure that the watcher didn't explicitly catch any errors and print them on
TT_METAL_WATCHER=10 ./your_program
...
Always | WARNING | Watcher detected NOC error and stopped device: bad alignment in NOC transaction.
Always | WARNING | Device 0 worker core(x= 0,y= 0) phys(x= 1,y= 1): brisc using noc0 tried to access DRAM core w/ physical coords (x=0,y=11) DRAM[addr=0x00003820,len=102400], misaligned with local L1[addr=0x00064010]
Always | INFO | Last waypoint: NARW, W, W, W, W
Always | INFO | While running kernels:
Always | INFO | brisc : tests/tt_metal/tt_metal/test_kernels/dataflow/dram_copy.cpp
Always | INFO | ncrisc: blank
Always | INFO | triscs: blank
Test | INFO | Reported error: Device 0 worker core(x= 0,y= 0) phys(x= 1,y= 1): brisc using noc0 tried to access DRAM core w/ physical coords (x=0,y=11) DRAM[addr=0x00003820,len=102400], misaligned with local L1[addr=0x00064010]
Always | FATAL | Watcher detected NOC error and stopped device: bad alignment in NOC transaction.
- If no such error is reported, but the program is hanging, check the watcher log generated in
generated/watcher/watcher.log
. There is a legend at the top of the log showing how to interpret it, and a sample portion of a log is shown below:
Legend:
Comma separated list specifices waypoint for BRISC,NCRISC,TRISC0,TRISC1,TRISC2
I=initialization sequence
W=wait (top of spin loop)
R=run (entering kernel)
D=done (finished spin loop)
X=host written value prior to fw launch
A single character status is in the FW, other characters clarify where, eg:
NRW is "noc read wait"
NWD is "noc write done"
noc<n>:<risc>{a, l}=an L1 address used by NOC<n> by <riscv> (eg, local src address)
noc<n>:<riscv>{(x,y), a, l}=NOC<n> unicast address used by <riscv>
noc<n>:<riscv>{(x1,y1)-(x2,y2), a, l}=NOC<n> multicast address used by <riscv>
rmsg:<c>=brisc host run message, D/H device/host dispatch; brisc NOC ID; I/G/D init/go/done; | separator; B/b enable/disable brisc; N/n enable/disable ncrisc; T/t enable/disable TRISC
smsg:<c>=slave run message, I/G/D for NCRISC, TRISC0, TRISC1, TRISC2
k_ids:<brisc id>|<ncrisc id>|<trisc id> (ID map to file at end of section)
...
Dump #7 at 8.992s
Device 0 worker core(x= 0,y= 0) phys(x= 1,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 1,y= 0) phys(x= 2,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 2,y= 0) phys(x= 3,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 3,y= 0) phys(x= 4,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 4,y= 0) phys(x= 6,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 5,y= 0) phys(x= 7,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 6,y= 0) phys(x= 8,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 7,y= 0) phys(x= 9,y= 1): GW, W, W, W, W rmsg:D0D|BNT smsg:DDDD k_ids:14|13|15
Device 0 worker core(x= 0,y= 7) phys(x= 1,y=10): NTW,UAPW, W, W, W rmsg:H1G|bNt smsg:GDDD k_ids:0|2|0
Device 0 worker core(x= 1,y= 7) phys(x= 2,y=10): NTW, HQW, W, W, W rmsg:H1G|bNt smsg:GDDD k_ids:0|1|0
Device 0 worker core(x= 2,y= 7) phys(x= 3,y=10): NTW, HQW, W, W, W rmsg:H1G|bNt smsg:GDDD k_ids:0|3|0
Device 0 worker core(x= 3,y= 7) phys(x= 4,y=10): NTW,UAPW, W, W, W rmsg:H1G|bNt smsg:GDDD k_ids:0|7|0
Device 0 worker core(x= 4,y= 7) phys(x= 6,y=10): NABD, W, W, W, W rmsg:H0G|Bnt smsg:DDDD k_ids:4|0|0
Device 0 worker core(x= 5,y= 7) phys(x= 7,y=10): NABD, W, W, W, W rmsg:H0G|Bnt smsg:DDDD k_ids:6|0|0
Device 0 worker core(x= 6,y= 7) phys(x= 8,y=10): GW, W, W, W, W rmsg:H0D|bnt smsg:DDDD k_ids:0|0|0
Device 0 worker core(x= 7,y= 7) phys(x= 9,y=10): GW, W, W, W, W rmsg:H0D|bnt smsg:DDDD k_ids:0|0|0
k_id[0]: blank
k_id[1]: tt_metal/impl/dispatch/kernels/cq_prefetch.cpp
k_id[2]: tt_metal/impl/dispatch/kernels/cq_dispatch.cpp
k_id[3]: tt_metal/impl/dispatch/kernels/cq_prefetch.cpp
k_id[4]: tt_metal/impl/dispatch/kernels/packet_mux.cpp
k_id[5]: tt_metal/impl/dispatch/kernels/eth_tunneler.cpp
k_id[6]: tt_metal/impl/dispatch/kernels/packet_demux.cpp
k_id[7]: tt_metal/impl/dispatch/kernels/cq_dispatch.cpp
k_id[13]: tests/tt_metal/tt_metal/test_kernels/dataflow/reader_matmul_tile_layout.cpp
k_id[14]: tests/tt_metal/tt_metal/test_kernels/dataflow/writer_matmul_tile_layout.cpp
k_id[15]: tests/tt_metal/tt_metal/test_kernels/compute/matmul_large_block_zm.cpp
- In the log above, relevant debug information is displayed for each code. Of particular note is the
k_ids
field, and the waypoint status.- The
k_ids
field reports the kernel currently running on the core, using the mapping at the end of the dump. Checking which kernels are running at the time of the hang (the latest dump in the log) shows which files to debug further, and should be included in any filed issues. - The waypoint field show the latest waypoint that each kernel has run past. The typical application of these is to put a waypoint before and after any kernel code that could hang, which can be used to pinpoint a hang from the log.
- Further debug features are available, such as a debug ring buffer on each core. For more information, see the Watcher documentation.
- The
- If you're able to deterministically reproduce the hang, the relevant kernel code can be instrumented with more debug features and iterated on to find the source of the hang.
- For multicast operations, you should check that the parameters are correct and you are calling the right variant of the method. Some examples of what to watch out for are the following:
- The number of destinations has to be non-zero.
- If the source node is in the destination set, you need to use the
loopback_src
variant of the method. - The
loopback_src
variant will not do anything if the set of destination nodes consists entirely of the source node.
- For multicast operations, you should check that the parameters are correct and you are calling the right variant of the method. Some examples of what to watch out for are the following:
- If a hang happens only when watcher is disabled, it is likely that the extra code added by watcher is affecting a timing-related issue. In this case you can try disabling certain watcher features to attempt to bring the timing closer.
- The most invasive watcher features is the NoC sanitization, try disabling it with:
TT_METAL_WATCHER=10 TT_METAL_WATCHER_DISABLE_NOC_SANITIZE=1 ./your_program
- If you still cannot reproduce the hang, try disabling the waypoint and assert features. This will reduce visiblity into the hang, but is better than nothing:
TT_METAL_WATCHER=10 TT_METAL_WATCHER_DISABLE_NOC_SANITIZE=1 TT_METAL_WATCHER_DISABLE_WAYPOINT=1 ./your_program
TT_METAL_WATCHER=10 TT_METAL_WATCHER_DISABLE_NOC_SANITIZE=1 TT_METAL_WATCHER_DISABLE_WAYPOINT=1 TT_METAL_WATCHER_DISABLE_ASSERT=1 ./your_program
- If the hang is not reproducible with watcher enabled, or for whatever reason watcher cannot be enabled for the run that hangs, then you can use the
watcher_dump
tool to poll watcher data after the fact. Even if the initial program is not run with watcher features, this can at least show the kernels that were running on each core at the time of the hang.
# Note that if the PCIe or ethernet connection to a chip goes down then this tool won't be able to access on-device data.
./build/tools/watcher_dump --devices=<ids of devices to dump>
cat generated/watcher/watcher.log # See k_ids field for each core in the last dump in the log
- In the future, this tool will be expanded to show more debug information available from the host side.
-
Every source file must have the appropriate SPDX header at the top following the Linux conventions for C++ source files, RST files, ASM files, and scripts. For Python files, we are to use this convention:
# SPDX-FileCopyrightText: © 2023 Tenstorrent Inc. # SPDX-License-Identifier: Apache-2.0
For C++ header files, we will treat them as C++ source files and use this convention:
// SPDX-FileCopyrightText: © 2023 Tenstorrent Inc. // // SPDX-License-Identifier: Apache-2.0
- Revert commits on main which fail post-commit tests immediately.
- There shall be a periodic discussion among the technical leads of this
project concerning:
- Certain codeowners and project-specific members review current tests in post-commit.
- Certain codeowners and project-specific members decide whether to remove/add any current tests in post-commit as project priorities change on an ongoing basis.
- Certain codeowners and project-specific members decide if we need to change owners or add more as project priorities change on an ongoing basis.
- Communication channels for these decisions and meetings shall be kept internal to Tenstorrent with the intent of having such discussions in the open later.
- Non-post-commit pipelines will not necessarily mean we have to revert the breaking commit, however any broken pipelines will be considered a priority bug fix.
- The responsibility of identifying, announcing status-tracking, and escalating
broken non-post-commit pipelines will be the responsibility of codeowners
whose tests are in the said non-post-commit pipeline.
- In the case of the model performance test pipeline, there are codeowners for such tests. However, it is the collective responsibility of all developers to ensure that we do not regress this pipeline.
-
There are some automated checks upon opening a PR. These checks are part, but not all, of the post-commit test suite. They must pass, but are not enough to ensure your PR will not be reverted.
-
To run any CI pipeline on GitHub Actions, please navigate to the actions page.
Next, you can navigate to any pipeline on the left side of the view. For example, you can run the entire post-commit CI suite by clicking on on the link to all post-commit workflows, clicking "Run workflow", selecting your branch, and pressing "Run workflow".
You can see the status of your CI run by clicking on the specific run you dispatched.
We have a sizeable number of workflows, so don't forget to press "Show more workflows...".
-
Unfortunately, we currently do not do automatic checks of all required workflows upon opening a PR. There are various reasons for this, such as limited machine resources. This means that developer and reviewer discretion is still the most important factor in ensuring PRs are merged successfully and without CI failure.
- Any API changes must be accompanied with appropriate documentation changes.
-
Any commit message must be accompanied with an appropriate GitHub issue number with a colon and following message. The message must start with an imperative verb and descripton of what was done. Preferably a reason is included. Ex.
#41: Fix data format error in Gelu op.
-
The following is not allowed in commit messages:
- Commit messages which state that a code review or comments are being addressed. You must explicitly state what you are doing in each commit even if it's just cosmetic.
-
If you are working on a branch and would like to skip the Git commit hooks, you may delete the
git_hooks
Makefile directive in/module.mk
before your first build. However, you are responsible for making sure your final submission follows the contribution guidelines. Failure to do so constitutes a violation of these contribution guidelines. -
Merge commits are not allowed in our main branch. We enforce a linear history.
-
You can use either of the following methods to merge your branch on the GitHub UI:
- Squash and merge
- Rebase and merge
If you use squashing, when GitHub asks you to enter a new commit message, ensure that your commit message follows our required format as outlined above in this section. Failure to do so is a violation of our standards.
- A PR must be opened for any code change with the following criteria:
- Be approved, by a maintaining team member and any codeowners whose modules are relevant for the PR.
- Pass any required post-commit pipelines, updated to the latest main. These
pipelines will generally, but not always, be defined in
.github/workflows/all-post-commit-workflows.yaml
. - Pass any acceptance criteria mandated in the original issue.
- Pass any testing criteria mandated by codeowners whose modules are relevant for the PR.
- Avoid opening/re-opening/push new commits to PRs before you're ready for review and start running pipelines. This is because we don't want to clog our pipelines with unnecessary runs that developers may know will fail anyways.
- New or changing features require the following accompanying documentation:
- An architectural change plan approved by maintaining team members.
- A design plan with associated GitHub project/large containing issue. with sub-issues for proper documentation of project slices.
- An appropriate test plan with issues.
- Any release must be externally-available artifacts generated by a workflow on a protected branch.
- Demo models and tags conform to the rules set forth in the models README.
- Use Loguru for Python logging.
- Use Tenstorrent logger for C++ logging.
If a Tenstorrent chip seems to hang and/or is producing unexpected behaviour, you may try a software reset of the board.
For single-card: tt-smi -r 0
For T3000 (QuietBox, LoudBox etc.): tt-smi -r 0,1,2,3
If the software reset does not work, unfortunately you will have to power cycle the board. This usually means rebooting the host of a board.