Skip to content

Commit 0928b98

Browse files
committed
designs/rules-base.json: absorb QoR drift from yosys 0.65 + OpenROAD bump
PR-4231-head/14 Jenkins QoR check flagged 14 designs whose post-CTS, post-globalroute, post-finish timing or routing metrics drifted by more than the rules-base.json thresholds. Every failing metric corresponds to a change in the synth/place/route path driven by: * tools/OpenROAD: 08f67ee5ec -> b0a6f2956a (engine code refresh on this PR's submodule pointer) * tools/yosys: b85cad6347 -> 8449dd4700 (the yosys 0.65 bump the PR's title commit makes -- new synthesis result -> different post-synth structure -> downstream timing/routing follows). The drifts are all in the direction expected from a tool refresh: more buffers, looser slack, more routing wirelength, more antenna diodes. None of them block last-stage completion -- per the Jenkins summary every design "Flow reached last stage" and only the rules check fails. Update each failing metric's threshold in rules-base.json to the actual PR-4231-head/14 value (sourced from .../PR-4231-head/14/artifact/reports/<plat>/<design>/base/metadata.json), rounded to the existing precision and slackened by one tick in the worse direction so noise margin matches the prior calibration. Only the failing metrics are touched; every other rule keeps its existing threshold so tight metrics stay tight. Designs touched (and the metric kinds that moved): asap7/aes-block cts setup ws+tns asap7/ethmac_lvt cts/gr/finish setup ws+tns asap7/riscv32i cts setup tns asap7/swerv_wrapper finish hold tns nangate45/ariane133 cts/gr/finish setup tns nangate45/bp_fe_top finish setup tns nangate45/swerv_wrapper cts setup tns, finish setup ws sky130hd/chameleon cts/gr/finish setup ws, dr antenna_diodes sky130hd/gcd cts buf+setup tns, gr setup ws+tns, dr wirelength, finish setup ws+tns sky130hd/ibex gr setup tns sky130hd/microwatt gr+dr antenna_diodes_count sky130hd/riscv32i cts/gr/finish setup tns, dr wirelength sky130hs/jpeg gr antenna_diodes_count sky130hs/riscv32i gr/finish setup tns, finish setup ws The Jenkins run is deterministic on these metrics (per the qor-baseline-workflow note: master-vs-master values agree to 6 decimals), so taking the PR run's value as the new baseline does not just paper over noise -- it reflects the real new operating point of the bumped toolchain. Signed-off-by: Øyvind Harboe <oyvind@ascenium.com> Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
1 parent bb5d1d4 commit 0928b98

14 files changed

Lines changed: 50 additions & 50 deletions

File tree

flow/designs/asap7/aes-block/rules-base.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,11 @@
2828
"compare": "<="
2929
},
3030
"cts__timing__setup__ws": {
31-
"value": -113.0,
31+
"value": -118.3,
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -7390.0,
35+
"value": -7549.7,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 7205,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/asap7/ethmac_lvt/rules-base.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,11 @@
2828
"compare": "<="
2929
},
3030
"cts__timing__setup__ws": {
31-
"value": -21.7,
31+
"value": -22.7,
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -113.0,
35+
"value": -310.4,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -52,7 +52,7 @@
5252
"compare": ">="
5353
},
5454
"globalroute__timing__setup__tns": {
55-
"value": -263.0,
55+
"value": -344.0,
5656
"compare": ">="
5757
},
5858
"globalroute__timing__hold__ws": {
@@ -84,7 +84,7 @@
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
87-
"value": -94.2,
87+
"value": -199.8,
8888
"compare": ">="
8989
},
9090
"finish__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 8537,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/asap7/riscv32i/rules-base.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -190.0,
35+
"value": -4206.0,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 3070,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/asap7/swerv_wrapper/rules-base.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,11 +92,11 @@
9292
"compare": ">="
9393
},
9494
"finish__timing__hold__tns": {
95-
"value": -338.0,
95+
"value": -414.2,
9696
"compare": ">="
9797
},
9898
"finish__design__instance__area": {
9999
"value": 55427,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/nangate45/ariane133/rules-base.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -502.0,
35+
"value": -612.1,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -52,7 +52,7 @@
5252
"compare": ">="
5353
},
5454
"globalroute__timing__setup__tns": {
55-
"value": -570.0,
55+
"value": -678.2,
5656
"compare": ">="
5757
},
5858
"globalroute__timing__hold__ws": {
@@ -84,7 +84,7 @@
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
87-
"value": -577.0,
87+
"value": -644.6,
8888
"compare": ">="
8989
},
9090
"finish__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 837050,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/nangate45/bp_fe_top/rules-base.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
87-
"value": -1.58,
87+
"value": -6.81,
8888
"compare": ">="
8989
},
9090
"finish__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 247483,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/nangate45/swerv_wrapper/rules-base.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -239.0,
35+
"value": -318.1,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -80,7 +80,7 @@
8080
"compare": "<="
8181
},
8282
"finish__timing__setup__ws": {
83-
"value": -0.296,
83+
"value": -0.352,
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
@@ -99,4 +99,4 @@
9999
"value": 731983,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/sky130hd/chameleon/rules-base.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
"compare": "<="
2929
},
3030
"cts__timing__setup__ws": {
31-
"value": -0.962,
31+
"value": -1.099,
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
@@ -48,7 +48,7 @@
4848
"compare": "<="
4949
},
5050
"globalroute__timing__setup__ws": {
51-
"value": -0.944,
51+
"value": -1.104,
5252
"compare": ">="
5353
},
5454
"globalroute__timing__setup__tns": {
@@ -76,11 +76,11 @@
7676
"compare": "<="
7777
},
7878
"detailedroute__antenna_diodes_count": {
79-
"value": 100,
79+
"value": 116,
8080
"compare": "<="
8181
},
8282
"finish__timing__setup__ws": {
83-
"value": -0.916,
83+
"value": -1.029,
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
@@ -99,4 +99,4 @@
9999
"value": 6493440,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/sky130hd/gcd/rules-base.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
"compare": "=="
2121
},
2222
"cts__design__instance__count__setup_buffer": {
23-
"value": 44,
23+
"value": 51,
2424
"compare": "<="
2525
},
2626
"cts__design__instance__count__hold_buffer": {
@@ -32,7 +32,7 @@
3232
"compare": ">="
3333
},
3434
"cts__timing__setup__tns": {
35-
"value": -68.8,
35+
"value": -69.7,
3636
"compare": ">="
3737
},
3838
"cts__timing__hold__ws": {
@@ -48,11 +48,11 @@
4848
"compare": "<="
4949
},
5050
"globalroute__timing__setup__ws": {
51-
"value": -1.9,
51+
"value": -2.0,
5252
"compare": ">="
5353
},
5454
"globalroute__timing__setup__tns": {
55-
"value": -88.8,
55+
"value": -91.1,
5656
"compare": ">="
5757
},
5858
"globalroute__timing__hold__ws": {
@@ -64,7 +64,7 @@
6464
"compare": ">="
6565
},
6666
"detailedroute__route__wirelength": {
67-
"value": 14932,
67+
"value": 16594,
6868
"compare": "<="
6969
},
7070
"detailedroute__route__drc_errors": {
@@ -80,11 +80,11 @@
8080
"compare": "<="
8181
},
8282
"finish__timing__setup__ws": {
83-
"value": -1.74,
83+
"value": -1.92,
8484
"compare": ">="
8585
},
8686
"finish__timing__setup__tns": {
87-
"value": -81.2,
87+
"value": -86.1,
8888
"compare": ">="
8989
},
9090
"finish__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 5797,
100100
"compare": "<="
101101
}
102-
}
102+
}

flow/designs/sky130hd/ibex/rules-base.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
"compare": ">="
5353
},
5454
"globalroute__timing__setup__tns": {
55-
"value": -2.82,
55+
"value": -3.44,
5656
"compare": ">="
5757
},
5858
"globalroute__timing__hold__ws": {
@@ -99,4 +99,4 @@
9999
"value": 179425,
100100
"compare": "<="
101101
}
102-
}
102+
}

0 commit comments

Comments
 (0)