Skip to content

Secure fix missing para dbmodnet #4050

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

Open
openroad-ci wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Secure fix missing para dbmodnet #4050

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f953347
Add final check all drivers have parasitic annotation
povik Mar 11, 2026
590b815
fix missing para on dbterm
dsengupta0628 Mar 12, 2026
857bc84
Merge remote-tracking branch 'origin/master' into secure-fix_missing_…
dsengupta0628 Mar 20, 2026
cfc24d1
Merge branch 'master' into secure-fix_missing_para_dbmodnet
dsengupta0628 Mar 23, 2026
503c0d7
update openroad- again!
dsengupta0628 Mar 23, 2026
176bd4b
revert the proc to check missing para
dsengupta0628 Mar 23, 2026
04f2834
Merge branch 'master' into secure-fix_missing_para_dbmodnet
dsengupta0628 Mar 25, 2026
fe4b43a
update openroad
dsengupta0628 Mar 25, 2026
2f5d591
revert the final_report.tcl to original
dsengupta0628 Mar 25, 2026
b17fc9a
revert the final_report.tcl to original
dsengupta0628 Mar 25, 2026
ba6eaa3
merged with latest
dsengupta0628 Mar 30, 2026
7c57dc3
latest OR
dsengupta0628 May 18, 2026
9dd3494
Merge branch 'master' into secure-fix_missing_para_dbmodnet
dsengupta0628 May 18, 2026
fac3fa8
update OpenROAD for ORFS regoldening
dsengupta0628 May 18, 2026
ba99381
merged with latest orfs
dsengupta0628 May 19, 2026
d5b01c1
flow: update rules
github-actions[bot] May 19, 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
4 changes: 2 additions & 2 deletions flow/designs/asap7/swerv_wrapper/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -80.0,
"value": -319.0,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -320.0,
"value": -47000.0,
Copy link
Copy Markdown

@dsengupta0628 dsengupta0628 May 19, 2026

Choose a reason for hiding this comment

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

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

"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
2 changes: 1 addition & 1 deletion flow/designs/nangate45/swerv/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -459.0,
"value": -516.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
2 changes: 1 addition & 1 deletion 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 @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -113.0,
"value": -127.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
4 changes: 2 additions & 2 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 @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -1.74,
"value": -1.86,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -81.2,
"value": -87.8,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
4 changes: 2 additions & 2 deletions flow/designs/sky130hd/microwatt/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -1.34,
"value": -2.71,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -157.0,
"value": -336.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
2 changes: 1 addition & 1 deletion flow/designs/sky130hs/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": -7.77,
"value": -33.7,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand Down
2 changes: 1 addition & 1 deletion tools/OpenROAD
Open in desktop
Submodule OpenROAD updated 31 files
+10 −0 .bazelrc
+23 −0 .github/workflows/github-actions-macos-bazel.yml
+1 −0 BUILD.bazel
+3 −2 Jenkinsfile
+2 −2 MODULE.bazel
+309 −3 MODULE.bazel.lock
+14 −0 docs/agents/ci.md
+17 −0 docs/contrib/DeveloperGuide.md
+6 −1 docs/contrib/LintTargets.md
+4 −1 src/dbSta/include/db_sta/dbNetwork.hh
+18 −5 src/dbSta/src/dbNetwork.cc
+1 −0 src/dbSta/test/BUILD
+1 −0 src/dbSta/test/CMakeLists.txt
+18 −1 src/dbSta/test/cpp/TestDbSta.cc
+7 −0 src/dbSta/test/hier_highest_connected_net.ok
+97 −0 src/dbSta/test/hier_highest_connected_net.spef
+68 −0 src/dbSta/test/hier_highest_connected_net.tcl
+32 −0 src/dbSta/test/hier_highest_connected_net.v
+113 −33 src/rsz/src/MoveCommitter.cc
+9 −3 src/rsz/src/MoveCommitter.hh
+61 −28 src/rsz/src/MoveTracker.cc
+25 −0 src/rsz/src/MoveTracker.hh
+4 −8 src/rsz/src/policy/SetupLegacyBase.cc
+25 −0 src/rsz/test/BUILD
+30 −0 src/rsz/test/cpp/CMakeLists.txt
+367 −0 src/rsz/test/cpp/TestNestedJournal.cc
+1 −0 third-party/gif-h/BUILD.bazel
+1 −0 third-party/lodepng/BUILD
+1 −0 third-party/stb_truetype/BUILD
+12 −0 tools/lint/BUILD.bazel
+14 −0 tools/lint/linters.bzl