Skip to content

Update metrics after GRT blocked tracks fix #4417

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
eder-matheus merged 8 commits into from
Aug 13, 2026
Merged

Update metrics after GRT blocked tracks fix #4417

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1908306
update OR
jfgava Aug 11, 2026
304e576
Limit repair antennas iter DRT for microwatt
jfgava Aug 11, 2026
bdf15a6
metrics update after GRT FR blocked tracks fix
jfgava Aug 12, 2026
e1c0759
update OR
jfgava Aug 12, 2026
53cca2c
Merge branch 'master' of github.com:The-OpenROAD-Project-private/Open…
jfgava Aug 12, 2026
8b84a9f
update OR
jfgava Aug 12, 2026
3787772
update microwatt core utilization
jfgava Aug 12, 2026
025f6dc
update metrics public
jfgava Aug 13, 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
6 changes: 3 additions & 3 deletions flow/designs/asap7/ibex/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,11 @@
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -51.2,
"value": -51.1,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -209.0,
"value": -208.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand Down Expand Up @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -200.0,
"value": -436.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
6 changes: 3 additions & 3 deletions flow/designs/asap7/riscv32i/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -1480.0,
"value": -1890.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand Down Expand Up @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -81.2,
"value": -80.6,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -31800.0,
"value": -12100.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
6 changes: 3 additions & 3 deletions flow/designs/gf180/aes-hybrid/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -123.0,
"value": -129.0,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -169.0,
"value": -144.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand Down Expand Up @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -168.0,
"value": -144.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
4 changes: 2 additions & 2 deletions flow/designs/gf180/riscv32i/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -3.2,
"value": -5.5,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand Down Expand Up @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -2.85,
"value": -5.5,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
8 changes: 4 additions & 4 deletions flow/designs/ihp-sg13g2/ibex/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -1.6,
"value": -5.01,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 1012494,
"value": 1000843,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -5.34,
"value": -3.26,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand All @@ -106,7 +106,7 @@
"compare": ">="
},
"finish__design__instance__area": {
"value": 320304,
"value": 320206,
"compare": "<="
}
}
18 changes: 9 additions & 9 deletions flow/designs/nangate45/swerv_wrapper/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -0.431,
"value": -0.429,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -415.0,
"value": -459.0,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -58,11 +58,11 @@
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -0.466,
"value": -0.431,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -438.0,
"value": -504.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 4160494,
"value": 4834629,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -86,15 +86,15 @@
"compare": "<="
},
"detailedroute__antenna_diodes_count": {
"value": 106,
"value": 105,
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -0.425,
"value": -0.408,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -384.0,
"value": -442.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand All @@ -106,7 +106,7 @@
"compare": ">="
},
"finish__design__instance__area": {
"value": 734366,
"value": 733977,
"compare": "<="
}
}
18 changes: 9 additions & 9 deletions flow/designs/sky130hd/chameleon/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -2.67,
"value": -3.26,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -114.0,
"value": -136.0,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -54,15 +54,15 @@
"compare": ">="
},
"globalroute__antenna_diodes_count": {
"value": 224,
"value": 179,
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -2.32,
"value": -2.96,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -42.9,
"value": -48.8,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 725303,
"value": 853725,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -86,15 +86,15 @@
"compare": "<="
},
"detailedroute__antenna_diodes_count": {
"value": 158,
"value": 125,
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -2.06,
"value": -2.66,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -39.9,
"value": -41.9,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
18 changes: 9 additions & 9 deletions flow/designs/sky130hd/gcd/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@
"compare": "=="
},
"placeopt__design__instance__area": {
"value": 4681,
"value": 4663,
"compare": "<="
},
"placeopt__design__instance__count__stdcell": {
"value": 552,
"value": 550,
"compare": "<="
},
"detailedplace__design__violations": {
Expand All @@ -42,7 +42,7 @@
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -60.9,
"value": -65.5,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -58,11 +58,11 @@
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -1.66,
"value": -1.75,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -71.6,
"value": -75.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 13026,
"value": 14559,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -1.53,
"value": -1.68,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -67.0,
"value": -71.2,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand All @@ -106,7 +106,7 @@
"compare": ">="
},
"finish__design__instance__area": {
"value": 5996,
"value": 5812,
"compare": "<="
}
}
5 changes: 3 additions & 2 deletions flow/designs/sky130hd/microwatt/config.mk
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ export VERILOG_FILES = $(sort $(wildcard $(DESIGN_HOME)/src/$(DESIGN_NICKNAME)/*

export SDC_FILE = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/constraint.sdc

export DIE_AREA = 0 0 3020 3610
export CORE_AREA = 10 10 3010 3600
export CORE_UTILIZATION = 55

export ADDITIONAL_GDS = $(wildcard $(DESIGN_DIR)/gds/*.gds.gz)

Expand All @@ -34,6 +33,8 @@ export SETUP_SLACK_MARGIN = 0.2
# GRT non-default config
export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl

export MAX_REPAIR_ANTENNAS_ITER_DRT = 2

@gemini-code-assist gemini-code-assist Bot Aug 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

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.

@jfgava jfgava Aug 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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


ifeq ($(SYNTH_MOCK_LARGE_MEMORIES),1)
# ca. 3 minutes to run make synth
#
Expand Down
Loading