Skip to content

[runtime] fix core id interpretations #80

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bowwwang merged 3 commits into from
Apr 23, 2026
Merged

[runtime] fix core id interpretations #80

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6ef9c87
[runtime] fix core id interpretations
Apr 14, 2026
0e8db41
[ci] fix trailing space
Apr 22, 2026
b4a9c25
Merge branch 'main' into runtime-fix
bowwwang Apr 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions sw/snRuntime/src/team.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ uint32_t snrt_cluster_compute_core_num() {
}

uint32_t snrt_cluster_dm_core_idx() {
// TODO: Actually derive this from the device tree!
return snrt_cluster_core_num() - 1;
// Core 0 ties to DMA control in hardware
Comment thread
DiyouS marked this conversation as resolved.
Outdated
return 0;
}

uint32_t snrt_cluster_dm_core_num() {
Expand All @@ -68,13 +68,13 @@ uint32_t snrt_cluster_dm_core_num() {
}

int snrt_is_compute_core() {
// TODO: Actually derive this from the device tree!
return snrt_cluster_core_idx() < snrt_cluster_core_num() - 1;
// All Spatz CCs are compute core
return 1;
}

int snrt_is_dm_core() {
// TODO: Actually derive this from the device tree!
return !snrt_is_compute_core();
// Core 0 ties to DMA control in hardware
return !snrt_cluster_core_idx();
}

uint32_t _snrt_barrier_reg_ptr() {
Expand Down
Loading