-
Notifications
You must be signed in to change notification settings - Fork 396
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
3D NoC #2649
3D NoC #2649
Conversation
Some CI test failures due to vtr::isclose getting one float and one double; you'll have to explicitly cast or change a constant or some such (@soheilshahrouz tells me he has float constants and double variables so that is probably it). |
Until CI is sorted please run the major nightly tests on wintermute and list the results here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. A few changes. Also:
- Check UP/DOWN are consistently changed to NORTH/SOUTH
- We should have at least one test for each routing algorithm (can be separate PR).
re-launched CI. I also suggest one QoR test (Titan or VTR, could be just a circuit or two if you wish) to confirm no unexpected slowdown, and a test on the NoC designs to confirm the same. |
@soheilshahrouz tells me there is a strange behaviour: the NoC centroid move is working, but when enabled the initial temperature is very low (3 orders of magnitude lower). This did not occur in the earlier, FPL 2024 code. Only affects NoC designs. The QoR is very bad due to this lower temperature, so we aren't just finding a very good solution quickly. Action after this is figured out: turn this move type on by default, or have a CI test for it. |
Almost ready for merge ... NoC centroid move problem is resolved (was a memory bug). |
QoR results on a subset of synthetic benchmarks on a 2D noc to measure if the introduction of 3D NoC support has slowed down NoC optimization in 2D architectures
|
3D seems to have slightly improved things :). Merging! |
Description
Generalizing the NoC data structures, architecture description and routing algorithms to 3D (multi-layer) FPGAs.
Motivation and Context
VTR is mostly 3D now, but the NoCs are still only 2D.
How Has This Been Tested?
Created a new arch file and ran synthetic benchmarks on this new 3D NoC arch within the 3D Stratix IV arch. The 10x10 2D mesh is replaced by two 7x7 meshes stacked on top of each other. All the benchmarks passed, and the QoR metrics are slightly better as the 3D NoC topology shortens some traffic flows.
CI has been updated to run some tests on this 3D architecture.
Types of changes
Checklist: