Skip to content
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.

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

(WiP) More restrictive column control overlay generation #1739

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

erwei-xilinx
Copy link
Collaborator

@erwei-xilinx erwei-xilinx commented Aug 29, 2024

To ensure control overlay consistency when reconfiguring across multiple designs.

  • Control packet flows were routed before other flows, to ensure consistent switchbox master/slave port/arbiter/msel allocation across designs.
  • Control packet flows were sorted to ensure routing determinism.
  • New pass AIECtrlPacketInferTilesPass which infers tile ops from control packet op's addresses; tile ops are handles used to generate column control overlays.
  • New pass AIECtrlPacketToDmaPass which converts control packet ops into memcpy ops (and sync ops) representing data movements, sending control packets from DDR via shim to tiles.
  • Reformatted test ctrl_packet_reconfig to generate control packets using compiler passes, rather than copy-pasting from a text file.

@fifield
Copy link
Collaborator

fifield commented Sep 5, 2024

The new passes really should be separate from the PR updating the overlay pass.

@erwei-xilinx
Copy link
Collaborator Author

The new passes really should be separate from the PR updating the overlay pass.

Ok let me break this PR up into multiple PRs. Will change the PR to draft.

@erwei-xilinx erwei-xilinx marked this pull request as draft September 5, 2024 16:04
@erwei-xilinx erwei-xilinx changed the title More restrictive column control overlay generation (WiP) More restrictive column control overlay generation Sep 5, 2024
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.

2 participants