Update metrics after GRT blocked tracks fix - #4417
Merged
eder-matheus merged 8 commits intoAug 13, 2026
Merged
Conversation
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the OpenROAD submodule commit and adjusts the baseline rules across multiple designs to reflect changes in timing, area, wirelength, and antenna diode counts. However, limiting MAX_REPAIR_ANTENNAS_ITER_DRT to 2 in the microwatt configuration introduces a severe regression, increasing antenna violating nets from 8 to 96. It is recommended to address the root cause of the timeout or looping issue in DRT rather than capping the repair iterations.
| # GRT non-default config | ||
| export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl | ||
|
|
||
| export MAX_REPAIR_ANTENNAS_ITER_DRT = 2 |
Contributor
There was a problem hiding this comment.
Limiting MAX_REPAIR_ANTENNAS_ITER_DRT to 2 introduces a major regression, increasing antenna violating nets from 8 to 96. Capping the repair iterations to bypass timeouts compromises physical design integrity. The timeout/looping issue in DRT should be resolved at the root level rather than reducing the repair iterations.
Contributor
Author
There was a problem hiding this comment.
Agree. After Yosys 0.68 update, the number of diodes in DRT increased from 1700 to 2700. The problems with antennas got worse. With the new computation of blocked tracks, placement (routability) is affected, which affects all subsequent steps. The DRT step alone took more than 2h with 5 iterations of repair antennas. I can investigate this later
…ROAD-flow-scripts into update-metrics-grt-fix-blocked-tracks
Member
🔍 QoR checkMetrics reflect the PR merge build — i.e. what will land on the target branch. Advisory — results are log-only and do not affect build status. The authoritative QoR gate remains the local rules-file check. Commit 61 design(s) checked — 12 with regression(s), 0 without a comparable baseline.
❌ asap7/ibex base — 1 failing metric(s)
❌ asap7/riscv32i-mock-sram base — 1 failing metric(s)
❌ gf180/aes-hybrid base — 3 failing metric(s)
❌ ihp-sg13g2/ibex base — 1 failing metric(s)
❌ ihp-sg13g2/jpeg base — 1 failing metric(s)
❌ nangate45/mempool_group base — 1 failing metric(s)
❌ nangate45/swerv base — 2 failing metric(s)
❌ nangate45/swerv_wrapper base — 3 failing metric(s)
❌ sky130hd/chameleon base — 6 failing metric(s)
❌ sky130hd/gcd base — 5 failing metric(s)
❌ sky130hd/microwatt base — 2 failing metric(s)
❌ sky130hd/riscv32i base — 1 failing metric(s)
|
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
Contributor
Author
|
designs/sky130hd/microwatt/rules-base.json updates:
designs/ihp-sg13g2/ibex/rules-base.json updates:
designs/asap7/ibex/rules-base.json updates:
designs/sky130hd/gcd/rules-base.json updates:
designs/nangate45/swerv_wrapper/rules-base.json updates:
|
eder-matheus
approved these changes
Aug 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Microwatt is very sensitive to placement changes. I'm having timeout issues during DRT due to multiple loops after antenna repairs. To fix this, I limited the number of repair_antennas iterations during DRT (5 -> 2). I can take a deeper look later
designs/asap7/riscv32i/rules-base.json updates:
designs/gf180/aes-hybrid/rules-base.json updates:
designs/gf180/riscv32i/rules-base.json updates:
designs/sky130hd/chameleon/rules-base.json updates:
designs/sky130hd/gcd/rules-base.json updates:
designs/sky130hs/jpeg/rules-base.json updates:
designs/nangate45/swerv_wrapper/rules-base.json updates:
designs/sky130hd/microwatt/rules-base.json updates:
Related to #11093
CI Could not Update Rules
[ERROR] asap7/tinyRocket skipped. File 'rules-base.json' not found. Please create the file with 'make update_rules'.
Messages from CI
[INFO] asap7/minimal not included in CI.
[INFO] gf12 not included in the update.
[INFO] gf55 not included in the update.
[INFO] nangate45/bp_quad not included in CI.