Skip to content

flow: update MPL and designs configs - #4493

Open
joaomai wants to merge 3 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:mpl-halo-channel-update
Open

flow: update MPL and designs configs#4493
joaomai wants to merge 3 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:mpl-halo-channel-update

Conversation

@joaomai

@joaomai joaomai commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Updates ORFS to accommodate the changes introduced in #11292 from OR. RTLMP_MIN_CHANNEL_SIZE takes the place of MACRO_PLACE_HALO, and the usage of pin-awareness in MPL is now a flag: RTLMP_PIN_AWARE_CHANNELS is set to 1 by default if RTLMP_MIN_CHANNEL_SIZE is also set, otherwise it is set to 0.

No metrics change.

Signed-off-by: João Mai <jmai@precisioninno.com>
Signed-off-by: João Mai <jmai@precisioninno.com>
Signed-off-by: João Mai <jmai@precisioninno.com>
@joaomai joaomai self-assigned this Sep 1, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request replaces the deprecated MACRO_PLACE_HALO and MACRO_BLOCKAGE_HALO configuration variables with RTLMP_MIN_CHANNEL_SIZE and RTLMP_PIN_AWARE_CHANNELS. It updates the documentation, variable definitions, design/platform configuration files, and the macro placement utility script accordingly. Additionally, the OpenROAD submodule has been updated to a newer commit. There are no review comments, so I have no feedback to provide.

@openroad-ci

Copy link
Copy Markdown
Member

🔍 QoR check

Metrics 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 ca12891 · Jenkins build #1 · Baseline: build

63 design(s) checked — 4 with regression(s), 6 without a comparable baseline.
Design Variant Verdict local vs inline passed / failed / missing
asap7/aes base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes-block base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes-mbff base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/coralnpu ➖ inconclusive ➖ N/A no metrics produced — flow may have failed before writing metadata.json
asap7/cva6 ➖ inconclusive ➖ N/A no metrics produced — flow may have failed before writing metadata.json
asap7/ethmac base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/ethmac_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/gcd-ccs base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/jpeg_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/mock-alu base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/mock-cpu base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/riscv32i ➖ inconclusive ➖ N/A no metrics produced — flow may have failed before writing metadata.json
asap7/riscv32i-mock-sram ➖ inconclusive ➖ N/A no metrics produced — flow may have failed before writing metadata.json
asap7/swerv_wrapper base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/uart base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/aes base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/aes-hybrid base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/uart-blocks base ❌ fail ⚠️ MISMATCH 24 / 1 / 0
gt2n/aes base ✅ pass ✅ MATCH 25 / 0 / 0
gt2n/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
gt2n/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/aes base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/i2c-gpio-expander base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/spi base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/aes base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/ariane133 base ➖ inconclusive ➖ N/A 0 / 0 / 25
nangate45/ariane136 base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/black_parrot base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_be_top base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_fe_top base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_multi_top base ❌ fail ✅ MATCH 24 / 1 / 0
nangate45/cva6 base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/dynamic_node base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/mempool_group base ❌ fail ⚠️ MISMATCH 9 / 3 / 0
nangate45/swerv base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/swerv_wrapper base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/tinyRocket base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/aes base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/chameleon base ❌ fail ✅ MATCH 23 / 2 / 0
sky130hd/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/microwatt ➖ inconclusive ➖ N/A no metrics produced — flow may have failed before writing metadata.json
sky130hd/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/aes base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
❌ gf180/uart-blocks base — 1 failing metric(s)
Metric target base delta limit band
placeopt__design__instance__count__stdcell 653 559 16.815742397137747% 642.85 Padding 15%
❌ nangate45/bp_multi_top base — 1 failing metric(s)
Metric target base delta limit band
detailedroute__route__wirelength 3995402 3370611 18.536431525322858% 3876202.65 Padding 15%
❌ nangate45/mempool_group base — 3 failing metric(s)
Metric target base delta limit band
cts__timing__setup__tns -11837.1 -11431.7 3.546279206067339% -11432.3 PeriodPadding 20%
finish__timing__setup__tns -11756.5 -11436.4 2.798957713965933% -11437.0 PeriodPadding 20%
globalroute__timing__setup__tns -11870.8 -11499.3 3.2306314297392014% -11499.9 PeriodPadding 20%
❌ sky130hd/chameleon base — 2 failing metric(s)
Metric target base delta limit band
detailedroute__antenna_diodes_count 131 119 10.084033613445378% 119.119 Metric 0.1%
detailedroute__route__drc_errors 1 0 0 Direct 0%

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.

3 participants