Secure fix missing para dbmodnet by openroad-ci · Pull Request #4050 · The-OpenROAD-Project/OpenROAD-flow-scripts

openroad-ci · 2026-03-25T04:16:59Z

This PR needs metric change due to now vieweing parasitics correctly on modnets. Corresponds to this PR in OpenROAD The-OpenROAD-Project/OpenROAD#9745

CI Could not Update Rules

[ERROR] sky130hd/jpeg: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hd/microwatt: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hd/riscv32i: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/aes: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/gcd: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/ibex: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/jpeg: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/riscv32i: No metrics found, please check the latest CI build for issues.

CI Could not Update Rules

[ERROR] sky130hd/jpeg: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hd/microwatt: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hd/riscv32i: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/aes: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/gcd: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/ibex: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/jpeg: No metrics found, please check the latest CI build for issues.
[ERROR] sky130hs/riscv32i: No metrics found, please check the latest CI build for issues.

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/asap7/swerv_wrapper/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-80.0	-319.0	Failing
finish__timing__setup__tns	-320.0	-47000.0	Failing

[WARNING] Multiple clocks not supported. Will use first clock: clk_i: 3.0000.
designs/nangate45/swerv/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__tns	-460.0	-619.0	Failing

designs/sky130hd/gcd/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.74	-1.86	Failing
finish__timing__setup__tns	-81.2	-87.8	Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.
designs/sky130hd/microwatt/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.34	-2.71	Failing
finish__timing__setup__tns	-158.0	-337.0	Failing

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.

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/asap7/swerv_wrapper/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-80.0	-319.0	Failing
finish__timing__setup__tns	-320.0	-47000.0	Failing

[WARNING] Multiple clocks not supported. Will use first clock: clk_i: 3.0000.
designs/nangate45/swerv/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__tns	-459.0	-516.0	Failing

designs/nangate45/swerv_wrapper/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__tns	-113.0	-127.0	Failing

designs/sky130hd/gcd/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.74	-1.86	Failing
finish__timing__setup__tns	-81.2	-87.8	Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.
designs/sky130hd/microwatt/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.34	-2.71	Failing
finish__timing__setup__tns	-157.0	-336.0	Failing

designs/sky130hs/ibex/rules-base.json updates:

Metric	Old	New	Type
globalroute__timing__setup__tns	-7.77	-33.7	Failing

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.

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/asap7/swerv_wrapper/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-80.0	-319.0	Failing
finish__timing__setup__tns	-320.0	-47000.0	Failing

[WARNING] Multiple clocks not supported. Will use first clock: clk_i: 3.0000.
designs/nangate45/swerv/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__tns	-459.0	-516.0	Failing

designs/nangate45/swerv_wrapper/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__tns	-113.0	-127.0	Failing

designs/sky130hd/gcd/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.74	-1.86	Failing
finish__timing__setup__tns	-81.2	-87.8	Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.
designs/sky130hd/microwatt/rules-base.json updates:

Metric	Old	New	Type
finish__timing__setup__ws	-1.34	-2.71	Failing
finish__timing__setup__tns	-157.0	-336.0	Failing

designs/sky130hs/ibex/rules-base.json updates:

Metric	Old	New	Type
globalroute__timing__setup__tns	-7.77	-33.7	Failing

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.

Signed-off-by: Martin Povišer <povik@cutebit.org>

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

…para_dbmodnet

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

dsengupta0628 · 2026-05-18T23:55:52Z

designs/asap7/swerv_wrapper/ actually shows [INFO] finish__timing__setup__tns pass test: -46693.1 >= -47000 so it is not as catastrophic

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

dsengupta0628 · 2026-05-19T19:19:43Z

    },
    "finish__timing__setup__tns": {
-        "value": -320.0,
+        "value": -47000.0,


I am not sure why this number oscillates so much- yesterday baseline was -46993 and today it was somehow regoldened to -320 and my code makes it back to -47000. This is also the test that gets affected by my change- with correct para now, it fallsback to the PRIMA dcalc path and - without my isExternal fix, it would have crashed in PRIMA. So this testcase is really stress testing my change

It seems like the -320 value comes from PR one month ago. In any case I would expect your change to degrade (i.e. fix the reporting on) designs with hierarchy enabled, which swerv_wrapper seems to be one of.

weird. yesterday I saw it -46993 in my local run.

"finish__timing__setup__ws": -239.109, "finish__timing__setup__tns": -46693.1,

I wonder if this is exposing an issue in PRIMA delay calc now.. witherways, orthogonal..

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

maliberty · 2026-05-20T14:13:35Z

asap7/swerv_wrapper has a very large degradation. Do you understand why?

dsengupta0628 · 2026-05-20T14:27:56Z

asap7/swerv_wrapper has a very large degradation. Do you understand why?

Yes somewhat. Weird stuff going on in that design- but mainly wire caps are much larger so data path has higher delay. Slew at CLK pin Will discuss in mtng.

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

povik and others added 10 commits March 11, 2026 17:46

Add final check all drivers have parasitic annotation

f953347

Signed-off-by: Martin Povišer <povik@cutebit.org>

fix missing para on dbterm

590b815

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

Merge remote-tracking branch 'origin/master' into secure-fix_missing_…

857bc84

…para_dbmodnet

Merge branch 'master' into secure-fix_missing_para_dbmodnet

cfc24d1

update openroad- again!

503c0d7

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

revert the proc to check missing para

176bd4b

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

Merge branch 'master' into secure-fix_missing_para_dbmodnet

04f2834

update openroad

fe4b43a

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

revert the final_report.tcl to original

2f5d591

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

revert the final_report.tcl to original

b17fc9a

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

openroad-ci assigned dsengupta0628 Mar 25, 2026

dsengupta0628 mentioned this pull request Mar 25, 2026

Missing para dbmodnet The-OpenROAD-Project/OpenROAD#9745

Merged

dsengupta0628 requested review from jhkim-pii, maliberty and povik March 25, 2026 04:18

dsengupta0628 added the UpdateRules Starts GHA to update rules label Mar 25, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label Mar 25, 2026

maliberty added the UpdateRules Starts GHA to update rules label Mar 25, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label Mar 25, 2026

dsengupta0628 added 4 commits March 30, 2026 14:40

merged with latest

ba6eaa3

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

latest OR

7c57dc3

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

Merge branch 'master' into secure-fix_missing_para_dbmodnet

9dd3494

update OpenROAD for ORFS regoldening

fac3fa8

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

dsengupta0628 added the UpdateRules Starts GHA to update rules label May 18, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label May 18, 2026

merged with latest orfs

ba99381

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

openroad-ci force-pushed the secure-fix_missing_para_dbmodnet branch from d6b6f0f to ba99381 Compare May 19, 2026 17:14

dsengupta0628 added the UpdateRules Starts GHA to update rules label May 19, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label May 19, 2026

dsengupta0628 reviewed May 19, 2026

View reviewed changes

dsengupta0628 added 2 commits May 20, 2026 14:06

Merge branch 'master' into secure-fix_missing_para_dbmodnet

c9bbe29

latest ORFS merged

218ad1d

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

openroad-ci force-pushed the secure-fix_missing_para_dbmodnet branch from d5b01c1 to 218ad1d Compare May 20, 2026 14:23

dsengupta0628 added the UpdateRules Starts GHA to update rules label May 20, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label May 20, 2026

Merge branch 'master' into secure-fix_missing_para_dbmodnet

0230dc9

dsengupta0628 added the UpdateRules Starts GHA to update rules label May 20, 2026

maliberty approved these changes May 20, 2026

View reviewed changes

openroad-ci removed the UpdateRules Starts GHA to update rules label May 20, 2026

maliberty added 2 commits May 21, 2026 04:06

Merge branch 'master' into secure-fix_missing_para_dbmodnet

98871a1

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>

update OR

e262ba6

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>

maliberty added the UpdateRules Starts GHA to update rules label May 21, 2026

flow: update rules

425011e

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

openroad-ci removed the UpdateRules Starts GHA to update rules label May 21, 2026

maliberty approved these changes May 21, 2026

View reviewed changes

maliberty merged commit bd2a6b6 into The-OpenROAD-Project:master May 21, 2026
8 checks passed

maliberty deleted the secure-fix_missing_para_dbmodnet branch May 21, 2026 13:54

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Secure fix missing para dbmodnet#4050

Secure fix missing para dbmodnet#4050
maliberty merged 21 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-fix_missing_para_dbmodnet

openroad-ci commented Mar 25, 2026 •

edited

Loading

Uh oh!

dsengupta0628 commented May 18, 2026

Uh oh!

dsengupta0628 May 19, 2026

Uh oh!

povik May 19, 2026

Uh oh!

dsengupta0628 May 19, 2026 •

edited

Loading

Uh oh!

maliberty commented May 20, 2026

Uh oh!

dsengupta0628 commented May 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

openroad-ci commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Could not Update Rules

CI Could not Update Rules

Updated Rules

Messages from CI

Updated Rules

Messages from CI

Updated Rules

Messages from CI

Uh oh!

dsengupta0628 commented May 18, 2026

Uh oh!

dsengupta0628 May 19, 2026

Choose a reason for hiding this comment

Uh oh!

povik May 19, 2026

Choose a reason for hiding this comment

Uh oh!

dsengupta0628 May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

maliberty commented May 20, 2026

Uh oh!

dsengupta0628 commented May 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

openroad-ci commented Mar 25, 2026 •

edited

Loading

dsengupta0628 May 19, 2026 •

edited

Loading