Skip to content

Forward GEP stride config packets#307

Open
guosran wants to merge 1 commit into
tancheng:masterfrom
guosran:fix/controller-gep-stride-routing
Open

Forward GEP stride config packets#307
guosran wants to merge 1 commit into
tancheng:masterfrom
guosran:fix/controller-gep-stride-routing

Conversation

@guosran

@guosran guosran commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

Addresses #292 with a minimized controller config-routing fix. This PR only updates ControllerRTL.py.

Changes

  • Forward CMD_CONFIG_GEP_STRIDE packets to the tile control ring.
  • Keep GEP stride handling consistent with CtrlMemDynamicRTL, which already accepts this command.

Validation

Part of the master-based small conv passing set.

CGRA_CONV_NI=2 CGRA_CONV_NJ=2 CGRA_CONV_YAML=validation/test/conv/conv_small.yaml \
python3 -m pytest cgra/test/CgraRTL_conv4x4_test_from_yaml.py::test_homogeneous_4x4_conv_combinational_mem_access -q --tb=short

@guosran guosran marked this pull request as draft June 25, 2026 20:04
@guosran guosran marked this pull request as ready for review June 26, 2026 00:40
@guosran

guosran commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator Author

GEP stride config packets need to be routed to the tile/control path just like other loop/config packets.

controller sends CMD_CONFIG_GEP_STRIDE
|
v
tile ctrl memory / element config path

Without this, generated conv configs can contain a stride update that never reaches the tile logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant